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

Sushma Unnibhavi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 10 00:03:46 PDT 2021


sushmaunnibhavi added a comment.

@aemerson , @qcolombet , @RKSimon can you review the patch?

I think the clang-tidy errors come because the buildbot doesn't build with DLLVM_EXPERIMENTAL_TARGETS_TO_BUILD="M68k". But when built with DLLVM_EXPERIMENTAL_TARGETS_TO_BUILD="M68k" no such errors occur.

I have created an enableBigEndian() hook in llvm/CodeGen/GlobalISel/CallLowering.h which the targets can use if they want to use big-endian. For now only M68k uses this. Then in the IRTranslator.cpp file I check if the target wants to use big-endian, if yes that the reportTranslationError(*MF, *TPC, *ORE, R) will not be called.


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

https://reviews.llvm.org/D101819



More information about the llvm-commits mailing list