Skip to content
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

Move DEBUG_TYPE below LLVM headers #44997

Merged
merged 1 commit into from
Apr 16, 2022
Merged

Move DEBUG_TYPE below LLVM headers #44997

merged 1 commit into from
Apr 16, 2022

Conversation

Keno
Copy link
Member

@Keno Keno commented Apr 15, 2022

Some LLVM headers set DEBUG_TYPE internally (and undef it later),
causing build failures with the newly added STATISTICs. Fix that
by moving DEBUG_TYPE below the headers.

[1] https://github.com/llvm/llvm-project/blob/09c2b7c35af8c4bad39f03e9f60df8bd07323028/llvm/include/llvm/Support/GenericDomTreeConstruction.h#L49

@Keno Keno requested a review from pchintalapudi April 15, 2022 19:49
Some LLVM headers set DEBUG_TYPE internally (and undef it later),
causing build failures with the newly added STATISTICs. Fix that
by moving DEBUG_TYPE below the headers.

[1] https://github.com/llvm/llvm-project/blob/09c2b7c35af8c4bad39f03e9f60df8bd07323028/llvm/include/llvm/Support/GenericDomTreeConstruction.h#L49
@vtjnash vtjnash added the status:merge me PR is reviewed. Merge when all tests are passing label Apr 15, 2022
@giordano giordano merged commit 6106e6c into master Apr 16, 2022
@giordano giordano deleted the kf/mvdbgtype branch April 16, 2022 06:31
@giordano giordano added compiler:llvm For issues that relate to LLVM and removed status:merge me PR is reviewed. Merge when all tests are passing labels Apr 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compiler:llvm For issues that relate to LLVM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants