
<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On May 3, 2012, at 4:53 PM, Chandler Carruth wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote">On Thu, May 3, 2012 at 11:39 AM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; ">

I'm adding Doug as this is changing the AST in a non-trivial way, and his review is probably best there.<div><br></div><div>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.<br>

</div></blockquote><div><br></div><div>FYI, after chatting on IRC, Doug OK-ed this patch.</div><div><br></div><div>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.</div>

<div> </div></div>

</blockquote></div><div>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...</div><br><div>-bw</div><div><br></div></body></html>