Skip to content

Commit

Permalink
clang: Make rewrite-includes-macros.cpp runnable on non-Win
Browse files Browse the repository at this point in the history
As far as I can tell, there's nothing Windows-specific about the
test and it passes fine on other platforms.

I found this test when running

    rg clang_cl clang/test | rg '%s' | rg -v -- ' -- ' | rg -v not

after 547ee1c to see if other tests were missing `--`
before `%s` in `%clang_cl` invocations. This was the only one.
Since it used to run only on Windows, it wasn't needed, but as far
as I can tell there's no reason to run it only on Windows.

Differential Revision: https://reviews.llvm.org/D158279
  • Loading branch information
nico committed Aug 19, 2023
1 parent 985a72b commit 960881a
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions clang/test/Frontend/rewrite-includes-macros.cpp
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
// REQUIRES: system-windows
// RUN: %clang_cl /E -Xclang -frewrite-includes %s | %clang_cl /c -Xclang -verify /Tp -
// RUN: %clang_cl /E -Xclang -frewrite-includes -- %s | %clang_cl /c -Xclang -verify /Tp -
// expected-no-diagnostics

// This test uses dos-style \r\n line endings.
// Make sure your editor doesn't rewrite them to unix-style \n line endings.
int foo();
int bar();
#define HELLO \
Expand Down

0 comments on commit 960881a

Please sign in to comment.