[PATCH] D142123: [clang-tidy] Add check to suggest use of #pragma once

Kyle Edwards via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 19 08:52:59 PST 2023


KyleFromKitware added a comment.

We have this check in a custom clang-tidy module that runs on CMake's CI system. Other people <https://stackoverflow.com/questions/61404441/how-can-i-tell-clang-tidy-to-check-for-pragma-once-instead-of-llvm-style-header> have also sought out such a check, and I figured it made sense to upstream it instead of keeping it stuck in our CI. Even though it's not standard, other people might still want to use it. Perhaps there's a better category for it than `modernize`?


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

https://reviews.llvm.org/D142123



More information about the cfe-commits mailing list