[clang] [llvm] Assume (PR #97535)

Krzysztof Parzyszek via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 10 07:46:07 PDT 2024


================
@@ -760,6 +760,9 @@ void clang::getOpenMPCaptureRegions(
     case OMPD_parallel:
       CaptureRegions.push_back(OMPD_parallel);
       break;
+    case OMPD_assume:
+      CaptureRegions.push_back(OMPD_unknown);
+      break;
----------------
kparzysz wrote:

This is what happens with the directives at lines 790+ (before these changes).  We don't want to be adding OMPD_unknown as we go, because it can end up intermingled with other directives (in case of combined directives).  I understand that OMPD_assumes may be standalone, but we should treat it the same as the combinable ones for consistency.

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


More information about the llvm-commits mailing list