<div dir="ltr">Hi Charlie,<div><br></div><div>What Eric said makes sense.</div><div><br></div><div><span style="font-family:arial,sans-serif;font-size:13px">> if not 'hexagon' in config.target_triple:</span><br style="font-family:arial,sans-serif;font-size:13px"><span style="font-family:arial,sans-serif;font-size:13px">>    config.available_features.add(</span><span style="font-family:arial,sans-serif;font-size:13px">"object-emission")</span><br></div><div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">So the tests when running on hexagon will not have the object-emission feature. So, you can add</span></div><div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">> </span><span style="color:rgb(80,0,80);font-family:arial,sans-serif;font-size:13px">REQUIRES: object-emission</span></div><div><span style="color:rgb(80,0,80);font-family:arial,sans-serif;font-size:13px"><br></span></div><div><font color="#500050" face="arial, sans-serif">to the test case, under the RUN: line, to make it not run for hexagon.</font></div><div><font color="#500050" face="arial, sans-serif"><br></font></div><div><font color="#500050" face="arial, sans-serif">What am I missing?</font></div><div><font color="#500050" face="arial, sans-serif"><br></font></div><div><font color="#500050" face="arial, sans-serif">Cheers,</font></div><div><font color="#500050" face="arial, sans-serif"><br></font></div><div><font color="#500050" face="arial, sans-serif">James</font></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 1 October 2014 10:28, Charlie Turner <span dir="ltr"><<a href="mailto:Charlie.Turner@arm.com" target="_blank">Charlie.Turner@arm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Eric,<br>
<span class=""><br>
>> I now realize why hexagon was omitted from the original chain of if/else comparisons. I thought it was probably an oversight, but it doesn't make sense to add a test for this target, since there's no object emission. I have updated my refactoring to not handle the hexagon case, since it's not handled in the original code.<br>
>><br>
><br>
> You'll want:<br>
><br>
> REQUIRES: object-emission<br>
><br>
> for the test cases.<br>
<br>
</span>That's the default for all targets that are not Hexagon, so I don't think I'll need that in the arch-specific subdirs. From lit.cfg,<br>
<br>
if not 'hexagon' in config.target_triple:<br>
    config.available_features.add("object-emission")<br>
<br>
Are you suggesting I create a new Hexagon subdirectory in DebugInfo/ with just this one relocation test, which will be an XFAIL? IIUC, only tests in the top-level DebugInfo directory need REQUIRES: object-emission. I expected to not need a test case for Hexagon since the RelocVisitor doesn't know about Hexagon.<br>
<span class=""><br>
> That failed because you removed the -mtriple argument to llc (the -linux part forces ELF IIUC), which caused llc to default to COFF emission.<br>
><br>
> COFF should be ok. You should be able to use dwarf with cygwin/mingw.<br>
<br>
</span>I386 defaults to COFF, and RelocVisitor does not support COFF files, only ELF. These tests rely on the emitted object file being ELF, which is why I needed the -linux part of the triple.<br>
<span class="im HOEnZb"><br>
Thanks,<br>
Charlie.<br>
<br>
-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.<br>
<br>
ARM Limited, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No:  2557590<br>
ARM Holdings plc, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No:  2548782<br>
<br>
</span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>