[PATCH] D61691: [InstSimplify] fold fcmp (minnum, X, C1), C2

Cameron McInally via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 15 08:12:54 PDT 2019


cameron.mcinally accepted this revision.
cameron.mcinally added a comment.

Corner cases look good, so LGTM.



================
Comment at: llvm/lib/Analysis/InstructionSimplify.cpp:3463
+        llvm_unreachable("Unexpected fcmp predicate");
+    }
+    }
----------------
Looks like bracket doesn't line up with switch.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61691/new/

https://reviews.llvm.org/D61691





More information about the llvm-commits mailing list