Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Thanks for the patch.
Have you confirmed that codegen tests are not affected? You used the same isel patterns for all of the bit instructions (btst/bset/bclr etc.) which I don't think that's correct.
It's okay to add only the MC support for certain instructions but please leave the isel pattern empty.
Comment Actions
Is this still being worked on? We're still missing the BCHG, BCLR and BSET instructions.
Comment Actions
LGTM with minor issues, thanks!
llvm/lib/Target/M68k/M68kInstrBits.td | ||
---|---|---|
51 | nit: ISel pattern is initialized to empty list by default so you don't need to put a [] here. Ditto for other places in this patch. |
nit: ISel pattern is initialized to empty list by default so you don't need to put a [] here. Ditto for other places in this patch.