[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