diff options
| author | Craig Topper <craig.topper@sifive.com> | 2025-03-17 12:49:08 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-03-17 12:49:08 -0700 |
| commit | d2e1e3034801da85c495adfffadf1de87f8b4734 (patch) | |
| tree | b864392a5699fc2fbbb721fd406c47d4b156571b /llvm/lib/Target/RISCV/Disassembler/RISCVDisassembler.cpp | |
| parent | 9eb6b37e397eba7b286777f5502a08876e74d905 (diff) | |
| download | llvm-d2e1e3034801da85c495adfffadf1de87f8b4734.zip llvm-d2e1e3034801da85c495adfffadf1de87f8b4734.tar.gz llvm-d2e1e3034801da85c495adfffadf1de87f8b4734.tar.bz2 | |
[RISCV] Rename some DecoderNamespaces and cleanup debug messages. NFC (#131409)
Rename RISCV32GPRPair and RISCV32Only_ to RV32Only. This gives a more
natural home for the P extension RV32 conflicts.
While I was there I made some improvements to the debug messages.
Diffstat (limited to 'llvm/lib/Target/RISCV/Disassembler/RISCVDisassembler.cpp')
| -rw-r--r-- | llvm/lib/Target/RISCV/Disassembler/RISCVDisassembler.cpp | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/llvm/lib/Target/RISCV/Disassembler/RISCVDisassembler.cpp b/llvm/lib/Target/RISCV/Disassembler/RISCVDisassembler.cpp index f979af4..081101f 100644 --- a/llvm/lib/Target/RISCV/Disassembler/RISCVDisassembler.cpp +++ b/llvm/lib/Target/RISCV/Disassembler/RISCVDisassembler.cpp @@ -690,12 +690,10 @@ static constexpr DecoderListEntry DecoderList32[]{ {DecoderTableXCV32, XCVFeatureGroup, "CORE-V extensions"}, {DecoderTableXqci32, XqciFeatureGroup, "Qualcomm uC Extensions"}, {DecoderTableXRivos32, XRivosFeatureGroup, "Rivos"}, - {DecoderTable32, {}, "RISCV32"}, - {DecoderTableRV32GPRPair32, {}, "RV32GPRPair (rv32 and GPR pairs)"}, + {DecoderTable32, {}, "standard 32-bit instructions"}, + {DecoderTableRV32Only32, {}, "RV32-only standard 32-bit instructions"}, {DecoderTableZfinx32, {}, "Zfinx (Float in Integer)"}, - {DecoderTableZdinxRV32GPRPair32, - {}, - "ZdinxRV32GPRPair (rv32 and Double in Integer)"}, + {DecoderTableZdinxRV32Only32, {}, "RV32-only Zdinx (Double in Integer)"}, }; DecodeStatus RISCVDisassembler::getInstruction32(MCInst &MI, uint64_t &Size, @@ -714,7 +712,7 @@ DecodeStatus RISCVDisassembler::getInstruction32(MCInst &MI, uint64_t &Size, if (!Entry.haveContainedFeatures(STI.getFeatureBits())) continue; - LLVM_DEBUG(dbgs() << "Trying " << Entry.Desc << "table:\n"); + LLVM_DEBUG(dbgs() << "Trying " << Entry.Desc << " table:\n"); DecodeStatus Result = decodeInstruction(Entry.Table, MI, Insn, Address, this, STI); if (Result == MCDisassembler::Fail) @@ -728,16 +726,16 @@ DecodeStatus RISCVDisassembler::getInstruction32(MCInst &MI, uint64_t &Size, static constexpr DecoderListEntry DecoderList16[]{ // Vendor Extensions - {DecoderTableXqci16, XqciFeatureGroup, "Qualcomm uC 16bit"}, + {DecoderTableXqci16, XqciFeatureGroup, "Qualcomm uC 16-bit"}, {DecoderTableXqccmp16, {RISCV::FeatureVendorXqccmp}, "Xqccmp (Qualcomm 16-bit Push/Pop & Double Move Instructions)"}, {DecoderTableXwchc16, {RISCV::FeatureVendorXwchc}, "WCH QingKe XW"}, // Standard Extensions // DecoderTableZicfiss16 must be checked before DecoderTable16. - {DecoderTableZicfiss16, {}, "RVZicfiss (Shadow Stack)"}, - {DecoderTable16, {}, "RISCV_C (16-bit Instruction)"}, - {DecoderTableRISCV32Only_16, {}, "RISCV32Only_16 (16-bit Instruction)"}, + {DecoderTableZicfiss16, {}, "Zicfiss (Shadow Stack 16-bit)"}, + {DecoderTable16, {}, "standard 16-bit instructions"}, + {DecoderTableRV32Only16, {}, "RV32-only 16-bit instructions"}, // Zc* instructions incompatible with Zcf or Zcd {DecoderTableZcOverlap16, {}, |