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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to