Re: [PATCH] D41081: Fix clang Lexer Windows line-ending bug

2017-12-12 Thread Benoit Belley via cfe-commits
:zhen@autodesk.com>> Cc: Benoit Belley mailto:benoit.bel...@autodesk.com>>; reviews+d41081+public+5a71b504a12c1...@reviews.llvm.org<mailto:reviews+d41081+public+5a71b504a12c1...@reviews.llvm.org>; r...@google.com<mailto:r...@google.com>; cfe-commits@lists.llvm.org<mailto

RE: [PATCH] D41081: Fix clang Lexer Windows line-ending bug

2017-12-11 Thread Zhen Cao via cfe-commits
oit.bel...@autodesk.com>> Cc: reviews+d41081+public+5a71b504a12c1...@reviews.llvm.org<mailto:reviews%2bd41081%2bpublic%2b5a71b504a12c1...@reviews.llvm.org>; Zhen Cao mailto:zhen@autodesk.com>>; r...@google.com<mailto:r...@google.com> Subject: Re: [PATCH] D41081: Fix clang Lexer W

Re: [PATCH] D41081: Fix clang Lexer Windows line-ending bug

2017-12-11 Thread Zachary Turner via cfe-commits
are run. > > Cheers, > > Benoit > > > > *From: *Zachary Turner > *Date: *lundi 11 décembre 2017 à 15:22 > *To: *"reviews+d41081+public+5a71b504a12c1...@reviews.llvm.org" < > reviews+d41081+public+5a71b504a12c1...@reviews.llvm.org> > *Cc: *Zhen Cao

RE: [PATCH] D41081: Fix clang Lexer Windows line-ending bug

2017-12-11 Thread Zhen Cao via cfe-commits
: Zachary Turner [mailto:ztur...@google.com] Sent: Monday, December 11, 2017 3:42 PM To: Benoit Belley Cc: reviews+d41081+public+5a71b504a12c1...@reviews.llvm.org; Zhen Cao ; r...@google.com Subject: Re: [PATCH] D41081: Fix clang Lexer Windows line-ending bug To be honest, I'm not a fan of te

Re: [PATCH] D41081: Fix clang Lexer Windows line-ending bug

2017-12-11 Thread Zachary Turner via cfe-commits
See my other response. Maybe we don’t even need a substitution at all? On Mon, Dec 11, 2017 at 12:24 PM Benoit Belley via Phabricator < revi...@reviews.llvm.org> wrote: > belleyb added inline comments. > > > > Comment at: test/lit.cfg.py:52-57 > +if platform.system() in ['Windows'