[PATCH] D92514: Switch from llvm::is_trivially_copyable to std::is_trivially_copyable
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 3 10:06:16 PST 2020
MaskRay added a subscriber: mehdi_amini.
MaskRay added a comment.
In D92514#2431499 <https://reviews.llvm.org/D92514#2431499>, @dblaikie wrote:
> Generally - once code is sent for review, it should not be committed without approval: https://llvm.org/docs/CodeReview.html#code-review-workflow
For this one, it was just to have a check on premerge-checks (MSVC appears to have some edge-case issues). I think it meets likely-community-consensus requirements (previously jlebar and serge-sans-paille and probably others had similar cleanups this year, too)
Hmm. I think there was a similar discussion with @mehdi_amini :) ?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92514/new/
https://reviews.llvm.org/D92514
More information about the llvm-commits
mailing list