Page MenuHomePhabricator

compiler-rt/builtins: Make check-builtins run tests on macOS.
ClosedPublic

Authored by thakis on Thu, Aug 29, 5:19 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

thakis created this revision.Thu, Aug 29, 5:19 PM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Aug 29, 5:19 PM

delcypher, kubamracek : ping.

delcypher, kubamracek: Would you prefer if someone else looked at this?

@thakis This looks like a reasonable start but I'd like to test this before landing it. @arphaman might have opinions too.

The approach seems reasonable to me

What's the next step here?

I think @delcypher wanted to test it first

smaller diff

@thakis Sorry for the delay on this. I should have some time this afternoon to try this out.

@thakis I gave this patch a try. It seems to work apart from the i386 tests that will not work on macOS Catalina. We can deal with that later though.

I've never looked at the builtins before. There are some very weird things about the tests here that don't make any sense to me but that's not part of this patch so I won't dwell on them here.

delcypher accepted this revision.Fri, Sep 13, 5:29 PM
This revision is now accepted and ready to land.Fri, Sep 13, 5:29 PM
This revision was automatically updated to reflect the committed changes.