<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jul 14, 2015 at 5:22 PM, Chih-Hung Hsieh <span dir="ltr"><<a href="mailto:chh@google.com" target="_blank">chh@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">chh added a comment.<br>
<br>
So in IR, a global variable can have string attributes like<br>
<br>
  thread_local emulated-tls<br>
  thread_local(localdynamic) emulated-tls<br>
  thread_local(initialexec) emulated-tls<br>
  thread_local(localexec) emulated-tls<br>
<br>
I can give it a try.<br>
Not sure yet if that will have shorter or longer patch.<br></blockquote><div><br></div><div>Sorry, that looks like it was bad advice. The LLVM GlobalVariable class doesn't appear to support attributes. In that case, it's probably better to make it global and let it be controlled by the TargetMachine.</div><div><br></div><div>Ultimately, I think the patches will be smaller because we shouldn't need to change IR serialization.</div></div></div></div>