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

[Chandler Carruth]

On Thu, May 3, 2012 at 10:53 PM, Douglas Gregor <dgregor at apple.com> wrote:

>
>
> 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
>

I also agree fwiw.... my comment here was only meant to try to let folks
know about this. =]
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120503/c1224b59/attachment.html>