r222906 - Create a new 'flag_enum' attribute.

Hans Wennborg hans at chromium.org
Tue Jan 20 11:30:01 PST 2015


Reverted in r226593.

Thanks,
Hans

On Tue, Jan 20, 2015 at 9:48 AM, Aaron Ballman <aaron at aaronballman.com> wrote:
> Seconded; Hans, can you take care of this? It should be relatively
> safe as the feature was not in 3.5.
>
> Thanks!
>
> ~Aaron
>
> On Tue, Jan 20, 2015 at 12:46 PM, Sean Hunt <scshunt at csclub.uwaterloo.ca> wrote:
>> On Thu, Nov 27, 2014 at 7:53 PM, Sean Hunt <scshunt at csclub.uwaterloo.ca> wrote:
>>> Author: coppro
>>> Date: Thu Nov 27 18:53:20 2014
>>> New Revision: 222906
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=222906&view=rev
>>> Log:
>>> Create a new 'flag_enum' attribute.
>>>
>>> This attribute serves as a hint to improve warnings about the ranges of
>>> enumerators used as flag types. It currently has no working C++ implementation
>>> due to different semantics for enums in C++. For more explanation, see the docs
>>> and testcases.
>>>
>>> Reviewed by Aaron Ballman.
>>
>> After some discussion with dgregor, I've started work on a patch to
>> fix a few issues identified with this. Unfortunately, though, I don't
>> have the time to do this in time for 3.6. Can this please be reverted
>> on the 3.6 branch?
>>
>> Sean
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits



More information about the cfe-commits mailing list