[cfe-commits] [llvm-commits] PATCH: Fix non-deterministic debug info output for C++ inputs
chandlerc at google.com
Thu May 3 22:57:45 PDT 2012
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'
> 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...
More information about the cfe-commits