[all-commits] [llvm/llvm-project] 960881: clang: Make rewrite-includes-macros.cpp runnable o...

Nico Weber via All-commits all-commits at lists.llvm.org
Sat Aug 19 07:48:00 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 960881a7f32abc50fd0e5a349a9981437eba221a
      https://github.com/llvm/llvm-project/commit/960881a7f32abc50fd0e5a349a9981437eba221a
  Author: Nico Weber <thakis at chromium.org>
  Date:   2023-08-19 (Sat, 19 Aug 2023)

  Changed paths:
    M clang/test/Frontend/rewrite-includes-macros.cpp

  Log Message:
  -----------
  clang: Make rewrite-includes-macros.cpp runnable on non-Win

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 547ee1c81fceaabcb 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




More information about the All-commits mailing list