Nerixyz wrote:

> Turns out, the slash is even there if the driver is unable to resolve the 
> full path to the linker.

Should this be "Turns out, the slash isn't there if the driver is unable to 
resolve the full path to the linker."?

> Remove the leading slash from the test.

If I understand 
https://github.com/llvm/llvm-project/pull/200012#pullrequestreview-4374517119 
correctly, the concern was that the check could match too much. What about 
allowing `"` as a prefix? So `{{[/\\"]}}ld{{[^" ]*}}"`.

https://github.com/llvm/llvm-project/pull/200549
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to