<div dir="ltr">Hi Tim,<br><div class="gmail_extra"><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
In practice this means that existing targets put their TLS operand variants<br>
into the global namespace (VK_ARM_TLSGD, VK_Mips_DTPREL_HI, ...). However, the<br>
AArch64 target has more of these modifiers than all other targets combined and<br>
I felt it was inappropriate to pollute the global space for them.<br>
<br>
This patch adds a virtual callback to MCTargetExpr so that analysis doesn't<br>
stop at this barrier and the work can be delegated to the target.<br>
<br>
I believe this can be improved further; the implementatoin of<br>
MCTargetExpr::fixELFSymbolsInTLSFixups involves more duplication than I'd<br>
like. However, I would prefer to delay this further refactoring until AArch64<br>
is committed since it would provide an example use to reason from more easily.<br>
<br>
Can I commit it?<br></blockquote><div><br></div><div style>No objection per se, but it'd be nice to see the cleanups in at least one target as the</div><div style>motivation for the change?</div><div style><br></div>
<div style>-eric </div></div></div></div>