[PATCH] D82388: move "basic" builtins to TableGen

[Nathan Froyd via Phabricator via cfe-commits]

froydnj added a comment.

FWIW, an in-progress tree for the proposed move for all the architecture-specific builtins is:

https://github.com/froydnj/llvm-project/tree/move-builtins-to-tablegen

I don't know if the ARM/NEON/SVE builtins will be easily movable.  It looks like they are separately generated from existing `.td` files; while you could generate `.td` files containing all the appropriate `Builtin` etc. definitions, you cannot have tablegen input depend on tablegen'd output in the general case due to:

https://github.com/llvm/llvm-project/blob/c83ec0a633583e5b12e0aeb70627eb35f7cd4847/llvm/cmake/modules/TableGen.cmake#L11-L35

in the non-Ninja case.  If you try, you get circular dependencies.  One could maybe hack around this by giving certain files different suffixes, but that feels gross.

I'm not sure how to address that issue.


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

https://reviews.llvm.org/D82388