Skip to content

Stdlib: ldap_set_option()/ldap_get_option() JIT lowering missing — VM path only (ext/ldap/ldap.c) #32107

Description

@PurHur

Category

stdlib · php-src-strict · ext/ldap

Problem

ldap_set_option() and ldap_get_option() run on VM but JIT/AOT still throw LogicException from call() despite VM-native implementation in VmLdapNative.

php-src reference

PHP implementation target

  • ext/ldap/ldap_link_builtins.php
  • ext/ldap/JitLdapLink.php + ext/ldap/LdapLinkJitHelper.php
  • lib/JIT/Builtin/LdapRuntime.php

Done when

  • JIT/AOT accept LDAP\Connection|null, int $option, by-ref out arg for ldap_get_option
  • Return/errno/warning behavior matches VM for int/bool option subset currently implemented
  • Targeted unit/repro coverage added

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    area:compilerCompiler / CFG / JITbugSomething isn't workingimplementation-readySpec complete: repro, php-src ref, done-when — safe for workers to claimphase-4:stdlibPhase 4 – stdlib for web appsstdlib

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions