<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 13, 2015 at 8:42 PM, Rafael Espíndola <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">I am the one doing most of the work on ELF,</blockquote><div><br></div><div>I'm not sure how you guys have decided on divvying up the work, but as a person that works within earshot of Michael, it seems that a significant factor to that is that many of the changes you make are antagonistic to the changes he is trying to make. Deleting dead code that is used by the patch he is trying to land or</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> I am *totally* opposed to<br>
*ever* having a YAML test that can be written in assembly.<br>
<br>
If I can understand that entire YAML gunk is:<br>
<br>
foo:<br>
.section bar<br>
        call foo<br>
<br></blockquote><div><br></div><div>I like seeing bytes and symbolized ELF spec names. That is what I know best. That is what anybody working on this linker will know. I don't know all the weird corners of gas syntax (+ LLVM extensions apparently), nor would I expect any linker developer to know that. I *would* expect any linker developer to know the symbolic constants (or at least be able to easily google them).</div><div><br></div><div>The YAML file directly expresses the intent; yes there is lots of "gunk" surrounding it, but any linker developer knows how to skim that based on knowing the meaning of the symbolic constants. Do you find it hard to navigate the YAML files? We can improve the format if necessary if there is a particular pain point or two.</div><div><br></div><div>To put it another way, every "gas directives" file I see in the test directory, my immediate reaction is wanting to assemble it and readobj it to see what it "actually" is feeding to the linker. I particularly like the YAML because like readobj output, it is telling you pretty straight up what is there. One doesn't need to guess what flags the assembler set on that section, whether a particular relocation kind actually ends up in the output, etc. You just see it immediately in your editor.</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
in assembly.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
On 13 August 2015 at 23:38, Sean Silva <<a href="mailto:chisophugis@gmail.com">chisophugis@gmail.com</a>> wrote:<br>
><br>
><br>
> On Thu, Aug 13, 2015 at 6:03 PM, Rafael Espíndola<br>
> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>><br>
>> >> llvm-mc<br>
>> > This is testing explicit relocations. Assembly doesn't represent<br>
>> > specific relocations, and it's not obvious what gets generated.<br>
>><br>
>> This test is using a valid file that can be created with assembly.<br>
>> YAML is an unnecessary LLVM only invention here. Use assembly.<br>
><br>
><br>
> The YAML format may be an LLVM-only invention, but R_X86_64_PC32, SHT_RELA,<br>
> etc. are not. Anybody that understands the input format of the linker will<br>
> reasonably understand what is in the YAML file just based on those<br>
> waypoints. The same is not true for assembly (or  rather, "gas directives").<br>
><br>
> I personally have have a very hard time understanding your assembly test<br>
> cases; I think Michael does too. I think most people would have a hard time<br>
> since they don't really have any "assembly" content; they are basically a<br>
> soup of "gas directives". It's not "assembly" vs. "YAML"; it's "gas<br>
> directives (with whatever LLVM extensions)" vs "YAML". Judging by the<br>
> complete absence of assembly in test/COFF, I assume Rui also prefers<br>
> something tied directly to the object format.<br>
><br>
> @Rui, as the code owner here, do you think we should stick with YAML for the<br>
> relocation tests?<br>
><br>
> -- Sean Silva<br>
><br>
>><br>
>><br>
>> Cheers,<br>
>> Rafael<br>
><br>
><br>
</div></div></blockquote></div><br></div></div>