[PATCH] D17516: AMDGPU: Verify subtarget specific builtins
Eric Christopher via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 7 14:45:37 PDT 2016
echristo accepted this revision.
echristo added a comment.
This revision is now accepted and ready to land.
Seems pretty reasonable. If I missed anything we can pick it up later.
-eric
================
Comment at: lib/Basic/Targets.cpp:2010
@@ +2009,3 @@
+
+ // XXX - What does the member GPU mean if device name string passed here?
+ if (getTriple().getArch() == llvm::Triple::amdgcn) {
----------------
Means it was either passed in via -mcpu or is set on the function via an attribute.
http://reviews.llvm.org/D17516
More information about the cfe-commits
mailing list