[PATCH] D68115: Zero initialize padding in unions

JF Bastien via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 27 14:52:52 PDT 2020


jfb added a comment.

In D68115#1946757 <https://reviews.llvm.org/D68115#1946757>, @rsmith wrote:

> In D68115#1946668 <https://reviews.llvm.org/D68115#1946668>, @hubert.reinterpretcast wrote:
>
> > It sounds like we are looking for `-fzero-union-padding`. That's been where the discussion has left off twice for months.
>
>
> I believe the state of Clang prior to this patch is actually wrong.


That's my understanding as well. I'd like it if this patch (or a follow-up) got us back to standard behavior. In either case, I'd like the proposed behavior to bee on-by-default when doing initialization of stack variables.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68115





More information about the cfe-commits mailing list