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

Simon Tatham via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 6 06:47:57 PST 2019


simon_tatham added a comment.

Yes, I did. I prefer this way, because it comes out considerably shorter in the use case I had in mind. Compare these two (inside a templated class with a parameter `dag extraArgs` which might be, say, `(args 3, 4)`):

  dag d = !conp((MyRealOperator<...> a, b), extraArgs);
  dag d = !setop(MyRealOperator<...>, !con((args a, b), extraArgs));

But I can implement `!setop` if you prefer. (And while I'm at it, perhaps `!getop` too.)


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