[PATCH] D44214: Improve --warn-symbol-ordering.

[James Henderson via Phabricator via llvm-commits]

jhenderson added inline comments.


================
Comment at: lld/ELF/Writer.cpp:1063
+      auto *Sec = dyn_cast_or_null<InputSectionBase>(cast<Defined>(Sym)->Section);
+      if (!Sec) {
+        Warning("unable to order absolute symbol");
----------------
espindola wrote:
> This will now fire on symbols the linker defines relative to an output section, no?
> 
That would be a good thing, right? Attempting to order a symbol not associated with an OutputSection (whether linker-defined or not) would be impossible.


https://reviews.llvm.org/D44214