dexonsmith added a comment.

In D66556#1647762 <https://reviews.llvm.org/D66556#1647762>, @dexonsmith wrote:

> In D66556#1646229 <https://reviews.llvm.org/D66556#1646229>, @aganea wrote:
>
> > This failed the build - I've changed `unix2dos` to `svn:eol-style CRLF` 
> > instead.
>
>
> Can we count on `tr` when we have `shell`?  If so, I suggest this instead:
>
>   // REQUIRES: shell
>   // RUN: tr '\n' '\r\n' <%s | %clang_cc1 -DOTHER 
> -print-dependency-directives-minimized-source 2>&1 | FileCheck %s
>


Eh, nevermind, `tr` doesn't handle multi-character replacements.  Better to 
answer Alex's question.


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

https://reviews.llvm.org/D66556



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to