[lld] [LLD] [MinGW] Implement --dependent-load-flag option (PR #113814)

Martin Storsjö via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 22 02:36:45 PST 2024


mstorsjo wrote:

> > Anyway, to sum things up - this patch is fine and I guess there's no unfinished discussion thread that would block merging it - right?
> > @alvinhochun's great suggestion about warning if `-dependentloadflag:` is used without a load config being available, can be done by either of us as a separate later step.
> 
> Yeah, LGTM (with one minor inline comment). Though it doesn't seem like there is an urgent need for it to be in main right now, so I feel maybe it is beneficial to wait until a PR to add the warning is ready, so both changes can be merged together. Wdyt?

No strong opinion either way. On the other hand, even if we don't have all the nice convenience in place, it might be nice to get the option merged to align other parties around it (binutils, and users)?

https://github.com/llvm/llvm-project/pull/113814


More information about the llvm-commits mailing list