rjmccall 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:
> 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.
Yes, sorry, only your new declaration needs to be sunk.
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
