<div dir="ltr">lit changes look good, a couple of nits:<div><br></div><div><div>+def have_ld_plugin_support():</div><div>+    if not os.path.exists(config.llvm_shlib_dir + '/LLVMgold.so'):</div><div>+        return False</div>
<div>+</div></div><div><br></div><div>you can os.path.join(config.llvm_shlib_dir, LLVMgold.so) instead.</div><div><div>+if not 'ld_plugin' in  config.available_features or \</div><div>+   not 'X86' in config.root.targets:</div>
<div>+   config.unsupported = True</div></div><div><br></div><div>I'd prefer parens instead of backslash.</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jul 25, 2014 at 5:43 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">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">Also added a test for the recent -mtattr fix.<br>
<br>
Adding Alexey Samsonov for the lit bits.<br>
<div class="HOEnZb"><div class="h5"><br>
On 23 July 2014 18:24, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
> And the attached patch also works when running on ppc.<br>
><br>
> On 23 July 2014 17:41, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>> Hi Nick,<br>
>><br>
>> I cannot remember exactly why we never added direct tests for the gold<br>
>> plugin. It might be that it was uncommon to have a ld with plugin<br>
>> support or maybe it was just that our test infrastructure was not able<br>
>> to handle it.<br>
>><br>
>> In any case, it seems that it is now fairly easy to test the plugin<br>
>> when the tests are run in a machine with a ld that supports plugins.<br>
>> The attached patch adds a simple test for a previous fix.<br>
>><br>
>> I have tested that it works with both gold (passes) and bfd ld<br>
>> (unsupported) on linux.<br>
>><br>
>> I am currently building on a non-x86 machine to test that.<br>
>><br>
>> Cheers,<br>
>> Rafael<br>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Alexey Samsonov<br><a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a></div>
</div>