Probably qualifies as obvious. Also can you add some text to the assert while you're there?
On Fri, Jan 22, 2016, 5:07 PM Justin Lebar <jle...@google.com> wrote: > jlebar created this revision. > jlebar added a reviewer: tra. > jlebar added subscribers: cfe-commits, jhen, echristo. > > No functional changes. > > http://reviews.llvm.org/D16495 > > Files: > lib/Driver/Tools.cpp > > Index: lib/Driver/Tools.cpp > =================================================================== > --- lib/Driver/Tools.cpp > +++ lib/Driver/Tools.cpp > @@ -10651,8 +10651,7 @@ > const char *LinkingOutput) const { > const auto &TC = > static_cast<const toolchains::CudaToolChain &>(getToolChain()); > - assert(TC.getArch() == llvm::Triple::nvptx || > - TC.getArch() == llvm::Triple::nvptx64); > + assert(TC.getTriple().isNVPTX()); > > std::vector<std::string> gpu_archs = > Args.getAllArgValues(options::OPT_march_EQ); > @@ -10720,8 +10719,7 @@ > const char *LinkingOutput) const { > const auto &TC = > static_cast<const toolchains::CudaToolChain &>(getToolChain()); > - assert(TC.getArch() == llvm::Triple::nvptx || > - TC.getArch() == llvm::Triple::nvptx64); > + assert(TC.getTriple().isNVPTX()); > > ArgStringList CmdArgs; > CmdArgs.push_back("--cuda"); > > >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
