Closed llvmbot closed 4 years ago
Reopening. Probably I was too fast when closing this. There is still a list of options that are handled manually and even if we don't need them, r324322 would not catch all possible new options if any.
With r324322 I believe we are automatically compatible with gold plugin options now.
Do we need to implement anything else here ?
Possible patch for some of options: https://reviews.llvm.org/D36227
Extended Description
ThinLTO landed, and Sean pointed out that we currently ignore --plugin-opt and we could instead change LLD to DTRT (i.e. parse plugin-opt arguments and alias to existing lld options). There's an agreement this is a good thing to do, so let's keep track of this here.