Skip to content

[CIR] Merge two copies of CIRGenAction.h #1085

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Nov 8, 2024

Conversation

smeenai
Copy link
Collaborator

@smeenai smeenai commented Nov 8, 2024

We have both clang/include/clang/CIRFrontendAction/CIRGenAction.h and
clang/include/clang/cir/FrontendAction/CIRGenAction.h, which is a
historical artifact. The latter is what's being upstreamed, so merge the
former into it to avoid any confusion.

Copy link

github-actions bot commented Nov 8, 2024

✅ With the latest revision this PR passed the C/C++ code formatter.

@smeenai
Copy link
Collaborator Author

smeenai commented Nov 8, 2024

This is manually stacked on top of the namespace merging, I'll land it once that goes through.

@smeenai smeenai force-pushed the users/smeenai/mlir-cir-namespace-merge branch 2 times, most recently from 0a2410b to 9d758ef Compare November 8, 2024 18:55
Base automatically changed from users/smeenai/mlir-cir-namespace-merge to main November 8, 2024 18:56
We have both clang/include/clang/CIRFrontendAction/CIRGenAction.h and
clang/include/clang/cir/FrontendAction/CIRGenAction.h, which is a
historical artifact. The latter is what's being upstreamed, so merge the
former into it to avoid any confusion.
@smeenai smeenai force-pushed the users/smeenai/action-header-move branch from fd44d5a to a9165ad Compare November 8, 2024 19:00
@smeenai smeenai merged commit 2fc3593 into main Nov 8, 2024
6 checks passed
@smeenai smeenai deleted the users/smeenai/action-header-move branch November 8, 2024 20:10
lanza pushed a commit that referenced this pull request Mar 18, 2025
We have both clang/include/clang/CIRFrontendAction/CIRGenAction.h and
clang/include/clang/cir/FrontendAction/CIRGenAction.h, which is a
historical artifact. The latter is what's being upstreamed, so merge the
former into it to avoid any confusion.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants