[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