[PATCH] D89844: [Clang][OpenMP] Fixed an issue of segment fault when using target nowait

Shilei Tian via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 5 17:08:39 PST 2021


tianshilei1992 added inline comments.


================
Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:9670
                                  Info.MapTypesArray, Info.MappersArray, Info,
-                                 {/*ForEndTask=*/false, HasDependClauses});
+                                 {/*ForEndTask=*/false, RequiresOuterTask});
     InputInfo.NumberOfTargetItems = Info.NumberOfPtrs;
----------------
mikerice wrote:
> It seems this change is no longer in the compiler, as it was reverted by the next commit to this file.  Was that intentional?  See: https://reviews.llvm.org/D90101.
Yeah, it is not needed anymore.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89844/new/

https://reviews.llvm.org/D89844



More information about the cfe-commits mailing list