Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
LGTM with a nit, hwr_ulr added, and the FIXME comments about the .set directives.
lib/Target/Mips/AsmParser/MipsAsmParser.cpp | ||
---|---|---|
1760 ↗ | (On Diff #14843) | Could you add 'hwr_ulr' ($29) too? GAS doesn't accept it at the moment, but it's in the spec. |
2255 ↗ | (On Diff #14843) | Nit: Multiple blank lines |
test/MC/Mips/mips-hwr-register-names.s | ||
6–9 ↗ | (On Diff #14843) | Could you add a FIXME comment about the .set lines? Eventually, I'd like the instruction printer for rdhwr to stop printing the .set directives around it (this doesn't need to be done in this patch). It makes some sense for CodeGen to do emit them but not for the assembler. |
test/MC/Mips/mips32r2/valid.s | ||
140–143 ↗ | (On Diff #14843) | I'd like to remove the checks for the .set directives but the encoding is on the '.set pop' line so we'd have to keep that for the moment. It would look strange to have a .set pop without a .set push so let's leave the .set directives in the test for now. Could you add a FIXME comment about them? |