[PATCH] D57097: Make llvm::Optional<T> trivially copyable when T is trivially copyable

[serge via Phabricator via llvm-commits]

serge-sans-paille added a comment.

@chandlerc there's indeed some redundancy that can be removed unfortunately, as you feared, defaulting seems incompatible with per-member specialization, or at least it hits my c++ knowledge limit.

Here is my attempt: https://godbolt.org/z/9_Lykd
I have an alternate version using inheritance under testing, I'll upload the patch during the day.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57097