[PATCH] D58510: [yaml2obj]Allow explicit symbol indexes in relocations and emit error for bad names

Fri Mar 1 01:13:54 PST 2019

jhenderson added a comment.
Herald added a subscriber: MaskRay.

In D58510#1406984 <https://reviews.llvm.org/D58510#1406984>, @jhenderson wrote:

> I just ran into https://bugs.llvm.org/show_bug.cgi?id=40818, which is preventing me landing this change as is. I could modify the llvm-objcopy compress-debug-sections.yaml file to not reference a symbol at all, but I think that loses some of the intention of that test (plus it would just hide the bug).

This bug has been fixed, but it revealed https://bugs.llvm.org/show_bug.cgi?id=40885, which also blocks this change unless I workaround it (I may do so for expediency, but I have other things to look at in the meantime).

Repository:
  rL LLVM

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

https://reviews.llvm.org/D58510