[PATCH] D10700: Driver: Add AMDGPU toolchain
    Douglas Katzman 
    dougk at google.com
       
    Mon Jul  6 07:52:58 PDT 2015
    
    
  
dougk added a subscriber: dougk.
================
Comment at: lib/Driver/ToolChains.h:726
@@ -725,1 +725,3 @@
 
+class LLVM_LIBRARY_VISIBILITY AMDGPU_TC : public Generic_ELF {
+protected:
----------------
Please use ToolChain instead of _TC as the suffix.
See http://reviews.llvm.org/D10609 (pending) which standardizes on that.
================
Comment at: lib/Driver/Tools.cpp:5759
@@ +5758,3 @@
+  CmdArgs.push_back("-target");
+  CmdArgs.push_back("amdgcn--");
+  AddLinkerInputs(getToolChain(), Inputs, Args, CmdArgs);
----------------
any reason not to insert getToolChain().getTripleString() vs hardcoding 'amdgcn--' ?
http://reviews.llvm.org/D10700
    
    
More information about the cfe-commits
mailing list