MIPS simulator doesn't build

RESOLVED FIXED in Firefox 50

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jonco, Unassigned)

Tracking

unspecified
mozilla50
Points:
---

Firefox Tracking Flags

(firefox50 fixed)

Details

Attachments

(1 attachment)

Reporter

Description

3 years ago
I get the following error trying to make a 64-bit MIPS simulator build:

/home/jon/clone/dev/js/src/jit/Ion.cpp:1893:18: error: cannot declare variable ‘lirgen’ to be of abstract type ‘js::jit::LIRGenerator’
     LIRGenerator lirgen(mir, graph, *lir);
                  ^
In file included from /home/jon/clone/dev/js/src/jit/Ion.cpp:40:0,
                 from /home/jon/clone/dev/js/src/mips64sim-optdebug-build/js/src/Unified_cpp_js_src14.cpp:38:
/home/jon/clone/dev/js/src/jit/Lowering.h:35:7: note:   because the following virtual functions are pure within ‘js::jit::LIRGenerator’:
 class LIRGenerator : public LIRGeneratorSpecific
       ^
In file included from /home/jon/clone/dev/js/src/jit/MIR.h:25:0,
                 from /home/jon/clone/dev/js/src/jit/MIRGenerator.h:23,
                 from /home/jon/clone/dev/js/src/jit/EffectiveAddressAnalysis.h:10,
                 from /home/jon/clone/dev/js/src/jit/EffectiveAddressAnalysis.cpp:7,
                 from /home/jon/clone/dev/js/src/mips64sim-optdebug-build/js/src/Unified_cpp_js_src14.cpp:2:
/home/jon/clone/dev/js/src/jit/MOpcodes.h:314:13902: note: 	virtual void js::jit::MDefinitionVisitor::visitCopySign(js::jit::MCopySign*)
In file included from /home/jon/clone/dev/js/src/jit/BaselineJIT.h:18:0,
                 from /home/jon/clone/dev/js/src/jit/BaselineIC.h:20,
                 from /home/jon/clone/dev/js/src/jit/MIR.h:20,
                 from /home/jon/clone/dev/js/src/jit/MIRGenerator.h:23,
                 from /home/jon/clone/dev/js/src/jit/EffectiveAddressAnalysis.h:10,
                 from /home/jon/clone/dev/js/src/jit/EffectiveAddressAnalysis.cpp:7,
                 from /home/jon/clone/dev/js/src/mips64sim-optdebug-build/js/src/Unified_cpp_js_src14.cpp:2:

It looks like this might have been broken by bug 1275224.
Reporter

Updated

3 years ago
Flags: needinfo?(bbouvier)
Flags: needinfo?(bbouvier)
Reporter

Updated

3 years ago
Attachment #8762074 - Flags: review?(jcoppeard) → review+
Reporter

Comment 2

3 years ago
Comment on attachment 8762074 [details]
Bug 1279524: Implement dummy copysign on MIPS;

https://reviewboard.mozilla.org/r/58962/#review55892

Cheers for the quick fix.  I checked that this fixed the build error.

Comment 3

3 years ago
Pushed by benj@benj.me:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2656ba248399
Implement dummy copysign on MIPS; r=jonco

Comment 4

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/2656ba248399
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.