[PATCH] D40911: [OpenMP] Fix PR35542: Correct adjusting of private reduction variable

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 6 11:07:32 PST 2017


ABataev added inline comments.


================
Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:1110
+            PrivateAddr.getPointer(),
+            SharedAddresses[N].first.getPointer()->getType());
+    llvm::Value *Ptr = CGF.Builder.CreateGEP(PrivatePointer, Adjustment);
----------------
Hahnfeld wrote:
> ABataev wrote:
> > Better to use `SharedAddresses[N].first.getAddress().getType()`
> Ok, was copied from below call to `castToBase()`. Should I adjust it there as well?
Yes, please


Repository:
  rC Clang

https://reviews.llvm.org/D40911





More information about the cfe-commits mailing list