Izaron added a comment. In D18914#2399705 <https://reviews.llvm.org/D18914#2399705>, @chfast wrote:
> This check can be useful in other case like this: > > inline constexpr const int x = 1; > > where `inline` and `const` are redundant. In your case `inline` is not actually redundant (at least for a global variable). `const` (therefore `constexpr`) means internal linkage, so every translation unit will just get itself an unique internal copy of this variable. We can see it if we print the address of `x` in different translation units. The addresses will be different without `inline` and the same with `inline`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D18914/new/ https://reviews.llvm.org/D18914 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
