<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Jul 7, 2015 at 1:27 PM Artem Belevich <<a href="mailto:tra@google.com">tra@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D9509-23200597&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=WSNsEpOwM184DNtRfppV_AOOrMSjUUlprrcudVwUrno&s=YmR4OF9Ktnm-QrfOFZ1VjV4Wi8f1-VW6qeKtltG2HYk&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D9509#200597</a>, @echristo wrote:<br>
<br>
> Couple of inline comments, and I'm still not a huge fan of the getTargetToolChain bits. It seems like we should be able to have a single interface to getting a target toolchain that works for both cases. Maybe compute the triple up front for use in the rest of the driver for the host?<br>
<br>
<br>
Existing code commingles two independent things -- figuring out target triple value and retrieving corresponding ToolChain for a given Triple. My change just refactored out Triple->ToolChain mapping.<br>
<br></blockquote><div><br></div><div>Right. That's a good start for sure. Let's also see if we can refactor out BindArchAction to be a Triple (or stringref of triple) instead of passing DarwinArchName around and just use the triple.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
----------------<br>
echristo wrote:<br>
> Can you rearrange the code so that it doesn't need the forward declaration?<br>
Nope. PrintActions1 and PrintActionList call each other recursively. At least one of them must have forward declaration.<br></blockquote><div><br></div><div>Hrm.</div><div><br></div><div>-eric </div></div></div>