Instructions: crc32b, crc32h, crc32w, crc32d,
crc32cb, crc32ch, crc32cw, crc32cd
Assembler directives: .set crc, .set nocrc, .module crc, .module nocrc
Attribute: crc
.MIPS.abiflags: CRC (0x8000)
Paths
| Differential D44176
[mips] Add support for CRC ASE ClosedPublic Authored by vstefanovic on Mar 6 2018, 2:13 PM.
Details Summary Instructions: crc32b, crc32h, crc32w, crc32d, crc32cb, crc32ch, crc32cw, crc32cd Assembler directives: .set crc, .set nocrc, .module crc, .module nocrc
Diff Detail
Event TimelineComment Actions This looks mostly ok. There's only some small changes required, and they are somewhat minor. The recurring change is that for test cases, when there is a run-on line with '\', then the continuation of the command line should be indented by two spaces. See my inlined comments.
This revision now requires changes to proceed.Mar 7 2018, 7:46 AM
vstefanovic retitled this revision from [mips] Add support for CRC ASE. to [mips] Add support for CRC ASE. vstefanovic added inline comments.
sdardis added a parent revision: D44299: [mips] Split out ASEPredicate from InsnPredicates (NFC).Mar 9 2018, 5:20 AM Comment Actions If you look at the dependency, you'll see the way I've separated out the ISA level from the ASEs. vstefanovic marked an inline comment as done. Comment ActionsUpdated the code to use ASEPRedicate from D44299. Comment Actions LGTM with inline nits addressed.
This revision is now accepted and ready to land.Mar 13 2018, 4:14 AM Closed by commit rL327511: [mips] Add support for CRC ASE (authored by petarj). · Explain WhyMar 14 2018, 7:17 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 137628 include/llvm/Support/MipsABIFlags.h
lib/Target/Mips/AsmParser/MipsAsmParser.cpp
lib/Target/Mips/Disassembler/MipsDisassembler.cpp
lib/Target/Mips/MCTargetDesc/MipsABIFlagsSection.h
lib/Target/Mips/MCTargetDesc/MipsTargetStreamer.cpp
lib/Target/Mips/Mips.td
lib/Target/Mips/Mips32r6InstrFormats.td
lib/Target/Mips/Mips32r6InstrInfo.td
lib/Target/Mips/Mips64r6InstrInfo.td
lib/Target/Mips/MipsInstrInfo.td
lib/Target/Mips/MipsSchedule.td
lib/Target/Mips/MipsSubtarget.h
lib/Target/Mips/MipsSubtarget.cpp
lib/Target/Mips/MipsTargetStreamer.h
test/MC/Disassembler/Mips/crc/valid-32r6-el.txt
test/MC/Disassembler/Mips/crc/valid-32r6.txt
test/MC/Disassembler/Mips/crc/valid-64r6-el.txt
test/MC/Disassembler/Mips/crc/valid-64r6.txt
test/MC/Mips/crc/invalid.s
test/MC/Mips/crc/invalid64.s
test/MC/Mips/crc/module-crc.s
test/MC/Mips/crc/module-nocrc.s
test/MC/Mips/crc/set-crc-directive.s
test/MC/Mips/crc/set-nocrc-directive.s
test/MC/Mips/crc/valid.s
test/MC/Mips/crc/valid64.s
tools/llvm-readobj/ELFDumper.cpp
|
Newline before and after what you've added here.