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

[Dominik Adamski via Phabricator via cfe-commits]

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


================
Comment at: clang/lib/CodeGen/CGStmtOpenMP.cpp:2640
       if (UseOMPIRBuilder) {
+        llvm::DenseMap<llvm::Value *, llvm::Value *> AlignedVars;
         // Emit the associated statement and get its loop representation.
----------------
jdoerfert wrote:
> Where is this map populated?
I added function `GetAlignedMapping `which collects arguments for assumption calls. Now Clang collects all arguments required for assumption calls. OMPIRBuilder only inserts assumption calls.

When I was adding C integration tests I realized that it would be easier to implement in that way. Please let me know if this approach is ok for you.


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

https://reviews.llvm.org/D133578