aaron.ballman added a comment.

In D91000#2469898 <https://reviews.llvm.org/D91000#2469898>, @lebedev.ri wrote:

> I think the question is, *why* are these checks being implemented?
> Just to claim that for some particular rule there is a check, and cross it 
> off a list?
> Or for them to be actually used?

We have an existing CERT module for CERT-specific checks. When the check is 
honoring a published coding standard, that's usually sufficient justification 
-- but I do think the check should implement what the CERT recommendation says. 
If the CERT recommendation seems deficient in some way, it's reasonable to 
reach out to the authors (they allow comments on their wiki and are pretty good 
about responding to those comments) for clarification or to discuss potential 
improvements to the recommendation that would make the check more palatable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91000

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

Reply via email to