<div class="gmail_quote">On Tue, Oct 19, 2010 at 12:12 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5">On 18 October 2010 20:45, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>
> This patch adds support for the --sysroot flag. It does so by passing the<br>
> -isysroot down to Clang's CC1. The existing logic already passed the flag to<br>
> the linker, which may or may not reject in on some systems.<br>
> Any concerns about this implementation strategy?<br>
<br>
</div></div>The comment looks correct, but aren't you adding a -isysroot to the<br>
cc1 invocation even if we have an explicit -isysroot?<br></blockquote><div><br></div><div>Doh! Added that bit.</div></div><br>