[llvm] [MCA] New option -scheduling-info (PR #130574)
Min-Yih Hsu via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 17 10:42:26 PDT 2025
================
@@ -29,82 +61,152 @@ void InstructionInfoView::printView(raw_ostream &OS) const {
IIVDVec IIVD(Source.size());
collectData(IIVD);
- TempStream << "\n\nInstruction Info:\n";
- TempStream << "[1]: #uOps\n[2]: Latency\n[3]: RThroughput\n"
- << "[4]: MayLoad\n[5]: MayStore\n[6]: HasSideEffects (U)\n";
+ if (PrintFullInfo) {
+ FOS << "\n\nResources:\n";
+ const MCSchedModel &SM = getSubTargetInfo().getSchedModel();
+ for (unsigned I = 1, ResourceIndex = 0, E = SM.getNumProcResourceKinds();
+ I < E; ++I) {
+ const MCProcResourceDesc &ProcResource = *SM.getProcResource(I);
+ unsigned NumUnits = ProcResource.NumUnits;
+ // Skip invalid resources with zero units.
+ if (!NumUnits)
+ continue;
+
+ FOS << '[' << ResourceIndex << ']';
+ FOS.PadToColumn(6);
+ FOS << "- " << ProcResource.Name << ':' << NumUnits;
+ if (ProcResource.SubUnitsIdxBegin) {
+ FOS.PadToColumn(20);
+ for (unsigned U = 0; U < NumUnits; ++U) {
+ FOS << SM.getProcResource(ProcResource.SubUnitsIdxBegin[U])->Name;
+ if ((U + 1) < NumUnits) {
+ FOS << ", ";
+ }
+ }
+ }
+ FOS << '\n';
+ ResourceIndex++;
+ }
+ }
+
+ std::vector<unsigned> paddings = {0, 7, 14, 21, 28, 35};
+ std::vector<std::string> fields = {"#uOps", "Latency",
+ "RThroughput", "MayLoad",
+ "MayStore", "HasSideEffects (U)"};
+ std::vector<std::string> end_fields;
+ unsigned LastPadding = 35;
+ if (PrintFullInfo) {
+ fields.push_back("Bypass Latency");
+ paddings.push_back(LastPadding + 7);
+ LastPadding += 7;
+ fields.push_back("Resources");
+ paddings.push_back(LastPadding + 7);
+ LastPadding += 7;
+ fields.push_back("LLVM Opcode Name");
+ paddings.push_back(LastPadding + 57);
+ LastPadding += 57;
+ }
if (PrintBarriers) {
- TempStream << "[7]: LoadBarrier\n[8]: StoreBarrier\n";
+ fields.push_back("LoadBarrier");
+ paddings.push_back(LastPadding + 7);
+ fields.push_back("StoreBarrier");
+ paddings.push_back(LastPadding + 14);
+ LastPadding += 14;
}
if (PrintEncodings) {
- if (PrintBarriers) {
- TempStream << "[9]: Encoding Size\n";
- TempStream << "\n[1] [2] [3] [4] [5] [6] [7] [8] "
- << "[9] Encodings: Instructions:\n";
+ paddings.push_back(LastPadding + 7);
+ paddings.push_back(LastPadding + 14);
+ paddings.push_back(LastPadding + 44);
+ LastPadding += 44;
+ fields.push_back("Encoding Size");
+ end_fields.push_back("Encodings:");
+ end_fields.push_back("Instructions:");
+ } else {
+ if (PrintFullInfo) {
+ paddings.push_back(LastPadding + 27);
+ LastPadding += 27;
} else {
- TempStream << "[7]: Encoding Size\n";
- TempStream << "\n[1] [2] [3] [4] [5] [6] [7] "
- << "Encodings: Instructions:\n";
+ paddings.push_back(LastPadding + 7);
+ LastPadding += 7;
}
- } else {
- if (PrintBarriers) {
- TempStream << "\n[1] [2] [3] [4] [5] [6] [7] [8] "
- << "Instructions:\n";
+ end_fields.push_back("Instructions:");
+ }
+
+ FOS << "\n\nInstruction Info:\n";
+ for (unsigned i = 0; i < fields.size(); i++) {
+ FOS << "[" << i + 1 << "]: " << fields[i] << "\n";
+ }
+ FOS << "\n";
+
+ for (unsigned i = 0; i < paddings.size(); i++) {
+ if (paddings[i] != 0)
+ FOS.PadToColumn(paddings[i]);
+ if (i < fields.size()) {
+ FOS << "[" << i + 1 << "]";
} else {
- TempStream << "\n[1] [2] [3] [4] [5] [6] "
- << "Instructions:\n";
+ FOS << end_fields[i - fields.size()];
}
}
+ FOS << "\n";
for (const auto &[Index, IIVDEntry, Inst] : enumerate(IIVD, Source)) {
- TempStream << ' ' << IIVDEntry.NumMicroOpcodes << " ";
- if (IIVDEntry.NumMicroOpcodes < 10)
- TempStream << " ";
- else if (IIVDEntry.NumMicroOpcodes < 100)
- TempStream << ' ';
- TempStream << IIVDEntry.Latency << " ";
- if (IIVDEntry.Latency < 10)
- TempStream << " ";
- else if (IIVDEntry.Latency < 100)
- TempStream << ' ';
-
+ FOS.PadToColumn(paddings[0] + 1);
+ FOS << IIVDEntry.NumMicroOpcodes;
+ FOS.PadToColumn(paddings[1] + 1);
+ FOS << IIVDEntry.Latency;
+ FOS.PadToColumn(paddings[2]);
if (IIVDEntry.RThroughput) {
double RT = *IIVDEntry.RThroughput;
- TempStream << format("%.2f", RT) << ' ';
- if (RT < 10.0)
- TempStream << " ";
- else if (RT < 100.0)
- TempStream << ' ';
+ FOS << format("%.2f", RT);
} else {
- TempStream << " - ";
+ FOS << " -";
+ }
+ FOS.PadToColumn(paddings[3] + 1);
+ FOS << (IIVDEntry.mayLoad ? "*" : " ");
+ FOS.PadToColumn(paddings[4] + 1);
+ FOS << (IIVDEntry.mayStore ? "*" : " ");
+ FOS.PadToColumn(paddings[5] + 1);
+ FOS << (IIVDEntry.hasUnmodeledSideEffects ? "U" : " ");
+ unsigned LastPaddingIdx = 5;
+
+ if (PrintFullInfo) {
+ FOS.PadToColumn(paddings[LastPaddingIdx + 1] + 1);
+ FOS << IIVDEntry.Bypass;
+ FOS.PadToColumn(paddings[LastPaddingIdx + 2] + 1);
+ FOS << IIVDEntry.Resources;
+ FOS.PadToColumn(paddings[LastPaddingIdx + 3] + 1);
+ FOS << IIVDEntry.OpcodeName;
+ LastPaddingIdx += 3;
}
- TempStream << (IIVDEntry.mayLoad ? " * " : " ");
- TempStream << (IIVDEntry.mayStore ? " * " : " ");
- TempStream << (IIVDEntry.hasUnmodeledSideEffects ? " U " : " ");
if (PrintBarriers) {
- TempStream << (LoweredInsts[Index]->isALoadBarrier() ? " * "
- : " ");
- TempStream << (LoweredInsts[Index]->isAStoreBarrier() ? " * "
- : " ");
+ FOS.PadToColumn(paddings[LastPaddingIdx + 1] + 1);
+ FOS << (LoweredInsts[Index]->isALoadBarrier() ? "*" : " ");
+ FOS.PadToColumn(paddings[LastPaddingIdx + 2] + 1);
+ FOS << (LoweredInsts[Index]->isAStoreBarrier() ? "*" : " ");
+ LastPaddingIdx += 2;
}
if (PrintEncodings) {
StringRef Encoding(CE.getEncoding(Index));
unsigned EncodingSize = Encoding.size();
- TempStream << " " << EncodingSize
- << (EncodingSize < 10 ? " " : " ");
- TempStream.flush();
- formatted_raw_ostream FOS(TempStream);
+ FOS.PadToColumn(paddings[LastPaddingIdx + 1] + 1);
+ FOS << EncodingSize;
+ FOS.PadToColumn(paddings[LastPaddingIdx + 2]);
for (unsigned i = 0, e = Encoding.size(); i != e; ++i)
FOS << format("%02x ", (uint8_t)Encoding[i]);
- FOS.PadToColumn(30);
- FOS.flush();
+ LastPaddingIdx += 2;
}
-
- TempStream << printInstructionString(Inst) << '\n';
+ FOS.PadToColumn(paddings[LastPaddingIdx + 1]);
+ FOS << printInstructionString(Inst);
+ if (PrintFullInfo) {
+ getComment(Inst, CommentString);
----------------
mshockwave wrote:
nit: can we move the declaration of `CommentString` to here, I think these two lines are the only users.
https://github.com/llvm/llvm-project/pull/130574
More information about the llvm-commits
mailing list