[cfe-dev] Broken sizeof(expr) location in SemaExpr.cpp

Xi Wang xi.wang at gmail.com
Sat May 28 15:09:57 PDT 2011


Cool.  Thanks a lot.  I would suggest to add the two parameters OpLoc
and ArgRange back.  This interface change also affects Sema.h and
TreeTransform.h.

BTW, the assertion you wanted might be "ArgEx->getExprLoc() == OpLoc" ;-)

On Sat, May 28, 2011 at 5:53 PM, Chandler Carruth <chandlerc at google.com> wrote:
> On Sat, May 28, 2011 at 1:53 PM, John McCall <rjmccall at apple.com> wrote:
>> This does seem wrong.  Since you've done so much of the
>> investigation on it, care to write up a patch? :)
>
> I'll look at this tonight. It's my bug, I made the change, and I see
> clearly the error. The weird part is that I tried to add an assertion
> to catch just this error and it didn't fire.
>




More information about the cfe-dev mailing list