[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