================
@@ -10356,15 +10356,24 @@ void Sema::CheckMemaccessArguments(const CallExpr
*Call,
bool NonTriviallyCopyableCXXRecord =
getLangOpts().CPlusPlus && RD->isCompleteDefinition() &&
!PointeeTy.isTriviallyCopyableType(Context);
+ // We don't warn about bzero or zero memsetting as these are an idiomatic
+ // mechanism for ensuring objects do not have stale data.
+ bool IsNonZeroInitMemcall = false;
----------------
cor3ntin wrote:
Maybe that should be initialized to `BId != Builtin::BIbzero` (I'm not quite
sure how the new code handle non memset, non bzero builtins) - and also maybe
call the variable `IsZeroInitMemcall` to avoid double negatives
https://github.com/llvm/llvm-project/pull/170577
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
