[PATCH] D92662: [Clang][Coroutine] Drop const attribute on pthread_self when coroutine is enabled

[James Y Knight via Phabricator via cfe-commits]

jyknight added a comment.

I don't think we should change the meaning of `__attribute__((const))` to exclude depending on thread-id.

However, if we do want to do so, and call the existing uses of `__attribute__((const))` in glibc invalid, we need to special case many more functions. Looking through it a little, I see `__errno_location`, `__rpc_thread_variables`, `__ctype_b_loc`, `__ctype_tolower_loc`, `__ctype_toupper_loc`, `__libc_tsd_address`...and I gave up looking after that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92662