<div class="gmail_quote">+  /// Retrieve a formated string of the default features for<br>+  /// the specified target triple.<br>+  static std::string getDefaultSubtargetFeatures(const Triple &Triple);<br><br>Two t's in formatted.<br>

<br>2009/11/18 Rafael Espindola <span dir="ltr"><<a href="mailto:espindola@google.com">espindola@google.com</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

2009/11/17 Viktor Kutuzov <<a href="mailto:vkutuzov@accesssoftek.com">vkutuzov@accesssoftek.com</a>>:<br>
<div class="im">> Thanks for reviewing it, Rafael.<br>
><br>
> Now while we are both on the same waive, let's go with the rest of the patches.<br>
<br>
</div>Have you committed the previous one?<br>
<div class="im"><br>
> I have tried to combine them on the same manner as the last one.<br>
<br>
</div>Thanks!<br>
<br>
You have<br>
<br>
+        const std::string FeatureStr =<br>
+            SubtargetFeatures::getDefaultSubtargetFeatures(Triple.c_str());<br>
<br>
But getDefaultSubtargetFeatures takes a triple. You have to change the<br>
.c_str() to llvm::Triple().<br>
<br>
I think the patch is OK with that change. Nick, can you just comment<br>
on lib/Target/SubtargetFeature.cpp being the right place to put this?<br>
I think it is, but I am not as familiar with this code.<br></blockquote><div><br>I'm not familiar with most of lib/Target or lib/CodeGen either, but after reading through SubtargetFeature.{cpp,h} it looks like the right place to me.<br>

<br>Nick<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im"><br>
> The attached patch has similar OS and Vendor independent re-factoring for Subtarget features (attributes).<br>
><br>
> I moved Applce-specific code from tools/lto/LTOModule.cpp to lib/Target/SubtargetFeature.cpp with related header files update.<br>
><br>
> Cheers,<br>
> Viktor.<br>
><br>
<br>
</div><div><div></div><div class="h5">Cheers,<br>
<br>
--<br>
Rafael Ávila de Espíndola<br>
</div></div></blockquote></div><br>