[PATCH] D101819: [DRAFT][M68k][GloballSel] Adding initial GlobalISel infrastructure

[Sushma Unnibhavi via Phabricator via llvm-commits]

sushmaunnibhavi added a comment.

In D101819#2799694 <https://reviews.llvm.org/D101819#2799694>, @qcolombet wrote:

> Hi,
>
>> 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?
>
> That kind of errors are you seeing in other targets?
>
> Also like Amara said, you can turn that into a warning on for M68k with a target hook.
>
> Cheers,
> -Quentin

Yes I see those errors only in other targets after removing reportTranslationError(). The M68k test case doesn't fail.
Yes, I will do as Amara said.


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

https://reviews.llvm.org/D101819