<div dir="ltr">(+Adrian and Paul in case they have any particular thoughts/context here)<br><br>Honestly I have no state here, two years later. Looking at the code, seems DwarfCompileUnit::addExpr is only used for TLS, but DwarfCompileUnit::addAddressExpr (the other use of DIEExpr) is used for PCOffset, which is created in DebugHandlerBase::getFunctionLocalOffsetAfterInsn - which doesn't have anything to do with thread locals. So, sure, guess it could be renamed back?<br><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 21, 2018 at 1:55 PM Simon Atanasyan <<a href="mailto:simon@atanasyan.com">simon@atanasyan.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi David,<br>
<br>
Commit message for the r294500 states that "As pointed out by David<br>
Blaikie in the post commit review of r292624, EmitDebugValue should be<br>
called EmitDebugThreadLocal".<br>
<br>
After that commit the `DIEExpr::EmitValue` always calls the<br>
`EmitDebugThreadLocal` for all architectures. Is it true that _any_<br>
DIE expression is related to thread local storage on _all_<br>
architectures?<br>
<br>
Regards,<br>
Simon<br>
</blockquote></div></div>