https://github.com/efriedma-quic commented:

It probably makes sense to add a helper like the following to CodeGenModule or 
something like that:

```
bool shouldZeroInitPadding() { 
  // Comment explaining reasoning for this rule
  return !CGM.getLangOpts().CPlusPlus;
}
```

So we don't have `// See comment in getPadding().` all over the place.

----

Is there any relevant code using EmitNullConstant()?  As far as I can tell, 
this patch doesn't  zero-fill implicit padding in null constants (like in 
`struct { int x; long y; }`.

https://github.com/llvm/llvm-project/pull/97121
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to