[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