rmaz added inline comments.
================ Comment at: clang/test/lit.cfg.py:60 +if platform.system() == 'Windows': + root_sep = 'C:\\' +else: ---------------- keith wrote: > rmaz wrote: > > `c:\`? > Do you mean lowercase or a single slash? I see ~2x the number of uppercase vs > lowercase in this codebase now, and the second is just because of escaping I meant the single slash, yes, wasn't sure if it needed to be escaped or not Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111457/new/ https://reviews.llvm.org/D111457 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits