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

Matt Arsenault via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 12 14:18:49 PDT 2016


arsenm added inline comments.

================
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;
+  }
+
----------------
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


http://reviews.llvm.org/D18473





More information about the cfe-commits mailing list