[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.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits