<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Oct 27, 2015 at 11:20 PM, Igor Kudrin <span dir="ltr"><<a href="mailto:ikudrin.dev@gmail.com" target="_blank">ikudrin.dev@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ikudrin added a subscriber: ikudrin.<br>
ikudrin added a comment.<br>
<br>
The default settings for hash section should be target-specific. At least MIPS targets don't support GNU-style hash sections at all.<br></blockquote><div><br></div><div>That's why I didn't submit this patch right away. Do MIPS compilers pass --hash-style=sysv to the linker? Also, why don't they support the GNU-style hash table?</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span><br>
<br>
================<br>
Comment at: test/elf2/shared.s:284<br>
</span>@@ -283,3 +283,1 @@<br>
<span><br>
-// CHECK:      HashTable {<br>
-// CHECK-NEXT:   Num Buckets: 4<br>
</span><span>----------------<br>
ruiu wrote:<br>
> grimar wrote:<br>
</span><span>> > Was the next part of test removed intentionaly ?<br>
> ><br>
> > // CHECK:      HashTable {<br>
> > // CHECK-NEXT:   Num Buckets: 4<br>
> > // CHECK-NEXT:   Num Chains: 4<br>
> > // CHECK-NEXT:   Buckets: [3, 0, 2, 0]<br>
> > // CHECK-NEXT:   Chains: [0, 0, 0, 1]<br>
> > // CHECK-NEXT: }<br>
> This is intentional.<br>
</span>It looks like we now have no tests which check HashTable section. I'd suggest to add -hash-style=sysv command line switch at least for this test case.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D14129" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14129</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>