[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