[PATCH] D86762: [ELF] Add documentation for --warn-backrefs: a layering check tool

[Nico Weber via Phabricator via llvm-commits]

thakis added a comment.

I don't think this should become the default.

- It's weird semantics that are an implementation artifact of bfd ld. ld64 and link.exe (and lld-link) don't have this requirement, and this is a usability win.

- It doesn't help much with layering since it only enforces that the library order on the link command is in some linear embedding of the topological walk of the dependency dag, which isn't all that useful. Layering checks want to look at the real build graph, so the linker is the wrong place for this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86762