zsrkmyn marked an inline comment as done.
zsrkmyn added inline comments.
================
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:3005
/*ForVTable=*/false);
+ auto Linkage = (FD->isCPUDispatchMultiVersion() ||
FD->isCPUSpecificMultiVersion())
+ ? llvm::Function::LinkOnceODRLinkage
----------------
erichkeane wrote:
> I think this can always just be LinkOnceODR.
Changing this also changes linkage for attribute(target()), should I also
change test cases for them? (including test/CodeGen{,CXX}/attr-*.ll)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67058/new/
https://reviews.llvm.org/D67058
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
