From 014700c1a8cba203fd21ff129426ba8a426ab244 Mon Sep 17 00:00:00 2001 From: David Greene Date: Mon, 13 Jul 2009 20:25:48 +0000 Subject: [PATCH] Add infrastructure to allow post instruction printing action triggers. We'll eventually use this to print comments in asm files and do other fun things. This adds interfaces to the AsmPrinter and changes TableGen to invoke the postInstructionAction when appropriate. It also add parameters to TargetAsmInfo to control comment layout. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@75490 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/CodeGen/AsmPrinter.h | 24 ++++++++++++++++++++++-- include/llvm/Target/TargetAsmInfo.h | 7 +++++++ lib/CodeGen/AsmPrinter/AsmPrinter.cpp | 13 +++++++++++++ lib/Target/TargetAsmInfo.cpp | 1 + utils/TableGen/AsmWriterEmitter.cpp | 23 ++++++++++++++++------- 5 files changed, 59 insertions(+), 9 deletions(-) diff --git a/include/llvm/CodeGen/AsmPrinter.h b/include/llvm/CodeGen/AsmPrinter.h index ef609e4efef..0129711346c 100644 --- a/include/llvm/CodeGen/AsmPrinter.h +++ b/include/llvm/CodeGen/AsmPrinter.h @@ -34,6 +34,7 @@ namespace llvm { class MachineConstantPoolEntry; class MachineConstantPoolValue; class MachineModuleInfo; + class MCInst; class DwarfWriter; class Mangler; class Section; @@ -64,7 +65,7 @@ namespace llvm { /// DW - If available, this is a pointer to the current dwarf writer. DwarfWriter *DW; - + public: /// Output stream on which we're printing assembly code. /// @@ -332,6 +333,17 @@ namespace llvm { /// debug tables. void printDeclare(const MachineInstr *MI) const; + /// postInstructionAction - Handling printing of items after the + /// instruction iteself has been printed (e.g. comments) + void postInstructionAction(const MachineInstr &MI) const { + postInstructionActionImpl(MI); + EmitComments(MI); + } + void postInstructionAction(const MCInst &MI) const { + postInstructionActionImpl(MI); + EmitComments(MI); + } + protected: /// EmitZeros - Emit a block of zeros. /// @@ -396,7 +408,7 @@ namespace llvm { /// printOffset - This is just convenient handler for printing offsets. void printOffset(int64_t Offset) const; - + private: const GlobalValue *findGlobalValue(const Constant* CV); void EmitLLVMUsedList(Constant *List); @@ -408,6 +420,14 @@ namespace llvm { void EmitGlobalConstantFP(const ConstantFP* CFP, unsigned AddrSpace); void EmitGlobalConstantLargeInt(const ConstantInt* CI, unsigned AddrSpace); GCMetadataPrinter *GetOrCreateGCPrinter(GCStrategy *C); + + /// EmitComments - Pretty-print comments for instructions + void EmitComments(const MachineInstr &MI) const; + /// EmitComments - Pretty-print comments for instructions + void EmitComments(const MCInst &MI) const; + + virtual void postInstructionActionImpl(const MachineInstr &MI) const {} + virtual void postInstructionActionImpl(const MCInst &MI) const {} }; } diff --git a/include/llvm/Target/TargetAsmInfo.h b/include/llvm/Target/TargetAsmInfo.h index 670b0996cc3..d031d746578 100644 --- a/include/llvm/Target/TargetAsmInfo.h +++ b/include/llvm/Target/TargetAsmInfo.h @@ -214,6 +214,10 @@ namespace llvm { /// measure inline asm instructions. char SeparatorChar; // Defaults to ';' + /// CommentColumn - This indicates the comment num (zero-based) at + /// which asm comments should be printed. + unsigned CommentColumn; // Defaults to 60 + /// CommentString - This indicates the comment character used by the /// assembler. const char *CommentString; // Defaults to "#" @@ -693,6 +697,9 @@ namespace llvm { char getSeparatorChar() const { return SeparatorChar; } + const unsigned getCommentColumn() const { + return CommentColumn; + } const char *getCommentString() const { return CommentString; } diff --git a/lib/CodeGen/AsmPrinter/AsmPrinter.cpp b/lib/CodeGen/AsmPrinter/AsmPrinter.cpp index ea27490fa14..c1cb97ccf72 100644 --- a/lib/CodeGen/AsmPrinter/AsmPrinter.cpp +++ b/lib/CodeGen/AsmPrinter/AsmPrinter.cpp @@ -24,6 +24,7 @@ #include "llvm/Analysis/DebugInfo.h" #include "llvm/Support/CommandLine.h" #include "llvm/Support/ErrorHandling.h" +#include "llvm/Support/FormattedStream.h" #include "llvm/Support/Mangler.h" #include "llvm/Support/raw_ostream.h" #include "llvm/Target/TargetAsmInfo.h" @@ -1748,3 +1749,15 @@ GCMetadataPrinter *AsmPrinter::GetOrCreateGCPrinter(GCStrategy *S) { cerr << "no GCMetadataPrinter registered for GC: " << Name << "\n"; llvm_unreachable(); } + +/// EmitComments - Pretty-print comments for instructions +void AsmPrinter::EmitComments(const MachineInstr &MI) const +{ + // No comments in MachineInstr yet +} + +/// EmitComments - Pretty-print comments for instructions +void AsmPrinter::EmitComments(const MCInst &MI) const +{ + // No comments in MCInst yet +} diff --git a/lib/Target/TargetAsmInfo.cpp b/lib/Target/TargetAsmInfo.cpp index 4fbe1ae55e4..3df09bc60a5 100644 --- a/lib/Target/TargetAsmInfo.cpp +++ b/lib/Target/TargetAsmInfo.cpp @@ -43,6 +43,7 @@ TargetAsmInfo::TargetAsmInfo(const TargetMachine &tm) MaxInstLength = 4; PCSymbol = "$"; SeparatorChar = ';'; + CommentColumn = 60; CommentString = "#"; GlobalPrefix = ""; PrivateGlobalPrefix = "."; diff --git a/utils/TableGen/AsmWriterEmitter.cpp b/utils/TableGen/AsmWriterEmitter.cpp index f34feef0c28..92ad96ec948 100644 --- a/utils/TableGen/AsmWriterEmitter.cpp +++ b/utils/TableGen/AsmWriterEmitter.cpp @@ -259,8 +259,6 @@ AsmWriterInst::AsmWriterInst(const CodeGenInstruction &CGI, unsigned Variant) { LastEmitted = VarEnd; } } - - AddLiteralString("\\n"); } /// MatchesAllButOneOp - If this instruction is exactly identical to the @@ -357,7 +355,6 @@ static void EmitInstructions(std::vector &Insts, } O << "\n"; } - O << " break;\n"; } @@ -385,8 +382,12 @@ FindUniqueOperandCommands(std::vector &UniqueOperandCommands, Command = " " + Inst->Operands[0].getCode() + "\n"; // If this is the last operand, emit a return. - if (Inst->Operands.size() == 1) + if (Inst->Operands.size() == 1) { + Command += " postInstructionAction(*MI);\n"; + // Print the final newline + Command += " O << \"\\n\";\n"; Command += " return true;\n"; + } // Check to see if we already have 'Command' in UniqueOperandCommands. // If not, add it. @@ -452,8 +453,12 @@ FindUniqueOperandCommands(std::vector &UniqueOperandCommands, std::string Command = " " + FirstInst->Operands[Op].getCode() + "\n"; // If this is the last operand, emit a return after the code. - if (FirstInst->Operands.size() == Op+1) + if (FirstInst->Operands.size() == Op+1) { + Command += " postInstructionAction(*MI);\n"; + // Print the final newline + Command += " O << \"\\n\";\n"; Command += " return true;\n"; + } UniqueOperandCommands[CommandIdx] += Command; InstOpsUsed[CommandIdx]++; @@ -564,10 +569,11 @@ void AsmWriterEmitter::run(raw_ostream &O) { // For the first operand check, add a default value for instructions with // just opcode strings to use. if (isFirst) { - UniqueOperandCommands.push_back(" return true;\n"); + // Do the post instruction processing and print the final newline + UniqueOperandCommands.push_back(" postInstructionAction(*MI);\n O << \"\\n\";\n return true;\n"); isFirst = false; } - + std::vector InstIdxs; std::vector NumInstOpsHandled; FindUniqueOperandCommands(UniqueOperandCommands, InstIdxs, @@ -739,6 +745,9 @@ void AsmWriterEmitter::run(raw_ostream &O) { EmitInstructions(Instructions, O); O << " }\n"; + O << " postInstructionAction(*MI);\n"; + // Print the final newline + O << " O << \"\\n\";\n"; O << " return true;\n"; } -- 2.34.1