On AIX, libraries are still being linked when -r is passed to the driver. This patch corrects this error.
nit: can we avoid the code duplication here? We can use a goto or add options::OPT_r checking below.
This patch is still WIP while I verify the libraries that we intend not to add. But I will add a test case once I've done so :)
LGTM. Would be nice to get @daltenty 's approval too.
is there a way to check that -l<anything> is not present?
This mostly looks good, but I'm not sure other toolchains omit use specified L options. I think we shouldn't either.
|1696 ↗||(On Diff #505691)|
I think this is redundant, we guard out this whole function call in the block above.
LGTM, with a small nit about the test that should be addressed before committing
nit: add the user -L option to the test to reflect the changes above