<p dir="ltr"><br>
On May 27, 2016 12:29 PM, "Simon Atanasyan" <<a href="mailto:simon@atanasyan.com">simon@atanasyan.com</a>> wrote:<br>
><br>
> On Fri, May 27, 2016 at 7:22 PM, Rafael Espíndola<br>
> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
> ><br>
> > On May 27, 2016 12:18 PM, "Simon Atanasyan" <<a href="mailto:simon@atanasyan.com">simon@atanasyan.com</a>> wrote:<br>
> >><br>
> >> R_MIPS_GPREL32 is for local symbols. But GNU linker does not show any<br>
> >> error if R_MIPS_GPREL32's target is preemptible symbol. So I still think it<br>
> >> is better to handle them uniformly too.<br>
> ><br>
> > If it doesn't show up in the wild, please don't check for it.<br>
><br>
> Do you suggest to check for relocation type R_MIPS_GPREL16 not for<br>
> relocation expression R_GOTREL in the new isPreemptible routine?</p>
<p dir="ltr">Correct. Also, we should make sure at least llvm creates a local symbol for use with R_MIPS_GPREL16.<br></p>
<p dir="ltr">Cheers,<br>
Rafael<br></p>
<p dir="ltr">> --<br>
> Simon Atanasyan<br>
</p>