Closed
Bug 1201460
Opened 9 years ago
Closed 9 years ago
Disallow asm.js compilation for classes/methods
Categories
(Core :: JavaScript Engine: JIT, defect)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
mozilla44
People
(Reporter: anba, Assigned: bbouvier)
References
Details
Attachments
(1 file)
3.91 KB,
patch
|
luke
:
review+
|
Details | Diff | Splinter Review |
Test case 1:
---
var c = class { constructor(stdlib, foreign, heap) { "use asm"; return {}; } };
c();
---
Expected: Throws TypeError
Actual: No TypeError
Test case 2:
---
var obj = ({m(stdlib, foreign, heap) { "use asm"; return {}; }});
new obj.m;
---
Expected: Throws TypeError
Actual: No TypeError
Reporter | ||
Updated•9 years ago
|
Blocks: harmony-classes
Assignee | ||
Comment 1•9 years ago
|
||
This blocks compilation when establishing preconditions. An alternative would be to allow compiling asm.js methods and checking in LinkAsmJS, but so far we've kept it simple and disallowed compilation for generators and arrow functions. So let's keep it that way.
![]() |
||
Updated•9 years ago
|
Attachment #8674813 -
Flags: review?(luke) → review+
Comment 3•9 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox44:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
Comment 4•9 years ago
|
||
Comment 6•9 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•