[PATCH] D147131: [PoC][TabgleGen] Add new bang operator !apply

[Artem Belevich via Phabricator via llvm-commits]

tra added a comment.

While we're on the subject of lambdas, the pattern we currently use to deal with the lack of lambdas or functions is to instantiate a record with a field evaluated to be a particular value. A lot of those case could be better served by lambdas, but they would need to be able to deal with all tablegen types, not just integers.

I think the current patch's functionality could already be done using this approach, even if it's a bit more verbose.
E.g: https://github.com/llvm/llvm-project/blob/main/llvm/lib/Target/AMDGPU/SIInstrInfo.td#L305
https://github.com/llvm/llvm-project/blob/main/llvm/lib/Target/AMDGPU/SIInstrInfo.td#L2469

We may be just looking for a better syntax, not a new functionality.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147131