I saw the problem last patch had on ia32. Without explicit call in rtl template, scheduler may schedule the sp adjusting insn across tls descriptor and break the alignment assumption. I am testing the updated patch on x86_64.
Can we combine the last two patches, both adding call explicitly in rtl template for tls_local_dynamic_base_32/tls_global_dynamic_32, and set ix86_tls_descriptor_calls_expanded_in_cfun to true only after reload complete? Regards, Wei. On Wed, Mar 12, 2014 at 5:33 PM, H.J. Lu <hjl.to...@gmail.com> wrote: > On Wed, Mar 12, 2014 at 5:28 PM, Wei Mi <w...@google.com> wrote: >>>> Does my patch fix the original problem? >>> >>> Yes, it works. I am doing bootstrap and regression test for your patch. >>> Thanks! >>> >> >> The patch passes bootstrap and regression test on x86_64-linux-gnu. >> > > My patch fails to handle ia32. Here is the updated one. > > -- > H.J.