[Openmp-commits] [PATCH] D94361: [OpenMP] Remove copy constructor of `RTLInfoTy`

Shilei Tian via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Sat Jan 9 09:18:27 PST 2021


tianshilei1992 added a comment.

In D94361#2488663 <https://reviews.llvm.org/D94361#2488663>, @tianshilei1992 wrote:

> `std::mutex` is non-copyable and non-movable. Any struct containing it will be marked non-copyable and non-movable implicitly. Actually I'm not sure how compiler handles this struct because if I explicitly add `RTLInfoTy(RTLInfoTy &&) = default` and `RTLInfoTy(const RTLInfoTy &) = delete`, the compiler complains "no matching constructor for initialization of 'RTLInfoTy'". However, everything seems all good if I don't do anything...

Well, stupid me. `AllRTLs` is a `std::list`...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94361



More information about the Openmp-commits mailing list