[PATCH] D91345: [AArch64] Fix rottype use in complex instr defs.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 8 13:09:22 PST 2020


fhahn added a comment.

In D91345#2393191 <https://reviews.llvm.org/D91345#2393191>, @dmgreen wrote:

> Like I said on the other review, sometimes reviews can be split up too much into things that are not even testable.

Agreed, this is probably why it was broken in the first place. I have patches on top of this one, which actually instantiate the pattern and work as expected. Unfortunately they will need a bit more discussing, so I think it's best to get this in independently.

> This look simple enough on it's own though. LGTM

Thank you very much for taking a look!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91345



More information about the llvm-commits mailing list