[PATCH] D133578: [OpenMP][OMPIRBuilder] Add generation of SIMD align assumptions to OMPIRBuilder

Dominik Adamski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 11 05:08:34 PDT 2022


domada marked an inline comment as done.
domada added inline comments.


================
Comment at: llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp:1871
+  }
+}
+TEST_F(OpenMPIRBuilderTest, ApplySimdlen) {
----------------
jdoerfert wrote:
> This doesn't actually test that 2 assumes with alignments have been found, one for each alloca.
The latest change moved `alloca` handling to the Clang. That's why it is not necessary to check it here.


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

https://reviews.llvm.org/D133578



More information about the cfe-commits mailing list