<div dir="ltr">I'll go ahead and add a check for e_machine==EM_HEXAGON then. Now that Rafael and Roland point it out, that should be a check. Hopefully this doesn't get too hairy to support all these architectures.<br><br><div class="gmail_quote"><div dir="ltr">On Mon, Sep 11, 2017 at 4:45 PM Roland McGrath <<a href="mailto:mcgrathr@google.com" target="_blank">mcgrathr@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I don't personally care about any targets that use machine-specific or<br>
OS-specific st_shndx values, so just SHN_ABS and SHN_COMMON is fine.<br>
<br>
It's certainly the case that SHN_HEXAGON_SCOMMON* can be safely treated the<br>
same as SHN_COMMON.  But to be pedantic, you need to check that<br>
e_machine==EM_HEXAGON before assuming those [SHN_LOPROC,SHN_HIPROC] values<br>
have that particular meaning.<br>
</blockquote></div></div>