[cfe-commits] [llvm-commits] PATCH: Fix non-deterministic debug info output for C++ inputs

Douglas Gregor dgregor at apple.com
Thu May 3 22:53:37 PDT 2012

Sent from my iPhone

On May 3, 2012, at 6:58 PM, Bill Wendling <isanbard at gmail.com> wrote:

> On May 3, 2012, at 4:53 PM, Chandler Carruth wrote:
>> On Thu, May 3, 2012 at 11:39 AM, Chandler Carruth <chandlerc at gmail.com> wrote:
>> I'm adding Doug as this is changing the AST in a non-trivial way, and his review is probably best there.
>> Also, Bill, I'm CC-ing you because I'm worried this might be a release blocker... The bug has been here for a long time, I'm not even sure if its a regression, but we noticed this in the wild (not with contrived code), and non-deterministic output seems really really bad.
>> FYI, after chatting on IRC, Doug OK-ed this patch.
>> He also didn't think it was likely a release blocker. If anyone disagrees or cares greatly about the deterministic output of debug info in 3.1, give a shout.
> While it would be great to have no bugs, I don't know if I can accept this as a release blocker if it's not a major regression from 3.0. I already tagged release candidate 2, so any changes now would disrupt the testers' lives...

I agree. This does not need to go into 3.1
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120503/2cf1e190/attachment.html>

More information about the cfe-commits mailing list