No problem. Could either of you commit this for me? Thanks,
-- Geoff Berry Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -----Original Message----- From: Diego Novillo [mailto:[email protected]] Sent: Friday, May 08, 2015 11:42 AM To: Duncan P. N. Exon Smith; [email protected] Cc: [email protected]; cfe commits Subject: Re: [PATCH] Fix crash in handling back-end warnings On Fri, May 8, 2015 at 8:28 AM, Duncan P. N. Exon Smith <[email protected]> wrote: > I probably broke this at some point when I was transitioning to the > new debug info hierarchy. The old hierarchy's accessors checked for > NULL and give default answers, whereas the new hierarchy just uses > pointers directly. > > At some point I switched the accessors to start asserting and fixed > any bugs that uncovered, but I guess there wasn't any test coverage > here so I missed it. Yeah. The backend tests that we have for diagnostics all include debug info and if you go via the FE, it always makes sure that debuglocs are always generated. Geoff, your patch LGTM. Thanks for the fix! Diego. _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
