The patch implements microMIPS CACHEE and PREFE instructions.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
LGTM with some nits.
| lib/Target/Mips/MCTargetDesc/MipsMCCodeEmitter.cpp | ||
|---|---|---|
| 773–774 ↗ | (On Diff #30997) | Indentation |
| lib/Target/Mips/MCTargetDesc/MipsMCCodeEmitter.h | ||
| 176–177 ↗ | (On Diff #30997) | Indentation |
| lib/Target/Mips/MicroMipsInstrFormats.td | ||
| 925 ↗ | (On Diff #30997) | This doesn't follow the naming convention but that's ok since microMIPS in general hasn't migrated yet. |
| lib/Target/Mips/MicroMipsInstrInfo.td | ||
| 110 ↗ | (On Diff #30997) | Shouldn't it be simm9? |