[PATCH] D43515: More math intrinsics for conservative math handling

[Kevin P. Neal via Phabricator via llvm-commits]

kpn marked 6 inline comments as done.
kpn added inline comments.


================
Comment at: lib/CodeGen/StrictFP.cpp:171
+
+  auto *t = cast<IntegerType>(I->getType());
+  APInt IntMaxAP(APInt::getSignedMinValue(DL->getTypeStoreSize(t) * 8));
----------------
craig.topper wrote:
> What if the intrinsic uses a vector type?
It would have been caught by the IR verifier. A vector would have been rejected there.


https://reviews.llvm.org/D43515