<div dir="ltr">Wait, can you give a rundown of what your TargetTuple is doing etc? Maybe a mail to the list? This seems weird.<br><div><br></div><div>Thanks!</div><div><br></div><div>-eric</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jul 7, 2015 at 8:13 AM Daniel Sanders <<a href="mailto:daniel.sanders@imgtec.com">daniel.sanders@imgtec.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> About the Tuple+CPU+Features (and others), unless you're planning to do that move in this streak, I think it'd be best to keep a small FIXME comment to use CPU/Features/Options in the Tuple itself.<br>
<br>
<br>
I'm not planning to merge CPU name and Features into the TargetTuple in this series, but I probably should do this at some point. I'll add some FIXME's for now<br>
<br>
<br>
================<br>
Comment at: include/llvm/Support/TargetRegistry.h:119<br>
@@ -117,1 +118,3 @@<br>
+                                              const TargetTuple &TT,<br>
+                                              StringRef CPU);<br>
   typedef MCTargetAsmParser *(*MCAsmParserCtorTy)(<br>
----------------<br>
rengolin wrote:<br>
> I think this one deserves a FIXME. CPU should be already uniquely identifiable in the tuple.<br>
> CPU should be already uniquely identifiable in the tuple.<br>
<br>
I don't think that's true at the moment since Triples/TargetTuples lack the variety of values that the CPU name has. For example, -mcpu=pentium would cause CPU to be 'pentium' but the Triple/TargetTuple doesn't know about this string.<br>
<br>
Are you thinking of the SubArch field that ARM and Kalimba use?<br>
<br>
<br>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10969&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=4LjXUIFyZ4WXQ32y1zNH1l2Z8knUEQMZ6x10rA5c0zU&s=O38ojraZjcbABN6ERHK0PQHvdW_5FeFmAh_1moMr4Gs&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D10969</a><br>
<br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>