[llvm] [llvm-objdump] Add inlined function display support (PR #142246)

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 5 01:31:03 PDT 2025


https://github.com/jh7370 commented:

This deserves a release note, I think, including highlighting the renamed indent option.

Also, I think it would be useful to have a test showing the interaction between the variable and inlined function options, so how a variable is displayed inside an inlined function when both are enabled.

On that note, does it even make sense for the two to have different values? E.g. one having unicode and the other ASCII? I wonder if we need to revisit how the option works. I'm not sure what would be the best approach, but perhaps there could be a "format" option and a separate option (or options) to determine what to enable. Alternatively, we just go with it and accept that ASCII + unicode is allowed.

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


More information about the llvm-commits mailing list