Bug 196103 – testB3::testReportUsedRegistersLateUseFollowedByEarlyDefDoesNotMarkUseAsDead() has a validation failure in debug mode

• Status: RESOLVED
• Resolution: FIXED
• Component: JavaScriptCore

[Robin Morisset]

On top-of-tree (only in debug mode):

BB#0: ; frequency = 1.000000
    Void @0 = Patchpoint($42(@1):Register(%rax), $42(@1):Register(%rcx), $42(@1):Register(%rdx), $42(@1):Register(%rbx), $42(@1):Register(%rsi), $42(@1):Register(%rdi), $42(@1):Register(%r8), $42(@1):Register(%r9), $42(@1):Register(%r10), $42(@1):Register(%r11), $42(@1):Register(%r12), $42(@1):Register(%r13), $42(@1):Register(%r14), $42(@1):Register(%r15), generator = 0x1187a2250, earlyClobbered = [], lateClobbered = [], usedRegisters = [], resultConstraint = WarmAny, ExitsSideways|ControlDependent|WritesPinned|ReadsPinned|Fence|Writes:Top|Reads:Top)
    Int32 @1 = Const32(42)
    Void @2 = Patchpoint($10(@3):LateRegister(%rax), $10(@3):LateRegister(%rcx), $10(@3):LateRegister(%rdx), $10(@3):LateRegister(%rbx), $10(@3):LateRegister(%rsi), $10(@3):LateRegister(%rdi), $10(@3):LateRegister(%r8), $10(@3):LateRegister(%r9), $10(@3):LateRegister(%r10), $10(@3):LateRegister(%r11), $10(@3):LateRegister(%r12), $10(@3):LateRegister(%r13), $10(@3):LateRegister(%r14), $10(@3):LateRegister(%r15), generator = 0x118f18c30, earlyClobbered = [], lateClobbered = [], usedRegisters = [], resultConstraint = WarmAny, ExitsSideways|ControlDependent|WritesPinned|ReadsPinned|Fence|Writes:Top|Reads:Top)
    Int32 @3 = Const32(10)
    Int32 @4 = Patchpoint(generator = 0x118f18c48, earlyClobbered = [], lateClobbered = [], usedRegisters = [], resultConstraint = SomeEarlyRegister, ExitsSideways|ControlDependent|WritesPinned|ReadsPinned|Fence|Writes:Top|Reads:Top)
    Void @5 = Return(Terminal)
B3 VALIDATION FAILURE
    valueIndex.get(value) > valueIndex.get(child) (./b3/B3Validate.cpp:104)
    At @0->$42(@1)
    After initial

This test was introduced in https://bugs.webkit.org/show_bug.cgi?id=195303

[Radar WebKit Bug Importer]

<rdar://problem/57808549>

[Robin Morisset]

Created attachment 401956 [details]
Patch

[Keith Miller]

Comment on attachment 401956 [details]
Patch

r=me

[Robin Morisset]

(In reply to Keith Miller from comment #3)
> Comment on attachment 401956 [details]
> Patch
> 
> r=me

Thanks for the review.

[EWS]

Committed r263068: <https://trac.webkit.org/changeset/263068>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 401956 [details].