[llvm] [TableGen] DecoderEmitter clean-ups and modernization. (PR #84832)

James Y Knight via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 12 09:01:41 PDT 2024


jyknight wrote:

> One thing I've wanted to do for a long time, but never got around to: I think we should move the long blobs of helper functions out of this file, and into a separate header which gets included from the output.
> 
> I don't think there's any real value in having them be emitted textually into the output code, vs included, and it'll be a lot easier to read/edit that way.

Sorry, I meant to add: I didn't mean that as a requirement for this PR, just a suggestion for potential future work.

https://github.com/llvm/llvm-project/pull/84832


More information about the llvm-commits mailing list