Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clear the QAST annotations when building inlining info. #356

Merged
merged 2 commits into from Jan 19, 2015
Merged

Clear the QAST annotations when building inlining info. #356

merged 2 commits into from Jan 19, 2015

Conversation

donaldh
Copy link
Contributor

@donaldh donaldh commented Jan 19, 2015

Fixes the issue with serializing NQPMatch objects into CORE.setting (with the CORE.setting text)

Issue described in https://gist.github.com/donaldh/91a212b38725642131be

@donaldh
Copy link
Contributor Author

donaldh commented Jan 19, 2015

Before
CORE.setting.jar 2.9M
% jar tvf CORE.setting.jar
25 Mon Jan 19 11:58:26 GMT 2015 META-INF/MANIFEST.MF
10787719 Mon Jan 19 11:58:26 GMT 2015 07E78BB789BE74AF2F60C21C9A44E246EB0613F6.class
8321848 Mon Jan 19 11:58:26 GMT 2015 07E78BB789BE74AF2F60C21C9A44E246EB0613F6.serialized

After
CORE.setting.jar 2.6M
% jar tvf CORE.setting.jar
25 Mon Jan 19 12:03:04 GMT 2015 META-INF/MANIFEST.MF
9176483 Mon Jan 19 12:03:04 GMT 2015 7C526785F6867A079A6061202A27DDB0F8CE8C25.class
8178018 Mon Jan 19 12:03:04 GMT 2015 7C526785F6867A079A6061202A27DDB0F8CE8C25.serialized

FROGGS added a commit that referenced this pull request Jan 19, 2015
Clear the QAST annotations when building inlining info.
@FROGGS FROGGS merged commit 0bf4263 into rakudo:nom Jan 19, 2015
zoffixznet added a commit that referenced this pull request May 3, 2017
NQP Bump brought these changes:
Raku/nqp@2017.04-48-g745c61d...2017.04-72-gc60df1e

c60df1e Bump MoarVM version
7c4bd48 Merge pull request #357 from MasterDuke17/zero_slot_when_popping_an_array
7f71ec8 Zero slot when popping an array
e3c6be0 Stop testing that nqp::sprintf converts a string that isn't a number to 0
f42720e Fix incorrect use of == when eq should be used
ae21dc4 Stop converting a '6 cute' to string
d8e1773 Avoid numifying '!'
3dbc977 Merge pull request #356 from MasterDuke17/sql_profile_output_changes
3641654 Add primary and foreign key constraints to SQL
c5fcacd Some cleanup to the SQL profile output
78b233e [js] Work around for unicode issue
2bfafb8 Reindent code
a5adc67 Test nqp::chr with an argument that bigger than 2**16
b82f609 [js] Make nqp::chr supports arguments that are bigger than 2**16
526275c [js] Fix nqp::encode with unsigned buffers
508cfaa Test method_not_found_error
bbfb1bd [js] Implement method_not_found_error
75c7430 [js] Fix bad argument name
4954f24 Test numifying 'NaN', 'Inf', '-Inf' and '+Inf'
02bdfce [js] Numify 'NaN', 'Inf', '+Inf' and '-Inf' like other backends
172950b Test QAST::Want
2f6536c [js] Remove useless NYI
cdbfe6f [js] Implement cloning integers
ef08375 [js] Remove TODO comment that doesn't provide any information

MoarVM Bump brought these changes:
MoarVM/MoarVM@2017.04-44-gf0db8822...2017.04-56-g8ad18b8

8ad18b8 Merge pull request #589 from MasterDuke17/zero_slot_when_popping_an_array
b53b05f Pull body->start + body->elems out into a variable
5fa7d8a Zero slot when popping an array
66c6dda Merge branch 'configure-telemeh-probe-rdtscp'
b373dcd give __rdtscp a more proper stub implementation
fdc653d we always define MVM_RDTSCP, but sometimes with 0 instead of 1.
387a771 fix copy-pasto, MasterDuke17++
4ba8a17 include or exclude telemetry log env var in usage/--help
1850cb9 document --no-telemeh and --telemeh
b77a3b8 enable --telemeh, --no-telemeh, and probe for rdtscp
e5b527e Rename h_vars to H_vars to make haystack vars distinguishable from n_vars
1fd9e49 lineno annotations don't have deopt index
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.

None yet

2 participants