[PATCH] D118743: [clang-tidy] Add `modernize-use-inline-const-variables-in-headers` check

Evgeny Shulgin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 1 16:11:50 PST 2022


Izaron added a comment.

For example, Google Code Style strongly advises C++17 `inline variables`. There are company guides how to use them: `https://abseil.io/tips/168`, `https://abseil.io/tips/140`. I believe other codestyles also use the feature.

//P.S. If this review is eventually approved, kindly please merge the commit on my behalf =) As I don't have merge access. My name is `Evgeny Shulgin` and email is `izaronplatz at gmail.com`. Sorry for inconvenience!//


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118743



More information about the cfe-commits mailing list