[PATCH] D10836: [ConstFolding] Allow constfolding of llvm.sin.* and llvm.cos.* intrinsics

Michael Zolotukhin mzolotukhin at apple.com
Tue Jul 7 11:39:35 PDT 2015


mzolotukhin added a subscriber: mzolotukhin.
mzolotukhin accepted this revision.
mzolotukhin added a reviewer: mzolotukhin.

This revision is now accepted and ready to land.

Looks good to me.


================
Comment at: test/Transforms/InstCombine/intrinsics.ll:22-23
@@ -20,1 +21,4 @@
+declare double @llvm.sin.f64(double %Val) nounwind readonly
+
+
 
----------------
Please don't add empty lines.

================
Comment at: test/Transforms/InstCombine/intrinsics.ll:452
@@ +451,2 @@
+}
+
----------------
Another unnecessary empty line.


http://reviews.llvm.org/D10836







More information about the llvm-commits mailing list