[PATCH] D43655: TableGen: simplify BitsInit::resolveReferences

[Nicolai Hähnle via Phabricator via llvm-commits]

nhaehnle marked 2 inline comments as done.
nhaehnle added inline comments.


================
Comment at: test/TableGen/UnsetBitInit.td:3
 // XFAIL: vg_leak
+
+// CHECK: --- Defs ---
----------------
tra wrote:
> More details would be helpful here. E.g.: `Verify that Inst still contains references to P and Q, instead of '?' which is their current value.`
Done locally.


================
Comment at: test/TableGen/UnsetBitInit.td:8
+// CHECK:   bits<8> Inst = { 1, 1, 1, 1, 1, 1, P, Q };
+// CHECK: }
+
----------------
tra wrote:
> Adding checks for P and Q here would help to illustrate the behavior the test verifies.
Done locally.


Repository:
  rL LLVM

https://reviews.llvm.org/D43655