Skip to content
This repository was archived by the owner on Feb 12, 2022. It is now read-only.

Adds a flag for Array.prototype method nested optimized functions #2404

Closed
wants to merge 6 commits into from
Closed

Adds a flag for Array.prototype method nested optimized functions #2404

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
2f5a6cb
Adds a flag for Array.prototype method nested optimized functions
trueadm Aug 9, 2018
e233d31
added explicit flag to tests
trueadm Aug 9, 2018
983b93a
add flag to test
trueadm Aug 9, 2018
03ad53d
Merge remote-tracking branch 'upstream/master' into flag-for-array-ne…
trueadm Aug 9, 2018
a6df9ed
Merge remote-tracking branch 'upstream/master' into flag-for-array-ne…
trueadm Aug 10, 2018
fd6da58
merge master and add flags
trueadm Aug 10, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
add flag to test
  • Loading branch information
@trueadm
trueadm committed Aug 9, 2018
commit 983b93aec726e51ab53f94715407f5af9a7ce6dc
1 change: 1 addition & 0 deletions test/serializer/optimized-functions/DefineOptFuncInsideFuncInsideOptFunc.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// arrayNestedOptimizedFunctionsEnabled
// skip lint
// The original issue here was that nested is defined inside of fn2 which is a non-optimized function
// called by fn (an optimized function). That caused Prepack to not detect that nested was nested
Expand Down