[PATCH] D63277: [CUDA][HIP] Don't set "comdat" attribute for CUDA device stub functions.

Konstantin Pyzhov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 19 11:06:37 PDT 2019


kpyzhov added inline comments.


================
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:4294
   setNonAliasAttributes(GD, Fn);
   SetLLVMFunctionAttributesForDefinition(D, Fn);
   if (const ConstructorAttr *CA = D->getAttr<ConstructorAttr>())
----------------
tra wrote:
> Perhaps this should be pushed further down into `shouldBeInCOMDAT()` which seems to be the right place to decide whether something is not suitable for comdat.
Good idea. Let me do that.


Repository:
  rC Clang

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

https://reviews.llvm.org/D63277





More information about the cfe-commits mailing list