[PATCH] D47374: [RFC][patch 3/3] Add support for variant scheduling classes in llvm-mca.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: CodeGen/X86/sse-schedule.ll:6228
 ; BTVER2-SSE:       # %bb.0:
-; BTVER2-SSE-NEXT:    xorps %xmm0, %xmm0 # sched: [1:0.50]
+; BTVER2-SSE-NEXT:    xorps %xmm0, %xmm0 # sched: [0:?]
 ; BTVER2-SSE-NEXT:    #APP
----------------
These are unfortunate - please can you raise an upstream bugzilla about throughput printing defaulting to issues width.


================
Comment at: llvm-mca/InstrBuilder.cpp:388
+  if (!SchedClassID)
+    llvm::report_fatal_error("unable to resolve this variant class.");
 
----------------
So !SchedClassID can only occur here due to variant resolution failing?


https://reviews.llvm.org/D47374