Bug 194079
| Summary: | Remove optLevel from B3 | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Robin Morisset <rmorisset> |
| Component: | JavaScriptCore | Assignee: | Robin Morisset <rmorisset> |
| Status: | NEW | ||
| Severity: | Normal | CC: | saam |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | 191802 | ||
| Bug Blocks: | |||
Robin Morisset
Now that the BBQ tier for WASM goes directly to Air, there is no more need for different optimization tiers inside B3. Both FTL and Warm OMG use B3 -O2.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Saam Barati
(In reply to Robin Morisset from comment #0)
> Now that the BBQ tier for WASM goes directly to Air, there is no more need
> for different optimization tiers inside B3. Both FTL and Warm OMG use B3 -O2.
I think it might be worth keeping this running just in case we find a use for it in the future.
Robin Morisset
(In reply to Saam Barati from comment #1)
> (In reply to Robin Morisset from comment #0)
> > Now that the BBQ tier for WASM goes directly to Air, there is no more need
> > for different optimization tiers inside B3. Both FTL and Warm OMG use B3 -O2.
>
> I think it might be worth keeping this running just in case we find a use
> for it in the future.
I am a bit wary of keeping dead code in the codebase, especially as it will no longer be tested going forward (unless we run extra copies of our tests with different optimization levels, but that sounds like it would heavily increase our already massive testing time).
If we find a use for it later, would it be that much of a burden to reinstate it?
Saam Barati
(In reply to Robin Morisset from comment #2)
> (In reply to Saam Barati from comment #1)
> > (In reply to Robin Morisset from comment #0)
> > > Now that the BBQ tier for WASM goes directly to Air, there is no more need
> > > for different optimization tiers inside B3. Both FTL and Warm OMG use B3 -O2.
> >
> > I think it might be worth keeping this running just in case we find a use
> > for it in the future.
>
> I am a bit wary of keeping dead code in the codebase, especially as it will
> no longer be tested going forward (unless we run extra copies of our tests
> with different optimization levels, but that sounds like it would heavily
> increase our already massive testing time).
We already do this
>
> If we find a use for it later, would it be that much of a burden to
> reinstate it?