[PATCH] D42711: AMDGPU: Support target triple OS component cuda
    Matt Arsenault via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Jan 30 14:55:36 PST 2018
    
    
  
arsenm requested changes to this revision.
arsenm added a comment.
This revision now requires changes to proceed.
You're using this just as an alias for AMDHSAOS. We shouldn't add something that behaves exactly the same
================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:5797
     llvm_unreachable("unhandled address space");
+    printf("------> unhandled address space %d\n", AS);
   }
----------------
Dead code
https://reviews.llvm.org/D42711
    
    
More information about the llvm-commits
mailing list