[PATCH] D80455: [Align] Remove operations on MaybeAlign that asserted that it had a defined value.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 22 16:06:08 PDT 2020


efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

Optional has `template <typename T> bool operator<(const Optional<T> &X, const T &Y)`; are you sure that doesn't trigger for MaybeAlign/Align?

Otherwise LGTM


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80455





More information about the llvm-commits mailing list