[PATCH] D71115: [TableGen] Add a permissive version of !con.

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 8 10:27:36 PST 2019


nhaehnle added a comment.

I don't feel particularly strongly either way, but it seems to me that allowing an unset operator could perhaps be the most elegant solution after all. I'm lacking the context of your particular use case, but it seems to me you're attempting to piece together DAG fragments in a generic way where the operator simply isn't known yet by some of the pieces of TableGen you're writing, and using an unset operator could be a more natural way of expressing that than coming up with some artificial operator that is simply thrown away later.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71115/new/

https://reviews.llvm.org/D71115





More information about the llvm-commits mailing list