Stdlib: chunk_split() separator enum TypeError (#6032)

ext/standard/chunk_split.php

@@ -8,8 +8,8 @@
 use PHPCompiler\Func\Internal;
 use PHPCompiler\JIT\BasicBlockHelper;
 use PHPCompiler\JIT\Context;
+use PHPCompiler\JIT\JitStringBuiltinArg;
 use PHPCompiler\JIT\Variable as JITVariable;
-use PHPCompiler\VM\Variable;
 use PHPLLVM\Value;
 
 /** chunk_split() — insert a separator every N bytes (subset of PHP). */
@@ -43,11 +43,12 @@ public function execute(Frame $frame): void
         }
         $separator = "\r\n";
         if (3 === $argc) {
-            $sepArg = $frame->calledArgs[2]->resolveIndirect();
-            if (Variable::TYPE_STRING !== $sepArg->type) {
-                throw new \LogicException('chunk_split() separator must be a string in this compiler build');
-            }
-            $separator = $sepArg->toString();
+            $separator = VmString::coerceStringBuiltinArg(
+                $frame->calledArgs[2],
+                'chunk_split',
+                2,
+                'separator'
+            );
         }
         $result = VmString::chunkSplit($string, $length, $separator);
         if (null === $frame->returnVar) {
@@ -73,7 +74,7 @@ public function call(Context $context, JITVariable ...$args): Value
             JitChunkSplit::emitRuntimeLengthGuard($context, $chunkLen);
         }
         if ($argc >= 3) {
-            $separator = $this->jitString($context, $args[2], 'chunk_split() argument #3');
+            $separator = JitStringBuiltinArg::lower($context, $args[2], 'chunk_split', 2, 'separator');
         } else {
             $separator = $context->builder->load($context->constantStringFromString("\r\n"));
         }

test/compliance/cases/stdlib/chunk_split_enum_separator.phpt

@@ -0,0 +1,13 @@
+--TEST--
+stdlib chunk_split() — backed enum case separator TypeError (#6032, ext/standard/string.c, php-src-strict)
+--FILE--
+<?php
+enum ES: string { case X = '-'; }
+try {
+    chunk_split('abc', 2, ES::X);
+    echo "uncaught\n";
+} catch (TypeError $e) {
+    echo $e->getMessage(), "\n";
+}
+--EXPECT--
+chunk_split(): Argument #3 ($separator) must be of type string, ES given

test/compliance/cases/stdlib/chunk_split_enum_separator_jit.phpt

@@ -0,0 +1,13 @@
+--TEST--
+stdlib chunk_split() JIT — backed enum case separator TypeError (#6032)
+--FILE--
+<?php
+enum ES: string { case X = '-'; }
+try {
+    chunk_split('abc', 2, ES::X);
+    echo "uncaught\n";
+} catch (TypeError $e) {
+    echo $e->getMessage(), "\n";
+}
+--EXPECT--
+chunk_split(): Argument #3 ($separator) must be of type string, ES given

test/fixtures/aot/compile-only/chunk_split_enum_separator_typeerror.php

@@ -0,0 +1,8 @@
+<?php
+// Compile-only (#6032): chunk_split() separator must lower enum-case TypeError guards for AOT.
+enum ES: string { case X = '-'; }
+try {
+    chunk_split('abc', 2, ES::X);
+} catch (TypeError $e) {
+    echo $e->getMessage(), "\n";
+}

test/repro/issue_6032_chunk_split_enum_separator.php

@@ -0,0 +1,9 @@
+<?php
+// Issue #6032 — chunk_split() separator enum case must TypeError (ext/standard/string.c)
+enum ES: string { case X = '-'; }
+try {
+    chunk_split('abc', 2, ES::X);
+    echo "uncaught\n";
+} catch (TypeError $e) {
+    echo $e->getMessage(), "\n";
+}

test/unit/CountCharsChunkSplitEnumJitCompileTest.php

@@ -65,4 +65,23 @@ enum EI: int { case A = 1; }
             $bc
         );
     }
+
+    public function testChunkSplitEnumSeparatorTypeErrorLowering(): void
+    {
+        $code = <<<'PHP'
+<?php
+enum ES: string { case X = '-'; }
+chunk_split('abc', 2, ES::X);
+PHP;
+        $runtime = new Runtime();
+        $block = $runtime->parseAndCompile($code, 'chunk_split_enum_separator_jit_compile.php');
+        $runtime->jitCompileBlock($block);
+
+        $context = $runtime->loadJitContext();
+        $bc = $context->module->printToString();
+        $this->assertStringContainsString(
+            'chunk_split(): Argument #3 ($separator) must be of type string',
+            $bc
+        );
+    }
 }