MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

LG.

A better test should have a few more stuff, see Xlinker-args.c
Instead of saying --defsym takes precedence over T_Group options, you can say 
that -T is the last.

> Few more word for this issue, the option order is matter for linker both for 
> GNU ld and lld, in the test @arcbbb provided, ABC will treat as undefined in 
> a.lds if the order is wrong.

While GNU people may not like it, I think relying on the order between --defsym 
and -T is highly problematic: 
https://sourceware.org/pipermail/libc-alpha/2020-March/111920.html


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100615

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to