Hi H.J., Could you show me why you postpone the setting ix86_tls_descriptor_calls_expanded_in_cfun until reload_complete and use ix86_tls_descriptor_calls_expanded_in_cfun instead of ix86_current_function_calls_tls_descriptor? Isn't ix86_current_function_calls_tls_descriptor useful to consider the case that tls call is optimized away?
Thanks, Wei. On Wed, Mar 12, 2014 at 2:07 PM, H.J. Lu <hjl.to...@gmail.com> wrote: > On Wed, Mar 12, 2014 at 2:03 PM, Wei Mi <w...@google.com> wrote: >>> There are several problems with this: >>> >>> 1. It doesn't work with C. >> >> Ok, I will change the testcase using C. >> >>> 2. IA32 has the same issue and isn't fixed. >> >> I thought IA32 didn't have the same issue because abi only requires 32 >> bit alignment for stack starting address. >> >> oh, I found the old patch >> http://gcc.gnu.org/ml/gcc-patches/2006-09/msg00298.html which changed >> the default alignment to 128bit. Ok, will remove the TARGET_64BIT >> constraint. >> >>> 3. There is no testcase for global dynamic model. >>> >>> -- >>> H.J. >> >> Will add the testcase. >> > > I posted a different patch in > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58066 > > -- > H.J.
