[cfg] Assign new check profiles for 6.22RC1 #3861
Conversation
gamesh411
force-pushed
the
6.22RC1-chekerprofiles
branch
from
f6712d8
to
cbc7a77
Compare
| @@ -4798,6 +4847,12 @@ | |||
| "profile:extreme", | |||
| "severity:LOW" | |||
| ], | |||
| "cppcoreguidelines-rvalue-reference-param-not-moved": [ | |||
There was a problem hiding this comment.
Shouldn't this be in "default" profile?
There was a problem hiding this comment.
This should not be in the default profile, because if only parts of the object are moved but not the whole object, then by default, the checker would warn. I don't have results to support my claim, but I think this could lead to annoying warnings.
I would leave this out of the default profile for now.
| "profile:sensitive", | ||
| "severity:LOW" | ||
| ], | ||
| "cppcoreguidelines-avoid-capturing-lambda-coroutines": [ |
There was a problem hiding this comment.
We use "high" severity if it causes runtime error. Does this checker indicate a runtime error? Maybe "default" profile?
There was a problem hiding this comment.
This can indeed directly lead to UB, so I would keep the HIGH severity.
There was a problem hiding this comment.
I have added it to the default profile.
- llvmlibc-inline-function-decl has STYLE instead of LOW severity - cppcoreguidelines-avoid-reference-coroutine-parameters is now in default profile
- cppcoreguidelines-avoid-capturing-lambda-coroutines is now in default profile
whisperity
added
analyzer 📈
No description provided.