yaxunl marked an inline comment as done.
yaxunl added inline comments.
================
Comment at: lib/Sema/SemaDeclAttr.cpp:4620
const TargetInfo &TI = Context.getTargetInfo();
- TargetInfo::CallingConvCheckResult A = TI.checkCallingConvention(CC);
+ auto *Aux = Context.getAuxTargetInfo();
+ if (LangOpts.CUDA) {
----------------
yaxunl wrote:
> rjmccall wrote:
> > Please sink these declarations into the CUDA-specific block. Also, please
> > add some comments to explain why different logic is needed for CUDA.
> will do. thanks.
variables A and TI are also used by else part of the if statement, so only
variable Aux can be moved in the if part.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57716/new/
https://reviews.llvm.org/D57716
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
