[Lldb-commits] [lldb] [LLDB][ELF] Fix section unification to not just use names. (PR #90099)
Alastair Houghton via lldb-commits
lldb-commits at lists.llvm.org
Mon Apr 29 02:26:57 PDT 2024
al45tair wrote:
> I saw that, but a textual test is definitely preferable, particularly after the linux xz fiasco. This wouldn't be the first test binary in our repo, but in this case, I think it actually adds a lot of value, since yaml2obj operates on the same level as what you are testing (so you can see the input of the test eyeball-verify that the expected output makes sense).
:-) Fair point (though this is very much not like the xz incident — I'm a colleague of Jonas's at Apple, albeit in a different team, with a fairly longstanding history of contributing to OSS projects, and doing something like that would be… very career limiting).
I'll have a go at changing things to use `yaml2obj`.
https://github.com/llvm/llvm-project/pull/90099
More information about the lldb-commits
mailing list