[llvm] [LAA] Make Ptr argument optional in isNoWrap. (PR #127410)
Ramkumar Ramachandra via llvm-commits
llvm-commits at lists.llvm.org
Sun Feb 16 12:16:23 PST 2025
================
@@ -1454,6 +1464,9 @@ static bool isNoWrapAddRec(Value *Ptr, const SCEVAddRecExpr *AR,
if (AR->getNoWrapFlags(SCEV::NoWrapMask))
return true;
----------------
artagnon wrote:
Also, the API isn't very clear, as AR is only ever used here, and what's actually checked is the Ptr, not AR. Perhaps the function should be renamed to isNoWrapGEP, and the AR check can be moved to the caller, assuming that it's useful?
https://github.com/llvm/llvm-project/pull/127410
More information about the llvm-commits
mailing list