[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