https://bugs.llvm.org/show_bug.cgi?id=39137

Andrey Churbanov <andrey.churba...@intel.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Fixed By Commit(s)|                            |r346862
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #1 from Andrey Churbanov <andrey.churba...@intel.com> ---
The function omp_set_num_threads() fixed to not write into internal structure
if requested number of threads is the same as the value of internal control
variable (ICV).

With num_threads() clause things are more complicated, because the write here
is cheaper and this write eliminates the need to read ICV twice - here and in
the following __kmp_fork_call. Attempt to avoid write op here caused some
performance regressions in our testing, so I'd prefer to leave the clause
implementation intact.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

Reply via email to