aganea added a comment.
I'm just wondering if D96363 <https://reviews.llvm.org/D96363> and all attached
subsequent patches shouldn't be reverted for now. This is quite trivial case
uncovered by tests. On re-land, I would then add a test validating the issue on
Windows:
$ cat -A rewrite-includes-clang-cl.cpp
// REQUIRES: windows^M$
// RUN: %clang_cl /E -Xclang -frewrite-includes %s | %clang_cl /c
/clang:-verify /Tp -^M$
// expected-no-diagnostics^M$
^M$
int foo();^M$
int bar();^M$
#define HELLO \^M$
foo(); \^M$
bar();^M$
^M$
int main() {^M$
HELLO^M$
return 0;^M$
}^M$
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99363/new/
https://reviews.llvm.org/D99363
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
