Closed Bug 1275508 Opened 5 years ago Closed 11 months ago

Remove werror

Categories

(Core :: JavaScript Engine, task)

task
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla76
Tracking Status
firefox76 --- fixed

People

(Reporter: jandem, Assigned: evilpie)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

No description provided.
See Also: → 1292137
Not working on this atm.
Assignee: jdemooij → nobody
Status: ASSIGNED → NEW
Blocks: 1620583

This passes jit-test and jstest locally. I still see some references
in jsapi-tests/binast, but I haven't looked into running those yet.

The asm.js warning stuff is a bit confusing. It seems like there are multiple mistakes?

asmLink claims that it should issue a warning. But it sets "werror", so this would always
throw an exception if linking actually threw an exception.

Some tests using assertAsmDirectiveFail aren't even valid JS programs ..

Assignee: nobody → evilpies
Status: NEW → ASSIGNED
Type: defect → task
See Also: → 1621603
Pushed by evilpies@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/6dd02c648edb
Remove werror. r=jandem
https://hg.mozilla.org/integration/autoland/rev/f34c6e1c6fb5
Remove JavaScript werror from browser. r=mccr8
Status: ASSIGNED → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla76
You need to log in before you can comment on or make changes to this bug.