[PATCH] D138781: [RISCV] Check missed Expected<T> result

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 1 13:15:38 PST 2022


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

Is it possible to test the actual error case too? (show that the error is propagated/reported?)

Looks good though - if you happen to be able to conjure up the failing test too, that'd be great!


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

https://reviews.llvm.org/D138781



More information about the llvm-commits mailing list