<div dir="ltr">It shouldn't be too hard to get the first part done at least with the MCOptions part.<br><br>-eric</div><br><div class="gmail_quote">On Mon Feb 23 2015 at 9:31:17 AM Vladimir Medic <<a href="mailto:Vladimir.Medic@imgtec.com">Vladimir.Medic@imgtec.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div>
<div style="direction:ltr;font-family:Tahoma;color:#000000;font-size:10pt">
<div style="direction:ltr;font-family:Tahoma;color:#000000;font-size:10pt">Could we do that in a separate patch? I would prefer to submit this one without initialization with old fixme comment and commit it in order to get the abi working. In the next patch
 we can initialize MCtargetOptions from AssemblerInvocation(adding the fields we need in the structure).<br>
<div style="font-family:Times New Roman;color:#000000;font-size:16px">
<hr>
<div style="direction:ltr"><font face="Tahoma" color="#000000"><b>From:</b> Eric Christopher [<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>]<br>
<b>Sent:</b> Friday, February 20, 2015 6:54 PM<br>
<b>To:</b> <a href="mailto:reviews%2BD7747%2Bpublic%2Bd517a63d5c1d02a4@reviews.llvm.org" target="_blank">reviews+D7747+public+d517a63d5c1d02a4@reviews.llvm.org</a>; Vladimir Medic; Daniel Sanders<br>
<b>Cc:</b> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<b>Subject:</b> Re: [PATCH] [clang][Driver][MCTargetOptions]Change clang driver to adopt new definition of createAsmBackend<br>
</font><br>
</div></div></div></div></div><div><div style="direction:ltr;font-family:Tahoma;color:#000000;font-size:10pt"><div style="direction:ltr;font-family:Tahoma;color:#000000;font-size:10pt"><div style="font-family:Times New Roman;color:#000000;font-size:16px">
<div></div>
<div>
<div dir="ltr"><br>
<br>
<div class="gmail_quote">On Fri Feb 20 2015 at 6:10:02 AM Vladimir Medic <<a href="mailto:Vladimir.Medic@imgtec.com" target="_blank">Vladimir.Medic@imgtec.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
================<br>
Comment at: tools/driver/cc1as_main.cpp:<u></u>358<br>
@@ -357,2 +357,3 @@<br>
       TheTarget-><u></u>createMCSubtargetInfo(Opts.<u></u>Triple, Opts.CPU, FS));<br>
+  MCTargetOptions Options = InitMCTargetOptionsFromFlags()<u></u>;<br>
<br>
----------------<br>
echristo wrote:<br>
> I'd prefer not to do this, clang already has its own option handling. Ideas on how to avoid this?<br>
Not right now, at least not a simple one. The FIXME comment that was removed said:"init MCTargetOptions from sanitizer flags here." So we should probably need to map clang options to MCTargetOptions.<br>
<br>
</blockquote>
<div><br>
</div>
<div>I think we should do that.</div>
<div><br>
</div>
<div>In particular here you should map the -mabi option.</div>
<div><br>
</div>
<div>Thanks!</div>
<div><br>
</div>
<div>-eric</div>
<div> </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<a href="http://reviews.llvm.org/D7747" target="_blank">http://reviews.llvm.org/D7747</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">
http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
<br>
<br>
</blockquote>
</div>
</div>
</div>
</div></div></div></div></blockquote></div>