-
Notifications
You must be signed in to change notification settings - Fork 13.4k
[DLCov] Move DebugLoc coverage macro to llvm-config.h #137787
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks! 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for fixing this!
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/11/builds/14115 Here is the relevant piece of the build log for the reference
|
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/146/builds/2817 Here is the relevant piece of the build log for the reference
|
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/161/builds/5529 Here is the relevant piece of the build log for the reference
|
This patch follows the reversion of llvm#107279, which caused errors by including `llvm/Config/config.h` in a public header. In order to reapply that patch, this PR moves the definition of the config option to the `llvm-config.h` header instead, so that it can be used publicly. This patch also adds `LLVM_` as a prefix to the define, since it is now in a public header.
This has been defined in llvm#137787 and affects in llvmorg-21-init-10310-g92195f6fc873 (llvm#137787).
This patch follows the reversion of llvm#107279, which caused errors by including `llvm/Config/config.h` in a public header. In order to reapply that patch, this PR moves the definition of the config option to the `llvm-config.h` header instead, so that it can be used publicly. This patch also adds `LLVM_` as a prefix to the define, since it is now in a public header.
This has been defined in llvm#137787 and affects in llvmorg-21-init-10310-g92195f6fc873 (llvm#137787).
This patch follows the reversion of llvm#107279, which caused errors by including `llvm/Config/config.h` in a public header. In order to reapply that patch, this PR moves the definition of the config option to the `llvm-config.h` header instead, so that it can be used publicly. This patch also adds `LLVM_` as a prefix to the define, since it is now in a public header.
This has been defined in llvm#137787 and affects in llvmorg-21-init-10310-g92195f6fc873 (llvm#137787).
This patch follows the reversion of llvm#107279, which caused errors by including `llvm/Config/config.h` in a public header. In order to reapply that patch, this PR moves the definition of the config option to the `llvm-config.h` header instead, so that it can be used publicly. This patch also adds `LLVM_` as a prefix to the define, since it is now in a public header.
This has been defined in llvm#137787 and affects in llvmorg-21-init-10310-g92195f6fc873 (llvm#137787).
This patch follows the reversion of #107279, which caused errors by including
llvm/Config/config.h
in a public header. In order to reapply that patch, this PR moves the definition of the config option to thellvm-config.h
header instead, so that it can be used publicly. This patch also addsLLVM_
as a prefix to the define, since it is now in a public header.