[llvm-dev] [InstCombine] Addrspacecast and GEP assumed commutative

Matt Arsenault via llvm-dev llvm-dev at lists.llvm.org
Tue Mar 24 07:23:01 PDT 2020

> On Mar 24, 2020, at 08:50, Thomas Faingnaert <thomas.faingnaert at hotmail.com> wrote:
> It appears that this behaviour of InstCombine is at least somewhat intended, as there are several tests that fail after the change mentioned before: cast.ll,gep-addrspace.ll and getelementptr.ll in test/Transforms/InstCombine.
> I feel a little uncomfortable applying the patch after knowing this, and removing those tests doesn't seem like a great solution.
> What are your thoughts?

These are checking for the current behavior, it’s only natural tests would need updating for this change. You should just update them to show the new result

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20200324/16bf6f64/attachment.html>

More information about the llvm-dev mailing list