[PATCH] D86354: [ADT] Make Optional a literal type.

[John McCall via Phabricator via llvm-commits]

rjmccall accepted this revision.
rjmccall added a comment.

This seems to only allow empty optionals to be `constexpr`.  Should the non-default constructor also be `constexpr`?  (Note that it's okay for a templated `constexpr` function to instantiate to something that does stuff that's not `constexpr`, so this wouldn't be a major new restriction on the constructor.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86354