[llvm] [RemoveDIs] Print non-intrinsic debug info in textual IR output (PR #79281)

Stephen Tozer via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 19 10:35:45 PST 2024


SLTozer wrote:

> It looks like it'll only be possible to get this format out of `opt` and other tools that print through the pass-manager, so things like `llvm-dis` won't immediately be able to print this format.

Yeah, the alternative would be to move the conversion step into `Module::print`, but that would affect debug printing, which isn't desirable I think, and we can easily move functionality piecemeal into tools that want it.

> I think the one area where we're limiting ourselves a little is if we add or remove operands to "records" (ex-intrinsics) in the future.

This is a limitation, but one where I decided it wasn't worth implementing any kind of variadic logic because these kind of signature changes happen less than once a year, and in return we can keep the parser logic simple.



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


More information about the llvm-commits mailing list