[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