Move getLocStart() in Sema::ConvertArgumentsForCall

Keno Fischer kfischer at college.harvard.edu
Wed Mar 19 11:53:20 PDT 2014


I'm not sure I can come up with a reasonably well isolated test case. I
encountered this in an application that uses clang as a library. From what
I can see most of the test cases use the clang driver to test for bugs. Is
there any examples of test cases that go through the C++ API (since I know
how to trigger it there)?


On Wed, Mar 19, 2014 at 11:07 AM, David Blaikie <dblaikie at gmail.com> wrote:

> A test case would be good, if possible
>
> On Tue, Mar 18, 2014 at 9:44 PM, Keno Fischer
> <kfischer at college.harvard.edu> wrote:
> > Sema::ConvertArgumentsForCall calls setNumArgs on the CallExpr. This
> changes
> > the result of getLocStart if the SourceLocation stored in the CallExpr is
> > invalid and in that case may lead to an assertion failure. This quick
> patch
> > fixes that.
> >
> > I don't have commit access so let me know if anything else is needed.
> >
> > _______________________________________________
> > cfe-commits mailing list
> > cfe-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140319/8787798b/attachment.html>


More information about the cfe-commits mailing list