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

Thu Oct 15 00:18:57 PDT 2020

dmgreen added a subscriber: Kayjukh.
dmgreen added a comment.

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.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89447