[PATCH] D133120: [OpenMP][Opt] Fix target region without parallel
Gheorghe-Teodor Bercea via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 2 12:21:24 PDT 2022
doru1004 marked 3 inline comments as done.
doru1004 added a comment.
Updated the patch to show all the comments have been addressed.
@jdoerfert if you could have a look again this time at the changes to the tests triggered by the removal of the conditional as follows:
For the spmdization test I had to add different prefixes to get the update script to not complain about conflicting code for the same CHECK and also prevent it from removing the generic state machine checks.
For the single threaded test I had to add a missing exec_mode global variable. The compiler was failing due to exec_mode variable missing (assert fail).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133120/new/
https://reviews.llvm.org/D133120
More information about the llvm-commits
mailing list