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

[Matt Arsenault via llvm-dev]

> 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

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