[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