[PATCH] D27587: Add "X / 1.0" to SimplifyFDivInst

David Majnemer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 8 12:30:48 PST 2016


majnemer added inline comments.


================
Comment at: test/Transforms/EarlyCSE/instsimplify-fdiv.ll:1-19
+; RUN: opt -O3 -S -mtriple=i686-pc-linux -print-after=early-cse < %s 2>&1 | FileCheck %s
+;
+; Check to make sure divide by 1.0 is simplified in early-cse.
+;
+
+define double @foo(double %x, i32 %n, double %e) #0 {
+entry:
----------------
I'd prefer if we replaced this test with once which was more specific in test/Transforms/InstSimplify/fdiv.ll


https://reviews.llvm.org/D27587





More information about the llvm-commits mailing list