Skip to content

Perl no get context new constsub flags#12

Open
bulk88 wants to merge 2 commits into
atoomic:masterfrom
bulk88:PERL_NO_GET_CONTEXT_newCONSTSUB_flags
Open

Perl no get context new constsub flags#12
bulk88 wants to merge 2 commits into
atoomic:masterfrom
bulk88:PERL_NO_GET_CONTEXT_newCONSTSUB_flags

Conversation

@bulk88

@bulk88 bulk88 commented Apr 22, 2025

Copy link
Copy Markdown
  • add #define PERL_NO_GET_CONTEXT to stop the Perl_get_context() calls

    • Perl_get_context() is a slow TLS getter func, and its only for src code
      compat with abandoned ancient pre-ithreads XS modules.
  • add newCONSTSUB_flags() so PP code can constant fold

bulk88 added 2 commits April 22, 2025 14:21
-Perl_get_context() is a slow TLS getter func, and its only for src code
compat with abandoned ancient pre-ithreads XS modules.
@atoomic

atoomic commented Apr 27, 2025

Copy link
Copy Markdown
Owner

@bulk88 It seems that the current change are failing for Perl <= 5.16
would you mind adjusting?
thanks

@atoomic atoomic left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Current changes are incompatible with Perl <= 5.16

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.

2 participants