This patch introduces microMIPSr6 as a subtarget.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
LGTM with a couple nits.
lib/Target/Mips/MCTargetDesc/MipsMCCodeEmitter.cpp | ||
---|---|---|
134 ↗ | (On Diff #22796) | Do we need to check both here? I'd expect isMicroMips(STI) to be true for all revisions of microMIPS like the way hasMips32() is true when Mips32r6() is true. |
lib/Target/Mips/MipsInstrInfo.td | ||
186 ↗ | (On Diff #22796) | Nit: indentation |
Comment Actions
In this patch the feature FeatureMicroMips32r6 is removed, isMicroMips(STI) is true for both mips32r2 and mips32r6. The type of microMIPS is determined according to processor which can be mips32r2 or mips32r6.