[PATCH] D130645: Move mlir LogicalResult to llvm Support library
David Blaikie via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 27 12:44:19 PDT 2022
dblaikie added inline comments.
================
Comment at: llvm/include/llvm/Support/LogicalResult.h:96
/// Hide the bool conversion as it easily creates confusion.
using Optional<T>::operator bool;
using Optional<T>::has_value;
----------------
Seems unfortunate that this type (`FailureOr`) will have different API than `ErrorOr` though - any chance this could keep an explicit operator bool?
& making something private that's inherited is pretty awkward - since you can still access the private things via a base reference, or slicing, etc.
I guess alternatively, maybe `ErrorOr` ends up with `operator LogicalResult` instead of `explicit operator bool`? (@lhames )
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130645/new/
https://reviews.llvm.org/D130645
More information about the llvm-commits
mailing list