KyleFromKitware added a comment.

In D142123#4066351 <https://reviews.llvm.org/D142123#4066351>, @njames93 wrote:

> and peoples eagerness to blindly enable all checks (or all checks from a 
> module)

Perhaps there's some way we can have it disabled by default and not enabled 
unless they explicitly enable it (similar to `-Wall` vs 
`-Wextra`/`-Weverything`)?

I agree that it should not be standard, but it works in the vast majority of 
use cases. Drake <https://github.com/RobotLocomotion/drake> is another very 
large C++ project that uses `#pragma once` everywhere. The demand for `#pragma 
once` exists, as does the demand for a check to enforce it. If we don't land 
this in LLVM proper, where would be a better place for it?


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

https://reviews.llvm.org/D142123

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to