[PATCH] D84260: [OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 3

Saiyedul Islam via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 24 22:50:49 PDT 2020


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


================
Comment at: clang/lib/CodeGen/CGExpr.cpp:129
+  auto *Alloca = Var.getPointer();
+  assert(isa<llvm::AllocaInst>(Alloca) ||
+         (isa<llvm::AddrSpaceCastInst>(Alloca) &&
----------------
@arsenm , it wasn't possible to post D78495 and this patch separately due to their interdependence. Test "clang/test/OpenMP/amdgcn_target_init_temp_alloca.cpp" below checks for this change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84260





More information about the cfe-commits mailing list