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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 10 01:52:29 PDT 2021


RKSimon added a comment.

In D101819#2809759 <https://reviews.llvm.org/D101819#2809759>, @sushmaunnibhavi wrote:

> I think the clang-tidy errors in M68kRegisterBankInfo.h 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.

That's exactly the issue - I don't think there's a way to force the buildbot to run for a particular LLVM_EXPERIMENTAL_TARGETS_TO_BUILD. This works in the other way too - changes to common code don't get tested on experimental targets. In both cases its up to the target maintainers to handle any regressions.


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

https://reviews.llvm.org/D101819



More information about the llvm-commits mailing list