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

[Quentin Colombet via Phabricator via llvm-commits]

qcolombet added a comment.

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



================
Comment at: llvm/lib/CodeGen/GlobalISel/IRTranslator.cpp:3131
+    // TODO : removing reportTranslationError until big endian is supported
+    // reportTranslationError(*MF, *TPC, *ORE, R);
   }
----------------
Instead of purely removing it, turn that into a warning.


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

https://reviews.llvm.org/D101819