[PATCH] D84825: [release/11.x only][ELF] Change tombstone value -1 to 0

Wed Jul 29 11:56:31 PDT 2020

MaskRay added a comment.

In D84825#2182460 <https://reviews.llvm.org/D84825#2182460>, @dblaikie wrote:

> Please implement a clear flag for
>
> - BFD semantics
> - New (-1/-2) semantics
>
> And default to the BFD semantics for now.
>
> The change in semantics is not ready to ship by default - there are lots of DWARF consumers out there and lots of people don't update them in lock-step with their compiler.

I don't know what the request is. The patch implemented:

- .debug_ranges & .debug_loc: -2
- .debug_*: 0  (GNU ld semantics)

If you meant full BFD semantics:

- .debug_ranges: 1
- .debug_*: 0

Sorry, I don't think that is necessary. I also don't want to add another option like `--gnu-ld-debug-reloc-semantics` or similar.

If you meant a shortcut to

- -z 'dead-reloc-in-nonalloc=.debug_loc=0xfffffffffffffffe'
- -z 'dead-reloc-in-nonalloc=.debug_ranges=0xfffffffffffffffe'
- -z 'dead-reloc-in-nonalloc=.debug_*=0xffffffffffffffff'

I don't that is necessary either. That option will be our compatibility burden. If would cause pain when we want to remove it in LLD 12 or 13.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84825