[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