<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 16, 2016 at 9:08 AM, George Rimar via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">




<div dir="ltr" style="font-size:12pt;color:rgb(0,0,0);font-family:Calibri,Arial,Helvetica,sans-serif;background-color:rgb(255,255,255)"><span class="">
<p>> <span style="color:rgb(33,33,33);font-size:12pt">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 </span><span style="color:rgb(33,33,33);font-size:12pt">doing,
 you may be changing the test in such a way that it would make no longer sense.</span></p>
<p><span style="color:rgb(33,33,33);font-size:12pt"><br>
</span></p>
</span><p><span style="color:rgb(33,33,33);font-size:12pt">I modified the test in part of my needs. That change seemed and seems obvious for me. I agree with your position though, let return back to this talk tomorrow after I`ll investigate this testcase more
 deeply.</span></p></div></blockquote><div><br></div><div>I wrote r259831 and it is simply an error in reducing the test case that I ended up using -shared. It avoids needing another file. If you can regenerate the .o file using the command line in the test using llvm33, that will be sufficient (adding appropriate other flags for -relocation-model). btw, the way I created this test was to have:</div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">// RUN: /path/to/llvm33/bin/llvm-mc -filetype=obj -triple=x86_64-unknown-linux %s -o %S/Inputs/llvm33-rela-outside-</span><span style="font-size:12.8px">group.o</span><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Only at the end when going to submit the test did I replace `</span><span style="font-size:12.8px">RUN: /path/to/llvm33/bin/llvm-mc` with `</span><span style="font-size:12.8px">llvm33/llvm-mc`.</span></div><div><br></div><div>Sorry for creating all this complexity for you! If you want, I can regenerate it when I'm back at my desk next week.</div><div><br></div><div>Another possibility is to change the test to not use -shared, but that involves adding a new .s file for _start and IMO is less preferred since it fails to emphasize the thing that is being fixed.</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr" style="font-size:12pt;color:rgb(0,0,0);font-family:Calibri,Arial,Helvetica,sans-serif;background-color:rgb(255,255,255)"><span class=""><font color="#888888">
<p><span style="color:rgb(33,33,33);font-size:12pt"><br>
</span></p>
<p><span style="color:rgb(33,33,33);font-size:12pt">George.</span></p>
</font></span></div>

<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div></div>