[PATCH] D52738: [AST] Pack the bit-fields of FunctionProtoType into Type.
John McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 1 11:48:23 PDT 2018
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
The `reinterpret_cast` is legal because it's dynamic storage that's never actually accessed as an `ExceptionType`. IIRC there are reasonable-sounding strict-aliasing arguments about the validity of doing pointer arithmetic using pointers of the wrong type for the objects stored there, but they've always come with a "and if we actually enforced the rules that way, we'd completely break the world" disclaimer.
I agree with all the changes you've made here; thank you for seeing this through.
Repository:
rC Clang
https://reviews.llvm.org/D52738
More information about the cfe-commits
mailing list