[cfe-commits] small bugfix to lib/Sema/SemaDeclAttr.cpp
supertri at google.com
Thu Jun 30 12:13:14 PDT 2011
Here is an updated version of the second patch, incorporating feedback
from code review.
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).
> 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:
>> 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.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 13881 bytes
Desc: not available
More information about the cfe-commits