[PATCH] D23178: [ADT] Add relation operators for Optional

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 11 11:19:41 PDT 2016


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

Fair enough - I felt like using the boolean literal rather than referring to the global made it a bit simpler/clearer (& consistent with the None cases). The operation is already mentioned in the template parameter. But yeah - nothing's great here, totally up to you.


https://reviews.llvm.org/D23178





More information about the llvm-commits mailing list