Closed Bug 1590101 Opened 5 years ago Closed 5 years ago

Bump Cranelift to da179e4fd83d49b7ad6c9f286b1ea04d4f64907e

Categories

(Core :: JavaScript: WebAssembly, task, P3)

task

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: bbouvier, Assigned: bbouvier)

References

Details

Attachments

(3 files)

No description provided.

The function used to translate a function how requires a
ModuleTranslationState, which contains state collected after reading early
sections of a wasm module. Currently, this is only used in Cranelift for the
multi-value feature (to read block type signatures), so we can use a dummy
state that's shared across all functions.

Gary, Christian: how big an issue it is for you when updating to a specific changeset results in not being able to build? I imagine your bisection script would use the skip option of git bisect, but just wanted to double-check this.

Context of this question: in a bug such as this one, it seems preferable to be able to look at what actual changes happened, separately of the output of the mach vendor rust command, which adds a lot of noise. So we'd prefer to be able to look in retrospect at commits one by one, instead of squashing them, but this will create small commit ranges in which it's impossible to build.

Flags: needinfo?(nth10sd)
Flags: needinfo?(choller)

I think we are entirely moving away from compiled bisections and want to use TC builds instead. For that purpose, having small ranges in history that don't build seem fine to me.

Flags: needinfo?(choller)

In any case, the current bisection scripts work on Mercurial but not Git for now. Historically they have been able to skip "impossible-to-build" changesets, up till a certain arbitrary number.

Flags: needinfo?(nth10sd)
Priority: -- → P3

Good to know, thanks! It's ideal because it means we can land as is, and be able to look at the history of commits and see actual changes. I assume the arbitrary number of skip revisions is low, but greater than or equal to 2, which is our case here.

Pushed by bbouvier@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/307a54c830da Bump Cranelift to da179e4fd83d49b7ad6c9f286b1ea04d4f64907e; r=jseward https://hg.mozilla.org/integration/autoland/rev/b321de6e4733 Result of mach vendor rust; r=jseward https://hg.mozilla.org/integration/autoland/rev/f1000962724e Update Baldrdash to latest Cranelift API; r=jseward
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
No longer blocks: cranelift
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: