[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