<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Jul 16, 2011, at 11:50 PM, David Blaikie wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">happy to checkout dragonegg & fix that too if it's practical.<br></blockquote><div><br>
Here's a blind attempt (since I don't have gcc source checked out) at fixing dragonegg, too - given the sed replacement worked universally in llvm & clang, I'd say the chances are good that this'll just work. (given my look at dragonegg, it doesn't seem to have any internal types called "Type" (unlike clang, where I only replaced the fully qualified "llvm::Type" due to the existence of "clang::Type"))<br>
<br>But yeah - if this avenue is still not preferred let me know & I'll look into making the llvm changes without the clang (using const cast on the parameters to llvm functions used by clients) changes & use that (clang's build) to verify that the change doesn't break clients.<br></div></div></blockquote></div><br><div>Applied in r135371, thanks! I'll apply your original patch as well after fixing llvm-gcc.</div><div><br></div><div>-Chris</div></body></html>