[PATCH] D127593: [clang] Fix trivially copyable for copy constructor and copy assignment operator

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 13 13:18:07 PDT 2022


erichkeane added a comment.

In D127593#3579518 <https://reviews.llvm.org/D127593#3579518>, @royjacobson wrote:

> I don't think the OpenMP tests are failing because of this PR, they are a bit flaky sometimes.
>
> @erichkeane - Since we haven't branched clang15 yet, I think it's Ok not to add the Ver15 value to the ABI options?

Ah, I think the fact that he's doing <= Ver14 here makes it fine to treat this as a 'latest' feature.  I think the 'Latest' addition like to allow this is actually reasonably new (or my memory of how these happen is defective), so i think you're right we don't need to change it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127593



More information about the cfe-commits mailing list