[PATCH] D32001: [APInt] Reorder fields to avoid a hole in the middle of the class
Hans Wennborg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 13 08:38:18 PDT 2017
hans added a comment.
In https://reviews.llvm.org/D32001#725651, @davide wrote:
> I wonder if there's an easy way of testing this so that it doesn't break in the future? Maybe an unit test checking `sizeof()`. Thoughts?
Clang (and GCC too I think) has a -Wpadded parning. Maybe we could #pragma enable it around this class? We have a lot of classes that are supposed to be carefully packed though.
Repository:
rL LLVM
https://reviews.llvm.org/D32001
More information about the llvm-commits
mailing list