[PATCH] D115715: [clang-tidy] Fix llvm-header-guard for Windows paths containing drive letter (e.g. C:).

Salman Javed via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 14 09:50:06 PST 2021


salman-javed-nz added a comment.

In D115715#3192085 <https://reviews.llvm.org/D115715#3192085>, @aaron.ballman wrote:

> It could be made to be useful outside of LLVM, but as it stands today, the check is only intended to be useful for LLVM. If we want to make it useful outside of LLVM, that's fine, but there's a bit more to do (for example, the check should be exposed outside of the `llvm` module) and that includes generalizing the check.

There is https://reviews.llvm.org/D61508 which tries to generalise the llvm-header-guard check. Might be good to build on top of this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115715



More information about the cfe-commits mailing list