[PATCH] D130689: [LLVM] Update C++ standard to 17

Tobias Hieta via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 1 01:11:08 PDT 2022


thieta added a comment.

In D130689#3690274 <https://reviews.llvm.org/D130689#3690274>, @nikic wrote:

> Given https://github.com/llvm/llvm-project/blob/2bb7c54621f31a957302a4deb3d25b752acb07bd/llvm/include/llvm/Support/RWMutex.h#L22-L27, it seems like this is supposed to be supported. This is probably just a matter of moving the shared_mutex use behind the LLVM_USE_RW_MUTEX_IMPL guard?

Yeah - I just realized that when I checked this. Just building the rest of the tree now to confirm this is the only change we need and I will publish this a different diff first.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130689



More information about the cfe-commits mailing list