[PATCH] D89671: [LLVM][OpenMP] Adding support for OpenMP sections construct in OpenMPIRBuilder
Chirag Khandelwal via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 29 22:36:21 PDT 2021
AMDChirag added inline comments.
================
Comment at: llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp:2236
+ EXPECT_EQ(OutlinedFn->arg_size(), 1U);
+ EXPECT_EQ(OutlinedFn->getBasicBlockList().size(), 11);
+
----------------
MaskRay wrote:
> I fixed a -Wsign-compare here. Will be great to test the change with clang:)
>
Thank you very much for the fix and the kind words :)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D89671/new/
https://reviews.llvm.org/D89671
More information about the llvm-commits
mailing list