<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Mar 16, 2016 at 8:32 AM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">grimar added a comment.<br>
<span class=""><br>
In <a href="http://reviews.llvm.org/D18190#376427" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18190#376427</a>, @ruiu wrote:<br>
<br>
> Ah, gotcha. So this change would break the existing test for r259831 because it would now fail because of "recompile with -fPIC" error, so you had to update it.<br>
><br>
> But you don't have to update the binary. Remove "-shared" from llvm33-rela-outside-group.s.<br>
<br>
<br>
</span>I was not familar with that test and commit. If you think we can just remove -shared, lets do that separately, before checking in this one ?<br></blockquote><div><br></div><div>If you are not familiar with that test, then please take time to understand it. You were about to modify the input to that particular test. If you don't understand what you are doing, you may be changing the test in such a way that it would make no longer sense.</div></div></div></div>