[cfe-commits] small bugfix to lib/Sema/SemaDeclAttr.cpp

Caitlin Sadowski supertri at google.com
Wed Jun 29 10:36:00 PDT 2011


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.patch
Type: text/x-patch
Size: 14957 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110629/5293bb3c/attachment.bin>


More information about the cfe-commits mailing list