[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