[cfe-commits] small bugfix to lib/Sema/SemaDeclAttr.cpp
Caitlin Sadowski
supertri at google.com
Thu Jun 30 14:27:27 PDT 2011
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_removefalse.patch
Type: text/x-patch
Size: 1564 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110630/e30171fb/attachment.bin>
More information about the cfe-commits
mailing list