[PATCH] D18473: AMDGPU: Add test for generic builtin behavior

Tom Stellard via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 13 05:58:48 PDT 2016


tstellarAMD accepted this revision.
This revision is now accepted and ready to land.

================
Comment at: lib/Basic/Targets.cpp:1854-1857
@@ -1853,1 +1853,6 @@
 
+  bool isCLZForZeroUndef() const override {
+    // It is -1 instead of expected for intrinsic.
+    return true;
+  }
+
----------------
arsenm wrote:
> tstellarAMD wrote:
> > Why do we need to add this if it's the same as the default?
> Don't really, other than it adds a comment. I can keep just the test
Ok, sounds good. LGTM.


http://reviews.llvm.org/D18473





More information about the cfe-commits mailing list