[lld] [LLD] [MinGW] Sync --thinlto-cache-dir option details with ELF (PR #77010)

Martin Storsjö via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 8 05:00:06 PST 2024


mstorsjo wrote:

> The patch looks good to me, but should it have a test case? I guess it doesn't matter much in this case, but it also seems easy enough to have one.

Not sure we explicitly want to have a test to disallow specific spellings of all potential options (unless we have a real world case for that) - the main reason here is that when looking into adding more options from lld/ELF into lld/MinGW, I noticed this discrepancy - which seems unintentional (all uses of the option use this one single form).

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


More information about the llvm-commits mailing list