ENCRYPTO_utils icon indicating copy to clipboard operation
ENCRYPTO_utils copied to clipboard

Parallelization bug #23 and cmake_constant_file

Open MartKro opened this issue 5 years ago • 1 comments

  • Fixed the parallelization bug #23 which prevents possible parallelization due to the introduced mutex. Now the mutex is also protected using __thread but since it is implementation dependent the flag can be easily disabled
  • Changed the introduction of constants from CMAKE to an extra file cmake_constants.h.in

MartKro avatar Jan 14 '20 14:01 MartKro

This doesn't introduce thread local mutexes when using OpenMP, right? Could you also declare the mutex threadprivate if OpenMP is enabled? (like it's done here with the context object: https://github.com/relic-toolkit/relic/blob/c25edf6520e5e4b39663f0d0594db7186440be5c/src/relic_core.c#L95)

codedust avatar Jan 15 '20 14:01 codedust