================
@@ -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

Reply via email to