[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).



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