[clang] [OpenMP][CodeGen] Improved codegen for combined loop directives (PR #72417)
David Pagan via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 11 14:47:36 PDT 2024
ddpagan wrote:
After some additional discussion with Alexey offline, he concluded that the current changes are okay, specifically for this reason:
_"Then I realized that actually it does not require AST nodes building. In this case, this helper class should be moved to CodeGenStmt and hidden in the anonymous namespace. But you also need to use it in CodeGenModule. In this case better to use a flag in statement, as you have it right now. I.e. having this analysis in Sema looks good_"
https://github.com/llvm/llvm-project/pull/72417
More information about the cfe-commits
mailing list