keith added inline comments.

================
Comment at: clang/test/lit.cfg.py:60
+if platform.system() == 'Windows':
+    root_sep = 'C:\\'
+else:
----------------
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


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

Reply via email to