[PATCH] D115853: [RISCV][ABI] Have ABI checks for ilp32e under AsmParser and TargetLowering
Luís Marques via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 16 06:19:09 PST 2021
luismarques added a comment.
In D115853#3197551 <https://reviews.llvm.org/D115853#3197551>, @luismarques wrote:
> The new error message in RISCVISelLowering.cpp isn't yet covered by tests but that line currently isn't reachable due to the `report_fatal_error` in line 52. It's not ideal that this can't have test coverage. @asb thoughts?
I was assuming that it would be a bad idea to remove the `report_fatal_error` because code generation would still occur (despite the error message) and possibly crash in unpleasant ways but maybe that's not significantly different from the other error conditions above where that also happens.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115853/new/
https://reviews.llvm.org/D115853
More information about the llvm-commits
mailing list