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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to