================
@@ -74,31 +76,114 @@ class MIRVocabulary {
friend class llvm::MIR2VecVocabLegacyAnalysis;
using VocabMap = std::map<std::string, ir2vec::Embedding>;
-private:
- // Define vocabulary layout - adapted for MIR
+ // MIRVocabulary Layout:
+ //
+-------------------+-----------------------------------------------------+
+ // | Entity Type | Description
|
+ //
+-------------------+-----------------------------------------------------+
+ // | 1. Opcodes | Target specific opcodes derived from TII, grouped
|
+ // | | by instruction semantics.
|
+ // | 2. Common Operands| All common operand types, except register operands,
|
+ // | | defined by MachineOperand::MachineOperandType enum.
|
+ // | 3. Physical | Register classes defined by the target, specialized
|
----------------
mtrofin wrote:
add "classes " for 3 and 4 in the left column. I know the right column says
"classes", but this would make it immediately more clear.
https://github.com/llvm/llvm-project/pull/163281
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits