[llvm] [OpenMP] Remove dependency on LLVM include directory from DeviceRTL (PR #136359)

Krzysztof Parzyszek via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 21 10:19:57 PDT 2025


kparzysz wrote:

> but there's no reason to make `openmp/` depend on LLVM headers just for a single constant `1` value.

There is definitely a reason, and that's to avoid hardcoding things.

There are precedents where definitions of certain enumerations are duplicated.  You could follow that---copy/paste the definition of the one that's introducing the dependence with a comment that it must be in sync with the one in LLVM.

https://github.com/llvm/llvm-project/pull/136359


More information about the llvm-commits mailing list