[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