[llvm] Support big endian in llvm-symbolizer's data location dwarf info parser (PR #67284)
Paul T Robinson via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 25 10:14:50 PDT 2023
pogo59 wrote:
> I've added a couple of colleagues who are more familiar with the details of the DWARF standard and library than me, as I don't think my knowledge in this area is strong enough to review the change.
yes, well, the code change looks completely appropriate (I suspect it was hard-coded to little endian originally due to lack of big-endian DWARF targets). On the other hand, I am unfamiliar with llvm-symbolizer so I have no idea what the test is testing.
https://github.com/llvm/llvm-project/pull/67284
More information about the llvm-commits
mailing list