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

[Douglas Gregor]

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/llvm-commits/attachments/20120503/2cf1e190/attachment.html>