[llvm] [LAA] Perform checks for no-wrap separately from getPtrStride. (PR #126971)

[Florian Hahn via llvm-commits]

https://github.com/fhahn commented:

Thanks for taking a look!

> Effectively, isNoWrap now duplicates code from getPtrStride. Can that functionalty somehow refactored into a common function?

yes at the moment it duplicates a bit of code, mostly initial construction of the AddRec. At the moment, I am not sure if there is a good way to refactor that without introducing additional indirections that make it more difficult to see what's going on vs some small duplication.

Some duplication I am planning to remove as a next step, by passing a scene directly to `isNoWrap`, as there are some cases where it is currently not used because no pointer is available. Will share a patch once I managed to create a test case.

https://github.com/llvm/llvm-project/pull/126971