jfb marked an inline comment as done.
jfb added inline comments.
================
Comment at: lib/AST/ASTContext.cpp:1965
+ Width = Target->getCharWidth();
+ Align = Target->getCharWidth();
+ } else if (Width <= Target->getMaxAtomicPromoteWidth()) {
----------------
rjmccall wrote:
> Alignment, unlike size, is definitely never 0. I think you should leave the
> original alignment in place; it's a weird case, but we honor over-aligned
> empty types in a bunch of other places.
Yeah that makes total sense. I turned it to an assert.
Repository:
rC Clang
https://reviews.llvm.org/D46613
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
