RE: [PATCH 2/2] [ARC] Add TLS support.

2016-04-28 Thread Claudiu Zissulescu
Fixing the loose ends. Committed r235559 Thanks, Claudiu > > > +(define_insn "tls_gd_load" > .. > > + ; if the linker has to patch this into IE, we need a long insns > > Typo: a long insn. > > arc_emit_call_tls_get_addr is missing a start-of-function comment. > > Otherwise this is OK.

Re: [PATCH 2/2] [ARC] Add TLS support.

2016-04-28 Thread Joern Wolfgang Rennecke
On 15/04/16 10:58, Claudiu Zissulescu wrote: TLS mods for ARC backend. OK to apply? Claudiu This is using an inefficient TLS global dynamic implementation that would not be expected in a new and/or well-tuned port. However, if you have to work with a legacy runtime, that can't be helped.