Hi Jordan, DeLesley, Please find the patch attached for issue reviewed/discussed at http://llvm-reviews.chandlerc.com/D1529 .
Thanks Karthik On Sat, Aug 31, 2013 at 5:31 AM, Jordan Rose <[email protected]> wrote: > The diff on Phabricator generally only works if you include the full file > as context (or generate it through the "arc" tool). This is probably a > Phabricator bug, since the line numbers are clearly in the rich display but > not through "Download Raw Diff". Can you upload the patch one more time > with full context? > > On Aug 30, 2013, at 10:21 , Karthik Bhat <[email protected]> > wrote: > > Thanks Jordan,Ted,Richard, > Could someone commit it for me as i don't have the permissions. > Thanks! > > > On Fri, Aug 30, 2013 at 10:44 PM, Ted Kremenek <[email protected]> wrote: > >> Looks good to me. >> >> On Aug 30, 2013, at 9:46 AM, Jordan Rose <[email protected]> wrote: >> >> > >> > P.S. Please remember to add cfe-commits as a CC for patches. >> > >> > http://llvm-reviews.chandlerc.com/D1529 >> > _______________________________________________ >> > cfe-commits mailing list >> > [email protected] >> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >> >> > >
dtorPatch.diff
Description: Binary data
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
