1393347 - mips spew

Status: RESOLVED FIXED

(Core :: JavaScript Engine: JIT, defect)

Description

[yuyin]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_5) AppleWebKit/603.2.4 (KHTML, like Gecko) Version/10.1.1 Safari/603.2.4

Steps to reproduce:

when we debug jit-code, we can set IONFLAGS=codegen, and then in debug mode, we can get instructions information along with LIR on x86 platform, like:

[Codegen] instruction StackArgT
[Codegen] movabsq $0xfff90000, %r11
[Codegen] movq   %r11, 0x0(%rsp)

but on mips platform, we can only get:

[Codegen] instruction StackArgT



Actual results:

no informations about what real instructions use.


Expected results:

instruction information along with LIR like x86.

Comment 1

[yuyin]

Attachment: 0001-MIPS-Output-some-more-infomation-when-debug-jit-code.patch

Comment 2

[hev [:hev]]

Comment on attachment 8900598 [details] [diff] [review]
0001-MIPS-Output-some-more-infomation-when-debug-jit-code.patch

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

::: js/src/jit/mips-shared/Assembler-mips-shared.cpp
@@ +1377,4 @@
>  BufferOffset
>  AssemblerMIPSShared::as_ceilws(FloatRegister fd, FloatRegister fs)
>  {
> +    spew("ceil.w.s%3s,%3s", fd.name(), fs.name());

no space.

::: js/src/jit/mips-shared/MacroAssembler-mips-shared.cpp
@@ +1579,4 @@
>      as_bal(BOffImm16(3 * sizeof(uint32_t)));
>      addPtr(Imm32(5 * sizeof(uint32_t)), ra);
>      // Allocate space which will be patched by patchCall().
> +    spew(".space 32bit initValue 0xffff ffff");

.word 0xffffffff

Comment 3

[yuyin]

ceil.w.s f0, f1. 
%3s

(In reply to Heiher [:hev] from comment #2)
> Comment on attachment 8900598 [details] [diff] [review]
> 0001-MIPS-Output-some-more-infomation-when-debug-jit-code.patch
> 
> Review of attachment 8900598 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: js/src/jit/mips-shared/Assembler-mips-shared.cpp
> @@ +1377,4 @@
> >  BufferOffset
> >  AssemblerMIPSShared::as_ceilws(FloatRegister fd, FloatRegister fs)
> >  {
> > +    spew("ceil.w.s%3s,%3s", fd.name(), fs.name());
> 
> no space.
> 
> ::: js/src/jit/mips-shared/MacroAssembler-mips-shared.cpp
> @@ +1579,4 @@
> >      as_bal(BOffImm16(3 * sizeof(uint32_t)));
> >      addPtr(Imm32(5 * sizeof(uint32_t)), ra);
> >      // Allocate space which will be patched by patchCall().
> > +    spew(".space 32bit initValue 0xffff ffff");
> 
> .word 0xffffffff

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2c395a5be25d
MIPS: Output some more infomation when debug jit code. r=luke

Comment 5

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/2c395a5be25d