[PATCH] D144351: llvm-tblgen: Let each emitter self-contained

Sat Feb 25 03:45:15 PST 2023

chapuni added a comment.

In D144351#4148987 <https://reviews.llvm.org/D144351#4148987>, @myhsu wrote:

> then yeah, moving it to public header might be a good idea.

I'm afraid if it would affect other *-tblgen. It was in public in my prototype.
I'd be happy to move it to public step-by-step.

When we could implement tblgen plugin, I wonder;

- If we took `-load foo-pluigin.so`, how it could be activated. I guess we should enhance `TableGen::Emitter::Opt` to activate myself. In my current implementation, `TableGen::Emitter::Opt` just registers an option.
- Or, could we enforce users to name their plugin as `libtblgen-foo-option.so`?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144351