[PATCH] D89447: [MachineInstr] Add support for instructions with multiple memory operands.

Thu Oct 15 05:53:23 PDT 2020

hliao added a comment.

In D89447#2331763 <https://reviews.llvm.org/D89447#2331763>, @dmgreen wrote:

> This looks familiar. It is the same as https://reviews.llvm.org/D80161? That patch was apparently reverted in 65cd2c7a8015577fea15c861f41d2e4b5768961f <https://reviews.llvm.org/rG65cd2c7a8015577fea15c861f41d2e4b5768961f>, because it was hitting timeouts on certain targets.
>
> I'm not sure if @Kayjukh remembers more? But it may be worth putting a limit on the total number of alias checks it can perform.
>
> The test changes seem fine to me.

Sorry, don't search for the relevant change. Yes, it's exactly the same purpose with minor differences on check conditions.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89447/new/

https://reviews.llvm.org/D89447