[PATCH] D32782: Add pthread_self function prototype and make it speculatable.

Joerg Sonnenberger via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 21 04:29:02 PDT 2017


joerg added a comment.

Strictly speaking, the glibc attribute violates the specification of const. I.e. we could turn a call for pthread_self() into a once() initialised static variable and that transform would be valid under the const rules. That clearly doesn't reflect the intention...

That said, I do believe it is fully reasonable to make pthread_self() speculatible by default as long as it can be turned off and is properly documented.


Repository:
  rL LLVM

https://reviews.llvm.org/D32782





More information about the llvm-commits mailing list