This is an archive of the discontinued LLVM Phabricator instance.

Differential D141995

[CSKY] Fix errors caused by change of compressInst
ClosedPublic

Authored by • pcwang-thead on Jan 18 2023, 12:09 AM.

Details

Reviewers
zixuan-wu
craig.topper
Commits
rGe4abfc4c3cf4: [CSKY] Fix errors caused by change of compressInst
Summary

We don't need MCContext since D141951.

Diff Detail

Unit TestsFailed

TimeTest
60,160 msx64 debian > Clang.CodeGen/RISCV/rvv-intrinsics-autogenerated/policy/non-overloaded::vloxseg.c
60,160 msx64 debian > Clang.CodeGen/RISCV/rvv-intrinsics-autogenerated/policy/non-overloaded::vluxseg.c
60,140 msx64 debian > Clang.CodeGen/RISCV/rvv-intrinsics-autogenerated/policy/overloaded::vloxseg.c
60,160 msx64 debian > Clang.CodeGen/RISCV/rvv-intrinsics-autogenerated/policy/overloaded::vluxseg.c

Event Timeline

• pcwang-thead created this revision.Jan 18 2023, 12:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 18 2023, 12:09 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
• pcwang-thead requested review of this revision.Jan 18 2023, 12:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 18 2023, 12:09 AM
Herald added a subscriber: llvm-commits. · View Herald Transcript
zixuan-wu accepted this revision.Jan 18 2023, 12:48 AM
This revision is now accepted and ready to land.Jan 18 2023, 12:48 AM
This revision was landed with ongoing or failed builds.Jan 18 2023, 1:01 AM
Closed by commit rGe4abfc4c3cf4: [CSKY] Fix errors caused by change of compressInst (authored by • pcwang-thead). · Explain Why
This revision was automatically updated to reflect the committed changes.
• pcwang-thead added a commit: rGe4abfc4c3cf4: [CSKY] Fix errors caused by change of compressInst.
Harbormaster completed remote builds in B208422: Diff 490058.Jan 18 2023, 1:06 AM

Revision Contents

PathSize
llvm/
lib/
Target/
CSKY/
AsmParser/
2 lines
2 lines

Diff 490058

llvm/lib/Target/CSKY/AsmParser/CSKYAsmParser.cpp

Show First 20 Lines • Show All 1,741 Lines • ▼ Show 20 Linesunsigned CSKYAsmParser::validateTargetOperandClass(MCParsedAsmOperand &AsmOp,
return Match_InvalidOperand; return Match_InvalidOperand;
} }
void CSKYAsmParser::emitToStreamer(MCStreamer &S, const MCInst &Inst) { void CSKYAsmParser::emitToStreamer(MCStreamer &S, const MCInst &Inst) {
MCInst CInst; MCInst CInst;
bool Res = false; bool Res = false;
if (EnableCompressedInst) if (EnableCompressedInst)
Res = compressInst(CInst, Inst, getSTI(), S.getContext()); Res = compressInst(CInst, Inst, getSTI());
if (Res) if (Res)
++CSKYNumInstrsCompressed; ++CSKYNumInstrsCompressed;
S.emitInstruction((Res ? CInst : Inst), getSTI()); S.emitInstruction((Res ? CInst : Inst), getSTI());
} }
extern "C" LLVM_EXTERNAL_VISIBILITY void LLVMInitializeCSKYAsmParser() { extern "C" LLVM_EXTERNAL_VISIBILITY void LLVMInitializeCSKYAsmParser() {
RegisterMCAsmParser<CSKYAsmParser> X(getTheCSKYTarget()); RegisterMCAsmParser<CSKYAsmParser> X(getTheCSKYTarget());
} }

llvm/lib/Target/CSKY/CSKYAsmPrinter.cpp

Show First 20 Lines • Show All 52 Lines • ▼ Show 20 Linesbool CSKYAsmPrinter::runOnMachineFunction(MachineFunction &MF) {
return AsmPrinter::runOnMachineFunction(MF); return AsmPrinter::runOnMachineFunction(MF);
} }
#define GEN_COMPRESS_INSTR #define GEN_COMPRESS_INSTR
#include "CSKYGenCompressInstEmitter.inc" #include "CSKYGenCompressInstEmitter.inc"
void CSKYAsmPrinter::EmitToStreamer(MCStreamer &S, const MCInst &Inst) { void CSKYAsmPrinter::EmitToStreamer(MCStreamer &S, const MCInst &Inst) {
MCInst CInst; MCInst CInst;
bool Res = compressInst(CInst, Inst, *Subtarget, OutStreamer->getContext()); bool Res = compressInst(CInst, Inst, *Subtarget);
if (Res) if (Res)
++CSKYNumInstrsCompressed; ++CSKYNumInstrsCompressed;
AsmPrinter::EmitToStreamer(*OutStreamer, Res ? CInst : Inst); AsmPrinter::EmitToStreamer(*OutStreamer, Res ? CInst : Inst);
} }
// Simple pseudo-instructions have their lowering (with expansion to real // Simple pseudo-instructions have their lowering (with expansion to real
// instructions) auto-generated. // instructions) auto-generated.
#include "CSKYGenMCPseudoLowering.inc" #include "CSKYGenMCPseudoLowering.inc"
▲ Show 20 Lines • Show All 259 Lines • Show Last 20 Lines