MaxEW707 wrote:

> Windows should use `-fuse-ld=lld` as well, not `-fuse-ld=lld-link`

Is `-fuse-ld=lld-link` intended to be an allowable option?

Right now we don't issue any error if a user passes in `lld-link`.

I am interpreting your statement as suggesting we should do something like 
following?
```
const Arg *A = Args.getLastArg(options::OPT_fuse_ld_EQ);
if (A->getValue() == "lld-link") {
    D.Diag(diag::err_drv_unsupported_option_argument) << A->getSpelling() << 
A->getValue();
}
```

https://github.com/llvm/llvm-project/pull/113966
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to