[PATCH] D117254: [ADT] Fix Optional<> with llvm::is_trivially_move_constructible

[James Player via Phabricator via llvm-commits]

jplayer-nv added inline comments.


================
Comment at: llvm/include/llvm/ADT/Optional.h:57
                                !std::is_move_constructible<T>::value) &&
                               (std::is_trivially_move_assignable<T>::value ||
                                !std::is_move_assignable<T>::value))>
----------------
I don't have a test in mind yet, but any concern for the same instantiation bug with `std::is_trivially_move_assignable` pre-gcc7.4?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117254