This is an archive of the discontinued LLVM Phabricator instance.

[m68k] Change compiler-rt settings
ClosedPublic

Authored by DavidSpickett on Mar 21 2023, 8:23 AM.

Details

Summary

For clang-m68k-linux-cross it appears that compiler-rt is
not needed. It does not build it for m68k and there are other
bots covering compiler-rt on x86 (which hosts the build).

I was unable to confirm the configuration of clang-m68k-linux
(https://lab.llvm.org/buildbot/#/builders/159 doesn't show any builds).
And cannot search for it on staging at the moment.

So I am opting it out of the runtimes build for compiler-rt
just to be on the safe side.

Diff Detail

Event Timeline

DavidSpickett created this revision.Mar 21 2023, 8:23 AM
Herald added a project: Restricted Project. · View Herald Transcript
DavidSpickett requested review of this revision.Mar 21 2023, 8:23 AM
myhsu accepted this revision.Mar 22 2023, 9:39 AM

Thanks for spotting this!
I agree that we should disable compiler-rt build for now

This revision is now accepted and ready to land.Mar 22 2023, 9:39 AM
glaubitz accepted this revision.Mar 22 2023, 9:48 AM

Fine with me.

This revision was automatically updated to reflect the committed changes.