Hahnfeld added inline comments.

================
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:6265-6268
+void CodeGenModule::printPostfixForExternalizedStaticVar(
+    llvm::raw_ostream &OS) const {
+  OS << ".static." << getContext().getCUIDHash();
+}
----------------
I've tried to use this with CUDA, but it errors out because `.` is not allowed 
in identifiers. Could you check if https://reviews.llvm.org/D108456 also works 
for HIP?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85223

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D85223: [CUDA][HIP]... Jonas Hahnfeld via Phabricator via cfe-commits

Reply via email to