[PATCH] D62703: [CVP] Simplify non-overflowing saturating add/sub

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 30 14:13:33 PDT 2019


lebedev.ri added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/CorrelatedValuePropagation.cpp:442
+  SI->eraseFromParent();
+  ++NumOverflows;
+}
----------------
Why not add a new statistic
"Number of saturating arithmetics converted to normal arithmetics"
?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62703





More information about the llvm-commits mailing list