[llvm-commits] [PATCH] some cleanup in DataLayout
Chris Lattner
clattner at apple.com
Tue Nov 13 23:01:58 PST 2012
On Nov 13, 2012, at 5:00 AM, Patrik Hägglund H <patrik.h.hagglund at ericsson.com> wrote:
> Here are two minor cleanup patches for DataLayout. (Found during preparing a series of patches for non-8-bit bytes.)
>
> Subject: [PATCH 1/2] Revert some redundant parts of r142605.
>
> This seems like redundant leftovers from r142288 - exposing
> TargetData::parseSpecifier to LLParser - which got reverted. Removes
> redunant td != NULL checks in parseSpecifier, and simplifies the
> interface to parseSpecifier and init.
Looks great to me,
> Subject: [PATCH 2/2] Change size of VoidTyID from 8 to 0 bits.
>
> According to Type.h, VoidTyID is a "type with no size".
This makes less sense. Asking for the size of void is nonsensical, and void can only be the result of a function. I'd rather that this function abort when asked for the size of void, but we'd obviously need to update the docs for the API and make sure no tests fail due to that.
-Chris
More information about the llvm-commits
mailing list