Justin Lebar (not reading bugmail) Reporter
	Description • 13 years ago

We run v8 on mozilla-inbound, but there's no way to force it on try.

This means that if I regress v8 on inbound (as I did in bug 729940), there's no way for me to test whether I fixed it without pushing to m-i.  (In bug 729940, I also regressed Dromaeo jslib, but only on Windows, while the v8 regression was on Linux as well, so finding the regression using Dromaeo will be much slower than using v8.)

If there's a good reason not to run v8 on try, I wonder why we're running it on m-i?

	Lukas Blakk [:lsblakk] use ?needinfo
	Comment 1 • 13 years ago

Why are we running this on m-i and not m-c?

	Phil Ringnalda (:philor)
	Comment 2 • 13 years ago

Because we ran it only on the TraceMonkey tree, because js hackers wanted to know the effect of their patches on it but didn't feel the need to have it run against every single non-js/src/ push, and then mozilla-inbound became the new TraceMonkey.

	Justin Lebar (not reading bugmail) Reporter
	Comment 3 • 13 years ago

(In reply to Phil Ringnalda (:philor) from comment #2)
> Because we ran it only on the TraceMonkey tree, because js hackers wanted to
> know the effect of their patches on it but didn't feel the need to have it
> run against every single non-js/src/ push, and then mozilla-inbound became
> the new TraceMonkey.

And the missing piece is that JS hackers used TM like try (was my understanding).  But we don't use m-i like try.

	Armen [:armenzg]
	Comment 4 • 12 years ago

Do we want this enabled everywhere? or just m-i and try?

	Justin Lebar (not reading bugmail) Reporter
	Comment 5 • 12 years ago

> But we don't use m-i like try.

Or at least, that's not sanctioned... :)

(In reply to Armen Zambrano G. [:armenzg] - Release Engineer from comment #4)
> Do we want this enabled everywhere? or just m-i and try?

I kind of feel like we should run it everywhere, if we think it's a good benchmark, or nowhere, if we don't.  But so long as it's on try, I'm happy.

	Joel Maher ( :jmaher ) (UTC -8)
	Comment 6 • 12 years ago

currently now we have v8 (version 7) in the dromaeojs suite.  This just landed today and it runs on all platforms.  I am looking to disable the old v8 for trunk based branches in bug 781290.

	Phil Ringnalda (:philor)
	Comment 7 • 11 years ago

Fixed by bug 781290, long ago.