[PATCH] D130645: Move mlir LogicalResult to llvm Support library

[Jakub Kuderski via Phabricator via llvm-commits]

kuhar added a comment.

+1, I like this direction. `bool` return values confused me numerous times in functions like `check*`, `verify*`, etc., and many places in LLVM would benefit from this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130645