Skip to content

Conversation

@jianchun
Copy link

No description provided.

ThomsonTan and others added 13 commits July 13, 2016 11:55
…mentally across blocks

Merge pull request chakra-core#1247 from ThomsonTan:IncBailOut

Capture values for bailout are built incrementally by selecting captured values in one of its predecessor block and mark all not uncommon symbols from all predecessor blocks as recapture when building new bailout.
…offset on InlineeEnd, and around boxing of inlinees in a jitted loop body
…n the tests

Merge pull request chakra-core#1299 from xanlpz:timezone-tests

This reduces the failing tests from 21 to 1 if the right timezone is not set system-wide. The failing test comes from using PST and not Pacific Daylight Time, but not sure how to magically get the daylight stuff in here. In any case I think this is already an improvement.
…ype jit-optimized, right bailout offset on InlineeEnd, and around boxing of inlinees in a jitted loop body

Merge pull request chakra-core#1295 from rajatd:Bugs

1. Inlinees in a loop body frame need not marked as candidates for boxing as we wouldn't have allocated nested functions on the stack for a jitted loop body frame (both for loop body and inlinees in it)

2. CheckObjType instruction shouldn't add a bytecode use of the symbol whose type it is checking. For this purpose, I'm marking the source operand as jit-optimized.

3. Bailout offset on InlineeEnd should be that of the next real instruction after the call to the inlinee.
…n and shorthand object pattern on param is valid syntax.

Merge pull request chakra-core#1315 from akroshg:destruct2
include  before PAL header file on non
Merge pull request chakra-core#1317 from obastemur:fix_build_break

include  `<string>` before PAL header file on non `_WIN32`
@chakrabot chakrabot merged commit 56fe248 into chakra-core:linux Jul 22, 2016
chakrabot pushed a commit that referenced this pull request Jul 22, 2016
…into linux

Merge pull request #1320 from jianchun:filinux
@jianchun jianchun deleted the filinux branch July 22, 2016 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants