[PATCH] D147572: [Clang][OpenMP] Fix failure with team-wide allocated variable

[Nuno Lopes via Phabricator via cfe-commits]

nlopes added inline comments.


================
Comment at: clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp:3355
+        llvm::GlobalValue::InternalLinkage,
+        CGM.getTriple().isAMDGCN() ? llvm::UndefValue::get(VarTy)
+                                   : llvm::Constant::getNullValue(VarTy),
----------------
Please use poison instead of undef wherever possible as we are tying to remove undef. The replacement is usually safe when you just need a placeholder.
Thank you!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147572