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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
