Page MenuHomePhabricator

[archie-client][mips] Define NOREGEX macro since MUSL doesn't provide re_{comp,exec}().

Authored by vkalintiris on Oct 21 2015, 6:21 AM.



Diff Detail

Event Timeline

vkalintiris retitled this revision from to [archie-client][mips] Define NOREGEX macro since MUSL doesn't provide re_{comp,exec}()..
vkalintiris updated this object.
vkalintiris added a reviewer: dsanders.
vkalintiris added a subscriber: llvm-commits.
dsanders added inline comments.Oct 23 2015, 2:03 AM

Shouldn't this disable the test if re_comp is unavailable? It should be defined/not-defined by the configure step.

vkalintiris added inline comments.Nov 11 2015, 2:14 AM

We can't rely on the configuration script for this at the moment (see

Even if the configure step was reliable, disabling the whole test, seems like an overkill to me. Given the transition to CMake, I suggest that we commit this and properly configure the test-suite for cross-compilation with CMake in the future.

dsanders added inline comments.Nov 11 2015, 2:23 AM

I'm not keen to change this test in this way for all Mips targets because of a MUSL specific problem. I don't mind a temporary solution but I'd like it to target MUSL specifically.

FWIW, I tested our out-of-tree buildbots and they are passing the test without any problem. This should be a temporary solution as long as we can't use rununder with the CMake build. With CMake it's really easy to check for the existence of these functions during configuration time.

vkalintiris abandoned this revision.Jun 5 2016, 10:39 AM