[Openmp-commits] [PATCH] D89898: changing OMP rtl to use shared memory instead of env variable

[Roger Ferrer Ibanez via Phabricator via Openmp-commits]

rogfer01 added a comment.

Hi, one question about this change:

We're seeing that sometimes the files in `/dev/shm` are not always removed, is this expected?

Leaving files is not a problem if the same user is running the application because the second `shm_open` in line 6416 of `kmp_runtime.cpp` above will be able to open the file. However for a user (other than the creator of the shm file) this returns `EACCES` and then the runtime refuses to start with a very confusing error.

This may not be easily observable in CI systems because they often use the same user (in the same nodes).

Kind regards,


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89898/new/

https://reviews.llvm.org/D89898