[PATCH] D32456: [ubsan] Make the alignment check work with some extern decls (PR32630)

Vedant Kumar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 25 13:18:48 PDT 2017


vsk added a comment.

In https://reviews.llvm.org/D32456#737228, @efriedma wrote:

> > If the alignment isn't explicitly set, it is initialized to 0.
>
> That's what I thought.  I don't like this; clang should explicitly set an alignment for every global.


Ok, I will set this aside for now and think of an alternate approach to address PR32630.


https://reviews.llvm.org/D32456





More information about the cfe-commits mailing list