[cfe-commits] small patch for lib/Sema/SemaDeclAttr.cpp

Caitlin Sadowski supertri at google.com
Thu Jun 30 15:24:22 PDT 2011


Here is a patch that consistifies Decl parameters in functions to be
all "D" instead of "d", AttributeList parameters to functions to be
"Attr" instead of "attr".

Cheers,

Caitlin

On Thu, Jun 30, 2011 at 2:27 PM, Caitlin Sadowski <supertri at google.com> wrote:
> Sorry, forgot to attach the patch.
>
> Cheers,
> Caitlin
>
> On Thu, Jun 30, 2011 at 2:26 PM, Caitlin Sadowski <supertri at google.com> wrote:
>> And here is a third patch that replaces instances of:
>>
>> "Attr.isInvalid() == false" with "!Attr.isInvalid()"
>>
>> Cheers,
>>
>> Caitlin
>>
>> On Thu, Jun 30, 2011 at 12:13 PM, Caitlin Sadowski <supertri at google.com> wrote:
>>> Here is an updated version of the second patch, incorporating feedback
>>> from code review.
>>>
>>> Cheers,
>>>
>>> Caitlin
>>>
>>> On Wed, Jun 29, 2011 at 10:36 AM, Caitlin Sadowski <supertri at google.com> wrote:
>>>> Actually, I meant to send out this version of the second patch
>>>> (identical except that there is unsigned int instead of int in the
>>>> helper function).
>>>>
>>>> Cheers,
>>>>
>>>> Caitlin
>>>>
>>>> On Tue, Jun 28, 2011 at 3:23 PM, Caitlin Sadowski <supertri at google.com> wrote:
>>>>> Here is a second refactoring patch, building on the 1 character patch
>>>>> sent out previously, which refactors SemaDeclAttr to use a helper
>>>>> method when checking if the number of args matches, if the expected
>>>>> number is the same as the check.
>>>>>
>>>>> Also, feel free to give code review comments:
>>>>>
>>>>> http://codereview.appspot.com/4635075
>>>>>
>>>>> Cheers,
>>>>>
>>>>> Caitlin
>>>>>
>>>>> On Mon, Jun 27, 2011 at 6:44 PM, Caitlin Sadowski <supertri at google.com> wrote:
>>>>>> This bugfix corrects a suspected mismatch between the number of
>>>>>> arguments expected and the number listed in the error message if less
>>>>>> arguments are presented in HandleReqdWorkGroupSize.
>>>>>>
>>>>>> Cheers,
>>>>>>
>>>>>> Caitlin
>>>>>>
>>>>>
>>>>
>>>
>>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: refactor_SemaDeclAttr_toUpper.patch
Type: text/x-patch
Size: 77021 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110630/10b7ac01/attachment.bin>


More information about the cfe-commits mailing list