Closed Bug 1437780 Opened 5 years ago Closed 5 years ago

Wasm baseline: Remove more platform dependencies

Categories

(Core :: JavaScript Engine: JIT, enhancement, P3)

Product:
Core
Component:
JavaScript Engine: JIT
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla60
Tracking Flags:
Tracking Status
firefox60 --- fixed

People

(Reporter: lth, Assigned: lth)

References

Details

Attachments

(1 file, 1 obsolete file)

bug1437780-rabaldr-remove-ifdefs-v2.patch
1.70 KB, patch
lth
: review+
Details | Diff | Splinter Review 
This patch pulls in changes from ongoing ARM64 (bug 1407535) and MIPS32/MIPS64 (bug 1284414) work, and removes platform dependencies in the wasm baseline compiler.  This is made possible by recent work on the porting APIs.  The ongoing porting work on those platforms will need to adapt to this change but the net result is further simplification of the code and of the patches.

        Attachment #8950501 -
        Flags: review?(bbouvier)

    
    

    
  
Comment on attachment 8950501 [details] [diff] [review]
bug1437780-rabaldr-remove-ifdefs.patch

Review of attachment 8950501 [details] [diff] [review]:
-----------------------------------------------------------------

Nice

        Attachment #8950501 -
        Flags: review?(bbouvier) → review+

    
    

    
  
Pushed by lhansen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4661476ec0a0
Wasm baseline, remove platform dependencies. r=bbouvier
Priority: -- → P3

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/4661476ec0a0
Status: ASSIGNED → RESOLVED
Closed: 5 years ago

          status-firefox60:
          --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla60

    
    

    
  


  
Rebased etc, ready to land.  Carrying r+.

        Attachment #8950501 -
        Attachment is obsolete: true

        Attachment #8954002 -
        Flags: review+

    
    

    
  
Pushed by lhansen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/63f61a9297ce
Wasm baseline, remove platform dependencies. r=bbouvier

          You need to log in
          before you can comment on or make changes to this bug.