[llvm] LAA: rework and rename stripGetElementPtr (PR #125315)

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 17 03:43:55 PST 2025


https://github.com/fhahn commented:

> > The title/description sound like this would just be updating comments ("clarify"), but the diff is code changes. If this is a refactoring/cleanup, please add NFC and update the title/description. It should start with what it is doing (the "Strip getGEPInductionOperand, rework and rename stripGetElementPtr" part), then elaborate the motivation for it.
> 
> I've updated the title, and the patch is not exactly a NFC, but I'm not sure it would be wise to add a test as @nikic is working on canonicalizing GEPs with variable offsets.

If it is possible to add a test to cover the new behavior it would be good to clarify the functional impact and make sure we don't accidentally regress it. I don't think the test would hinder any other efforts?

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


More information about the llvm-commits mailing list