[PATCH] D94323: [dsymutil] Add preliminary support for DWARF 5.

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 8 17:25:01 PST 2021


JDevlieghere added a comment.

In D94323#2487899 <https://reviews.llvm.org/D94323#2487899>, @avl wrote:

> If I understood correctly, It looks like this change is not enough to work properly for DWARF5.

Yeah, I knew that when I uploaded the patch but I was planning to put up a series of smaller patches but then I changed my mind. I went with a slightly different approach than what you suggested, but the idea is the essentially the same. At first I tried to shoehorn them into the `ValidRelocs` list but the separate vector simplifies things a lot.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94323/new/

https://reviews.llvm.org/D94323



More information about the llvm-commits mailing list