[PATCH] D158279: clang: Make rewrite-includes-macros.cpp runnable on non-Win

2023-08-19 Thread Nico Weber via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG960881a7f32a: clang: Make rewrite-includes-macros.cpp runnable on non-Win (authored by thakis). Herald added a project: clang. Repository: rG

[PATCH] D158279: clang: Make rewrite-includes-macros.cpp runnable on non-Win

2023-08-18 Thread Alexandre Ganea via Phabricator via cfe-commits
aganea accepted this revision. aganea added a comment. This revision is now accepted and ready to land. Sure. This was meant to catch an issue that only happened on Windows but it should work on all platforms. LGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158279/new/

[PATCH] D158279: clang: Make rewrite-includes-macros.cpp runnable on non-Win

2023-08-18 Thread Nico Weber via Phabricator via cfe-commits
thakis created this revision. thakis added a reviewer: aganea. Herald added a project: All. thakis requested review of this revision. 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