mstorsjo added a comment.

In D80876#2067532 <https://reviews.llvm.org/D80876#2067532>, @rnk wrote:

> This seems wrong for mingw, so check with @mstorsjo.


Thanks for the headsup!

I agree that this change would make sense and would make things more consistent 
in general, but @rnk is also right that, within a mingw environment, the 
expectation would probably be for the old behaviour.

I did one build test with a large link command (done via libtool), where 
libtool produed a response file, containing unix-style-quoted windows paths, 
like `some\\path\\to\\a\\file` - and if I added `--rsp-quoting=windows` to it, 
it still built fine, so while it would be expanded to actually contain double 
backslashes in that case (I presume?) it seemed harmless. So as long as the 
response files contain only relative pathnames, misinterpreting a unix-quoted 
file as windows quoted seems harmless.

But as for keeping the old default in mingw environments, would it make sense 
to, if on windows, check the default target triple, and if that's a mingw 
target (`Triple.isWindowsGNUEnvironment()`), keep the unix style quoting as 
default?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80876/new/

https://reviews.llvm.org/D80876



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [P... Sam Clegg via Phabricator via cfe-commits
    • ... Paul Robinson via Phabricator via cfe-commits
    • ... Sam Clegg via Phabricator via cfe-commits
    • ... Paul Robinson via Phabricator via cfe-commits
    • ... Paul Robinson via Phabricator via cfe-commits
    • ... Paul Robinson via Phabricator via cfe-commits
    • ... Paul Robinson via Phabricator via cfe-commits
    • ... Paul Robinson via Phabricator via cfe-commits
    • ... Reid "Away June-Sep" Kleckner via Phabricator via cfe-commits
    • ... Martin Storsjö via Phabricator via cfe-commits
    • ... Martin Storsjö via Phabricator via cfe-commits
    • ... Sam Clegg via Phabricator via cfe-commits
    • ... Sam Clegg via Phabricator via cfe-commits
    • ... Martin Storsjö via Phabricator via cfe-commits
    • ... Sam Clegg via Phabricator via cfe-commits
    • ... Martin Storsjö via Phabricator via cfe-commits
    • ... Mateusz Mikuła via Phabricator via cfe-commits
    • ... Sam Clegg via Phabricator via cfe-commits
    • ... Martin Storsjö via Phabricator via cfe-commits

Reply via email to