On Thu, Sep 03, 2015 at 01:09:23PM +0200, Sebastian Huber wrote: > >>>We have only thread attributes in this function: mutable_attr and attr. The > >>>attr is initialized with &gomp_thread_attr and gomp_thread_attr is supposed > >>>to be read-only by this function. Under certain conditions we have to > >>>modify > >>>the initial attributes. Since gomp_thread_attr is read-only, we have to > >>>copy > >>>it and then modify the copy. For this we need some storage: mutable_attr. > >So use local_thread_attr if you want to stress it, but IMHO thread_attr > >just just fine. I really don't like mutable_attr. > > Ok, if I don't rename thread_attr, is the patch ok?
Yes. Jakub