[PATCH] D101819: [DRAFT][M68k][GloballSel] Adding initial GlobalISel infrastructure
Sushma Unnibhavi via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 4 06:10:57 PDT 2021
sushmaunnibhavi added a comment.
In D101819#2780892 <https://reviews.llvm.org/D101819#2780892>, @qcolombet wrote:
> Hi,
>
>> LLVM ERROR: unable to translate in big endian mode (in function: f)
>
> You'll need to turn that into a warning, otherwise effectively, none of your GISel changes will be testable (as in running the generated code on an actual target) until the support is complete, which is not a realistic constraint to have when bringing up a new GISel backend.
>
> Cheers,
> -Quentin
Hi @qcolombet ,
When I do a git grep with that error message, I find that it comes from IRTranslator.cpp file and in the runOnMachineFunction() function where it checks for big endian and throws a reportTranslationError() which is where that error comes from. By removing the reportTranslationError() temporarily in order to test my implementation, that would cause errors in other targets. Can you please suggest me any other way to solve this issue?
Thanks,
Sushma
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D101819/new/
https://reviews.llvm.org/D101819
More information about the llvm-commits
mailing list