It seems like the order here is wrong. Types like i32 do not take any
arguments.
Currently this is not a problem, because the patterns are not actually
used with any nodes, but will fail once it is used with real ISD nodes.
Paths
| Differential D91345
[AArch64] Fix rottype use in complex instr defs. ClosedPublic Authored by fhahn on Nov 12 2020, 6:25 AM.
Details Summary It seems like the order here is wrong. Types like i32 do not take any Currently this is not a problem, because the patterns are not actually
Diff Detail
Unit TestsFailed
Event TimelineComment Actions Like I said on the other review, sometimes reviews can be split up too much into things that are not even testable. This look simple enough on it's own though. LGTM This revision is now accepted and ready to land.Nov 13 2020, 12:46 AM Comment Actions
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.
Thank you very much for taking a look! This revision was landed with ongoing or failed builds.Dec 8 2020, 1:12 PM Closed by commit rG4c69b1b98a9a: [AArch64] Fix rottype use in complex instr defs. (authored by fhahn). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 304815 llvm/lib/Target/AArch64/AArch64InstrFormats.td
|