[PATCH] Update NumTypeBits on Type to match the actual number of used bits.

Mark Lacey mark.lacey at apple.com
Wed Sep 25 13:43:19 PDT 2013


On Sep 25, 2013, at 1:35 PM, Eric Christopher <echristo at gmail.com> wrote:

> On Wed, Sep 25, 2013 at 9:45 AM, Mark Lacey <mark.lacey at apple.com> wrote:
>> Ping?
>> 
>> The change is simple and I suspect the current value there was an oversight from a past commit, so if there are no objections I’ll commit later today.
>> 
> 
> We don't generally do this. Once it's out it is generally considered
> polite to wait for review especially since a large number of the C++
> people are at the committee meeting this week.

Sorry - I didn’t mean to sound pushy and hadn’t considered that there was a committee meeting this week.

> That said two attributes were merged to one in r176164 and the
> bitfield wasn't updated so LGTM.

Thanks for taking a look!

Mark

> 
> -eric

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130925/fd12e0a2/attachment.html>


More information about the cfe-commits mailing list