qchateau accepted this revision. qchateau added a comment. This revision is now accepted and ready to land.
LGTM Small nits: - `operator()` is not `const` but `Next` is `mutable`, seems to me you intended to have `operator()` as `const` - memory orders for the atomic operations can probably be downgraded to `std::memory_order_acquire`/`std::memory_order_acq_rel`. I think the first load can even be `relaxed` but that I'm always careful with these Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93726/new/ https://reviews.llvm.org/D93726 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
