The platform-independent changes in patch set 2 look good.
I don't think anyone is planning to review the SH4-specific changes in
detail,
as AFAIK nobody on the team knows the SH4 architecture well enough to spot
any
mistakes.
The team has discussed this topic, and before we land SH4 support, it would
be
nice to understand the motivation behind the port in order for us to be
able to
decide to what extent we should support it.
Basically, there are two levels of support: there are the ports that
Google's V8
team currently fully supports and maintains (ia32, x64, ARM), and
there's "basic
support" (MIPS) where the code lives in our repository and we work with its
maintainers, e.g. by landing their patches and communicating with them about
issues or upcoming plans such as branch points or significant architectural
changes. We currently don't have any plans to extend the set of platforms
in the
former group. For the latter to work, the platform port must be in a shape
where
it fits into our infrastructure and workflow: we must at least be able to
compile it and run the test suite, which means it must have a simulator and
the
test expectations must be up to date so that the test suite passes. We're
not
willing to introduce dependencies on third-party tools for this, e.g. QEmu.
Before you reach the "basic support" stage, an option that could work quite
well
for you is to have a github clone of V8 that tracks our mirror. That way
you'll
be on top of the latest sources and can iterate on your port with minimal
latency.
Once you have all of the code in place (simulator, anything missing to make
the
majority of the tests work, expectation file for the others), we will
review the
style and general structure of the SH4 specific code. This may take a few
weeks.
To be clear, once the patch is approved, we will land it into the V8 tree,
but
it will continue to be your responsibility to make sure it keeps working and
stays up to date, and we reserve the right to remove the port at a future
date
if you are no longer able to maintain it.
https://codereview.chromium.org/11275184/
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
