<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Feb 5, 2018 at 8:01 PM, Rafael Avila de Espindola <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"><span class="">Rafael Avila de Espindola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> writes:<br>
</span><span class="">> From llvm's point of view I would say that it is all missing features in<br>
> grubs partial implementation of the ABI.<br>
><br>
> I am happy helping implement the missing bits in grub, but I don't think<br>
> we should avoid this change in llvm.<br>
<br>
</span>The attached file should implement the missing relocation on i386. How<br>
do I test it?<br></blockquote><div><br></div><div>Thanks for the quickly-implemented fix!  Honestly, I'm not sure how to fully test that this boots, but one of our grub developers has offered to test things for me if needed, so I'll give this a quick try tomorrow morning to make sure it builds in our build system, and then hand it off to him for testing.  Hopefully we can let you know tomorrow afternoon if this works and get you unblocked on re-committing the patch.</div><div><br></div><div>If this is indeed the extent of the change required for grub, I agree with your analysis that this shouldn't be a blocker.</div><div><br></div><div>- Brooks</div><div><br></div><div><br></div></div></div></div>