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

[Jean-Michel Gorius via Phabricator via llvm-commits]

Kayjukh added a subscriber: nemanjai.
Kayjukh added a comment.

This looks like the same as my old patch (https://reviews.llvm.org/D80161). There are additional details available on the commit's Phab page <https://reviews.llvm.org/rG7019cea26dfef5882c96f278c32d0f9c49a5e516>. To put it in a nutshell, some codes can trigger a very large amount of calls to the aliasing check (see the repro provided by @nemanjai: https://pastebin.com/tRtTQdSa), which results in a very large increase in compilation time.

Bounding the number of checks may be a good solution, even though it would be nicer to have something more clever that could allow all the operands to be checked. Not sure how feasible this would be though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89447