This allows to return the static value that we know at compile time.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
Now that this issue has been addressed and regressions tests pass, should we re-enable Cmake to build libomptarget by default?
Comment Actions
Yes, I already have a local patch which also takes care of restricting the tests to Clang versions newer than 6.0.0. I will post it for review once I've committed this revision.
Comment Actions
@jdoerfert suggested replacing this with a context selector. One less special case in clang.
Related, if this is a builtin (or in header), I think we can and should delete it from the devicertl. It's probably used as a branch condition so replacing with a 0/1 asap is winning for compilation time.