[PATCH] D44111: TableGen: Allow ? in lists

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 7 02:42:11 PST 2018


nhaehnle added inline comments.


================
Comment at: test/TableGen/dag-functional.td:11
 // CHECK: def A1 {
-//   dag ret = (ops 1:$a, 2:$b);
+//   dag ret = (ops ?:$a, 1:$b, 2);
 // }
----------------
tra wrote:
> I assume we do care about the value of the `dag ret`, not just the fact that the record got defined. If so, I think there's a CHECK missing here and in the rest of the checks in the file.
Ooops, thanks for pointing that out! :)


Repository:
  rL LLVM

https://reviews.llvm.org/D44111





More information about the llvm-commits mailing list