Come to think of it, these changes aren't quite enough; in particular, they will
make it impossible to run the test suite.
tools/disasm.py, tools/ll_prof.py, tools/test.py,
tools/test-wrapper-gypbuild.py, test/{mjsunit,mozilla,test262}/*.status need
updating too.

Please make sure "make mipsel.release.check" works as expected.


http://chromiumcodereview.appspot.com/10695114/diff/1/build/common.gypi
File build/common.gypi (right):

http://chromiumcodereview.appspot.com/10695114/diff/1/build/common.gypi#newcode167
build/common.gypi:167: 'mipscompiler': '<!($(echo ${CXX:-$(which g++)})
-v 2>&1 | grep -q "^Target: mips-" && echo "yes" || echo "no")',
Is "^Target: mips-" the right pattern to match against, or does your
cross-compiler identify itself differently? "Target: mipsel-" maybe?

http://chromiumcodereview.appspot.com/10695114/diff/1/build/standalone.gypi
File build/standalone.gypi (right):

http://chromiumcodereview.appspot.com/10695114/diff/1/build/standalone.gypi#newcode49
build/standalone.gypi:49:
s/x86_64/x64/;s/amd64/x64/;s/arm.*/arm/;s/mips.*/mips/")',
missing update, the last replacement must be:
s/mips.*/mipsel/

http://chromiumcodereview.appspot.com/10695114/

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to