{"action":"created","comment":{"url":"https://api.github.com/repos/Samsung/jerryscript/pulls/comments/41622505","id":41622505,"diff_hunk":"@@ -146,7 +253,10 @@ jsp_alloc_reg_for_temp (void)\n {\n   JERRY_ASSERT (jsp_reg_max_for_local_var == VM_IDX_EMPTY);\n \n-  vm_idx_t next_reg = jsp_reg_next++;\n+  vm_idx_t next_reg = find_free_reg ();\n+  set_reg_status (next_reg, REG_OCCUPIED);","path":"jerry-core/parser/js/opcodes-dumper.cpp","position":129,"original_position":129,"commit_id":"fa9fcbaf6c464e8dbd53bb68ebab5f87c1e67f0f","original_commit_id":"fa9fcbaf6c464e8dbd53bb68ebab5f87c1e67f0f","user":{"login":"galpeter","id":12067856,"avatar_url":"https://avatars.githubusercontent.com/u/12067856?v=3","gravatar_id":"","url":"https://api.github.com/users/galpeter","html_url":"https://github.com/galpeter","followers_url":"https://api.github.com/users/galpeter/followers","following_url":"https://api.github.com/users/galpeter/following{/other_user}","gists_url":"https://api.github.com/users/galpeter/gists{/gist_id}","starred_url":"https://api.github.com/users/galpeter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/galpeter/subscriptions","organizations_url":"https://api.github.com/users/galpeter/orgs","repos_url":"https://api.github.com/users/galpeter/repos","events_url":"https://api.github.com/users/galpeter/events{/privacy}","received_events_url":"https://api.github.com/users/galpeter/received_events","type":"User","site_admin":false},"body":"The call to the `find_free_reg` method already marks the register as used. So either we make the `find_free_reg` to not mark the register as used, or remove the `set_reg_status` call from here.","created_at":"2015-10-09T11:58:05Z","updated_at":"2015-10-09T11:58:05Z","html_url":"https://github.com/Samsung/jerryscript/pull/601#discussion_r41622505","pull_request_url":"https://api.github.com/repos/Samsung/jerryscript/pulls/601","_links":{"self":{"href":"https://api.github.com/repos/Samsung/jerryscript/pulls/comments/41622505"},"html":{"href":"https://github.com/Samsung/jerryscript/pull/601#discussion_r41622505"},"pull_request":{"href":"https://api.github.com/repos/Samsung/jerryscript/pulls/601"}}},"pull_request":{"url":"https://api.github.com/repos/Samsung/jerryscript/pulls/601","id":43493914,"html_url":"https://github.com/Samsung/jerryscript/pull/601","diff_url":"https://github.com/Samsung/jerryscript/pull/601.diff","patch_url":"https://github.com/Samsung/jerryscript/pull/601.patch","issue_url":"https://api.github.com/repos/Samsung/jerryscript/issues/601","number":601,"state":"open","locked":false,"title":"Add mechanism for reusage of tmp register variables","user":{"login":"sand1k","id":9209835,"avatar_url":"https://avatars.githubusercontent.com/u/9209835?v=3","gravatar_id":"","url":"https://api.github.com/users/sand1k","html_url":"https://github.com/sand1k","followers_url":"https://api.github.com/users/sand1k/followers","following_url":"https://api.github.com/users/sand1k/following{/other_user}","gists_url":"https://api.github.com/users/sand1k/gists{/gist_id}","starred_url":"https://api.github.com/users/sand1k/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sand1k/subscriptions","organizations_url":"https://api.github.com/users/sand1k/orgs","repos_url":"https://api.github.com/users/sand1k/repos","events_url":"https://api.github.com/users/sand1k/events{/privacy}","received_events_url":"https://api.github.com/users/sand1k/received_events","type":"User","site_admin":false},"body":"Fixes:\r\n - ch12/12.6/12.6.4/S12.6.4_A5.1 \r\n - ch12/12.6/12.6.4/S12.6.4_A5 \r\n - ch15/15.10/15.10.6/15.10.6.2/S15.10.6.2_A3_T2\r\n - ch15/15.10/15.10.6/15.10.6.2/S15.10.6.2_A3_T3\r\n - ch15/15.10/15.10.6/15.10.6.2/S15.10.6.2_A3_T4\r\n\r\nMeasurements on RPi2:\r\n\r\n                               Benchmark |         RSS<br>(+ is better) |        Perf<br>(+ is better) |\r\n                               --------- |                          --- |                         ---- |\r\n                              3d-cube.js |        132->   164 (-24.242) |      3.6648-> 3.732 (-1.834) | \r\n                  access-binary-trees.js |           88->    88 (0.000) |       2.7216-> 2.692 (1.088) | \r\n                      access-fannkuch.js |          40->    36 (10.000) |      9.892->10.0792 (-1.892) | \r\n                         access-nbody.js |           64->    60 (6.250) |      4.6504->4.7256 (-1.617) | \r\n             bitops-3bit-bits-in-byte.js |          36->    32 (11.111) |        3.244->3.2208 (0.715) | \r\n                  bitops-bits-in-byte.js |          36->    32 (11.111) |       4.432->4.4648 (-0.740) | \r\n                   bitops-bitwise-and.js |           32->    32 (0.000) |        4.188->4.1792 (0.210) | \r\n                controlflow-recursive.js |          220->   220 (0.000) |       3.176-> 3.228 (-1.637) | \r\n                    date-format-xparb.js |         100->    88 (12.000) |      2.3528->2.4624 (-4.658) | \r\n                          math-cordic.js |           48->    44 (8.333) |         4.8->4.8968 (-2.017) | \r\n                    math-partial-sums.js |          40->    36 (10.000) |        2.56->2.6368 (-3.000) | \r\n                   math-spectral-norm.js |           52->    48 (7.692) |       3.1568->3.1224 (1.090) | \r\n                         string-fasta.js |           56->    52 (7.143) |     15.0064->14.9768 (0.197) | \r\n                         Geometric mean: |       RSS reduction: 4.9704% |            Speed up: -1.071% |","created_at":"2015-08-27T09:02:18Z","updated_at":"2015-10-09T11:58:05Z","closed_at":null,"merged_at":null,"merge_commit_sha":"5043a721009ebc9760d3c80149f766b0de78a7b6","assignee":{"login":"galpeter","id":12067856,"avatar_url":"https://avatars.githubusercontent.com/u/12067856?v=3","gravatar_id":"","url":"https://api.github.com/users/galpeter","html_url":"https://github.com/galpeter","followers_url":"https://api.github.com/users/galpeter/followers","following_url":"https://api.github.com/users/galpeter/following{/other_user}","gists_url":"https://api.github.com/users/galpeter/gists{/gist_id}","starred_url":"https://api.github.com/users/galpeter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/galpeter/subscriptions","organizations_url":"https://api.github.com/users/galpeter/orgs","repos_url":"https://api.github.com/users/galpeter/repos","events_url":"https://api.github.com/users/galpeter/events{/privacy}","received_events_url":"https://api.github.com/users/galpeter/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/Samsung/jerryscript/milestones/3","html_url":"https://github.com/Samsung/jerryscript/milestones/Maintenance%20of%20ECMA%20functionality","labels_url":"https://api.github.com/repos/Samsung/jerryscript/milestones/3/labels","id":1188988,"number":3,"title":"Maintenance of ECMA functionality","description":"","creator":{"login":"egavrin","id":782710,"avatar_url":"https://avatars.githubusercontent.com/u/782710?v=3","gravatar_id":"","url":"https://api.github.com/users/egavrin","html_url":"https://github.com/egavrin","followers_url":"https://api.github.com/users/egavrin/followers","following_url":"https://api.github.com/users/egavrin/following{/other_user}","gists_url":"https://api.github.com/users/egavrin/gists{/gist_id}","starred_url":"https://api.github.com/users/egavrin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/egavrin/subscriptions","organizations_url":"https://api.github.com/users/egavrin/orgs","repos_url":"https://api.github.com/users/egavrin/repos","events_url":"https://api.github.com/users/egavrin/events{/privacy}","received_events_url":"https://api.github.com/users/egavrin/received_events","type":"User","site_admin":false},"open_issues":33,"closed_issues":62,"state":"open","created_at":"2015-07-01T13:20:58Z","updated_at":"2015-10-08T10:34:50Z","due_on":null,"closed_at":null},"commits_url":"https://api.github.com/repos/Samsung/jerryscript/pulls/601/commits","review_comments_url":"https://api.github.com/repos/Samsung/jerryscript/pulls/601/comments","review_comment_url":"https://api.github.com/repos/Samsung/jerryscript/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Samsung/jerryscript/issues/601/comments","statuses_url":"https://api.github.com/repos/Samsung/jerryscript/statuses/fa9fcbaf6c464e8dbd53bb68ebab5f87c1e67f0f","head":{"label":"Samsung:fix-not-enough-register-variables","ref":"fix-not-enough-register-variables","sha":"fa9fcbaf6c464e8dbd53bb68ebab5f87c1e67f0f","user":{"login":"Samsung","id":6210390,"avatar_url":"https://avatars.githubusercontent.com/u/6210390?v=3","gravatar_id":"","url":"https://api.github.com/users/Samsung","html_url":"https://github.com/Samsung","followers_url":"https://api.github.com/users/Samsung/followers","following_url":"https://api.github.com/users/Samsung/following{/other_user}","gists_url":"https://api.github.com/users/Samsung/gists{/gist_id}","starred_url":"https://api.github.com/users/Samsung/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Samsung/subscriptions","organizations_url":"https://api.github.com/users/Samsung/orgs","repos_url":"https://api.github.com/users/Samsung/repos","events_url":"https://api.github.com/users/Samsung/events{/privacy}","received_events_url":"https://api.github.com/users/Samsung/received_events","type":"Organization","site_admin":false},"repo":{"id":33339660,"name":"jerryscript","full_name":"Samsung/jerryscript","owner":{"login":"Samsung","id":6210390,"avatar_url":"https://avatars.githubusercontent.com/u/6210390?v=3","gravatar_id":"","url":"https://api.github.com/users/Samsung","html_url":"https://github.com/Samsung","followers_url":"https://api.github.com/users/Samsung/followers","following_url":"https://api.github.com/users/Samsung/following{/other_user}","gists_url":"https://api.github.com/users/Samsung/gists{/gist_id}","starred_url":"https://api.github.com/users/Samsung/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Samsung/subscriptions","organizations_url":"https://api.github.com/users/Samsung/orgs","repos_url":"https://api.github.com/users/Samsung/repos","events_url":"https://api.github.com/users/Samsung/events{/privacy}","received_events_url":"https://api.github.com/users/Samsung/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Samsung/jerryscript","description":"Ultra-lightweight JavaScript engine for Internet of Things. http://www.jerryscript.net","fork":false,"url":"https://api.github.com/repos/Samsung/jerryscript","forks_url":"https://api.github.com/repos/Samsung/jerryscript/forks","keys_url":"https://api.github.com/repos/Samsung/jerryscript/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Samsung/jerryscript/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Samsung/jerryscript/teams","hooks_url":"https://api.github.com/repos/Samsung/jerryscript/hooks","issue_events_url":"https://api.github.com/repos/Samsung/jerryscript/issues/events{/number}","events_url":"https://api.github.com/repos/Samsung/jerryscript/events","assignees_url":"https://api.github.com/repos/Samsung/jerryscript/assignees{/user}","branches_url":"https://api.github.com/repos/Samsung/jerryscript/branches{/branch}","tags_url":"https://api.github.com/repos/Samsung/jerryscript/tags","blobs_url":"https://api.github.com/repos/Samsung/jerryscript/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Samsung/jerryscript/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Samsung/jerryscript/git/refs{/sha}","trees_url":"https://api.github.com/repos/Samsung/jerryscript/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Samsung/jerryscript/statuses/{sha}","languages_url":"https://api.github.com/repos/Samsung/jerryscript/languages","stargazers_url":"https://api.github.com/repos/Samsung/jerryscript/stargazers","contributors_url":"https://api.github.com/repos/Samsung/jerryscript/contributors","subscribers_url":"https://api.github.com/repos/Samsung/jerryscript/subscribers","subscription_url":"https://api.github.com/repos/Samsung/jerryscript/subscription","commits_url":"https://api.github.com/repos/Samsung/jerryscript/commits{/sha}","git_commits_url":"https://api.github.com/repos/Samsung/jerryscript/git/commits{/sha}","comments_url":"https://api.github.com/repos/Samsung/jerryscript/comments{/number}","issue_comment_url":"https://api.github.com/repos/Samsung/jerryscript/issues/comments{/number}","contents_url":"https://api.github.com/repos/Samsung/jerryscript/contents/{+path}","compare_url":"https://api.github.com/repos/Samsung/jerryscript/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Samsung/jerryscript/merges","archive_url":"https://api.github.com/repos/Samsung/jerryscript/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Samsung/jerryscript/downloads","issues_url":"https://api.github.com/repos/Samsung/jerryscript/issues{/number}","pulls_url":"https://api.github.com/repos/Samsung/jerryscript/pulls{/number}","milestones_url":"https://api.github.com/repos/Samsung/jerryscript/milestones{/number}","notifications_url":"https://api.github.com/repos/Samsung/jerryscript/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Samsung/jerryscript/labels{/name}","releases_url":"https://api.github.com/repos/Samsung/jerryscript/releases{/id}","created_at":"2015-04-03T01:10:31Z","updated_at":"2015-10-09T11:38:54Z","pushed_at":"2015-10-09T06:44:17Z","git_url":"git://github.com/Samsung/jerryscript.git","ssh_url":"git@github.com:Samsung/jerryscript.git","clone_url":"https://github.com/Samsung/jerryscript.git","svn_url":"https://github.com/Samsung/jerryscript","homepage":"","size":12948,"stargazers_count":964,"watchers_count":964,"language":"C++","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":70,"mirror_url":null,"open_issues_count":113,"forks":70,"open_issues":113,"watchers":964,"default_branch":"master"}},"base":{"label":"Samsung:master","ref":"master","sha":"08c618e8c5d9946cab74a85c51757573898e34ec","user":{"login":"Samsung","id":6210390,"avatar_url":"https://avatars.githubusercontent.com/u/6210390?v=3","gravatar_id":"","url":"https://api.github.com/users/Samsung","html_url":"https://github.com/Samsung","followers_url":"https://api.github.com/users/Samsung/followers","following_url":"https://api.github.com/users/Samsung/following{/other_user}","gists_url":"https://api.github.com/users/Samsung/gists{/gist_id}","starred_url":"https://api.github.com/users/Samsung/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Samsung/subscriptions","organizations_url":"https://api.github.com/users/Samsung/orgs","repos_url":"https://api.github.com/users/Samsung/repos","events_url":"https://api.github.com/users/Samsung/events{/privacy}","received_events_url":"https://api.github.com/users/Samsung/received_events","type":"Organization","site_admin":false},"repo":{"id":33339660,"name":"jerryscript","full_name":"Samsung/jerryscript","owner":{"login":"Samsung","id":6210390,"avatar_url":"https://avatars.githubusercontent.com/u/6210390?v=3","gravatar_id":"","url":"https://api.github.com/users/Samsung","html_url":"https://github.com/Samsung","followers_url":"https://api.github.com/users/Samsung/followers","following_url":"https://api.github.com/users/Samsung/following{/other_user}","gists_url":"https://api.github.com/users/Samsung/gists{/gist_id}","starred_url":"https://api.github.com/users/Samsung/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Samsung/subscriptions","organizations_url":"https://api.github.com/users/Samsung/orgs","repos_url":"https://api.github.com/users/Samsung/repos","events_url":"https://api.github.com/users/Samsung/events{/privacy}","received_events_url":"https://api.github.com/users/Samsung/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Samsung/jerryscript","description":"Ultra-lightweight JavaScript engine for Internet of Things. http://www.jerryscript.net","fork":false,"url":"https://api.github.com/repos/Samsung/jerryscript","forks_url":"https://api.github.com/repos/Samsung/jerryscript/forks","keys_url":"https://api.github.com/repos/Samsung/jerryscript/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Samsung/jerryscript/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Samsung/jerryscript/teams","hooks_url":"https://api.github.com/repos/Samsung/jerryscript/hooks","issue_events_url":"https://api.github.com/repos/Samsung/jerryscript/issues/events{/number}","events_url":"https://api.github.com/repos/Samsung/jerryscript/events","assignees_url":"https://api.github.com/repos/Samsung/jerryscript/assignees{/user}","branches_url":"https://api.github.com/repos/Samsung/jerryscript/branches{/branch}","tags_url":"https://api.github.com/repos/Samsung/jerryscript/tags","blobs_url":"https://api.github.com/repos/Samsung/jerryscript/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Samsung/jerryscript/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Samsung/jerryscript/git/refs{/sha}","trees_url":"https://api.github.com/repos/Samsung/jerryscript/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Samsung/jerryscript/statuses/{sha}","languages_url":"https://api.github.com/repos/Samsung/jerryscript/languages","stargazers_url":"https://api.github.com/repos/Samsung/jerryscript/stargazers","contributors_url":"https://api.github.com/repos/Samsung/jerryscript/contributors","subscribers_url":"https://api.github.com/repos/Samsung/jerryscript/subscribers","subscription_url":"https://api.github.com/repos/Samsung/jerryscript/subscription","commits_url":"https://api.github.com/repos/Samsung/jerryscript/commits{/sha}","git_commits_url":"https://api.github.com/repos/Samsung/jerryscript/git/commits{/sha}","comments_url":"https://api.github.com/repos/Samsung/jerryscript/comments{/number}","issue_comment_url":"https://api.github.com/repos/Samsung/jerryscript/issues/comments{/number}","contents_url":"https://api.github.com/repos/Samsung/jerryscript/contents/{+path}","compare_url":"https://api.github.com/repos/Samsung/jerryscript/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Samsung/jerryscript/merges","archive_url":"https://api.github.com/repos/Samsung/jerryscript/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Samsung/jerryscript/downloads","issues_url":"https://api.github.com/repos/Samsung/jerryscript/issues{/number}","pulls_url":"https://api.github.com/repos/Samsung/jerryscript/pulls{/number}","milestones_url":"https://api.github.com/repos/Samsung/jerryscript/milestones{/number}","notifications_url":"https://api.github.com/repos/Samsung/jerryscript/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Samsung/jerryscript/labels{/name}","releases_url":"https://api.github.com/repos/Samsung/jerryscript/releases{/id}","created_at":"2015-04-03T01:10:31Z","updated_at":"2015-10-09T11:38:54Z","pushed_at":"2015-10-09T06:44:17Z","git_url":"git://github.com/Samsung/jerryscript.git","ssh_url":"git@github.com:Samsung/jerryscript.git","clone_url":"https://github.com/Samsung/jerryscript.git","svn_url":"https://github.com/Samsung/jerryscript","homepage":"","size":12948,"stargazers_count":964,"watchers_count":964,"language":"C++","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":70,"mirror_url":null,"open_issues_count":113,"forks":70,"open_issues":113,"watchers":964,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Samsung/jerryscript/pulls/601"},"html":{"href":"https://github.com/Samsung/jerryscript/pull/601"},"issue":{"href":"https://api.github.com/repos/Samsung/jerryscript/issues/601"},"comments":{"href":"https://api.github.com/repos/Samsung/jerryscript/issues/601/comments"},"review_comments":{"href":"https://api.github.com/repos/Samsung/jerryscript/pulls/601/comments"},"review_comment":{"href":"https://api.github.com/repos/Samsung/jerryscript/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Samsung/jerryscript/pulls/601/commits"},"statuses":{"href":"https://api.github.com/repos/Samsung/jerryscript/statuses/fa9fcbaf6c464e8dbd53bb68ebab5f87c1e67f0f"}}}}