LGTM with comments.
http://codereview.chromium.org/7740020/diff/1/build/standalone.gypi File build/standalone.gypi (right): http://codereview.chromium.org/7740020/diff/1/build/standalone.gypi#newcode58 build/standalone.gypi:58: 'v8_can_use_vfp_instructions%': 'true', You mentioned something about this... http://codereview.chromium.org/7740020/diff/1/build/standalone.gypi#newcode82 build/standalone.gypi:82: #'ldflags': [ '-pthread', ], Remove this line. http://codereview.chromium.org/7740020/diff/1/samples/samples.gyp File samples/samples.gyp (right): http://codereview.chromium.org/7740020/diff/1/samples/samples.gyp#newcode43 samples/samples.gyp:43: ], Can this be moved into common.gypi? For example under target_defaults. As far as I can see this is because v8::internal::Thread is implemented with pthread on Linux/Mac, and is part of v8 itself, not limited to d8 or the samples or cctests. http://codereview.chromium.org/7740020/diff/1/src/d8.gyp File src/d8.gyp (right): http://codereview.chromium.org/7740020/diff/1/src/d8.gyp#newcode80 src/d8.gyp:80: ], Ditto. http://codereview.chromium.org/7740020/diff/1/test/cctest/cctest.gyp File test/cctest/cctest.gyp (right): http://codereview.chromium.org/7740020/diff/1/test/cctest/cctest.gyp#newcode143 test/cctest/cctest.gyp:143: ], Ditto. http://codereview.chromium.org/7740020/diff/1/tools/gyp/v8.gyp File tools/gyp/v8.gyp (right): http://codereview.chromium.org/7740020/diff/1/tools/gyp/v8.gyp#newcode760 tools/gyp/v8.gyp:760: ], Ditto. http://codereview.chromium.org/7740020/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
