[PATCH] D142121: [clang-tidy] Refactor common functionality of HeaderGuardCheck into HeaderGuardBase

Kyle Edwards via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 19 10:21:04 PST 2023


KyleFromKitware added a comment.

Yes, this is for D142123 <https://reviews.llvm.org/D142123>. However, even if that doesn't land, I'd still like to land this. We use the code from D142123 <https://reviews.llvm.org/D142123> in CMake's CI system, and getting it to work without this change resulted in a massive amount of code duplication. This refactoring would allow us to eliminate such duplication on our end.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142121



More information about the cfe-commits mailing list