php / php/php-src

OPcode dump of `ZEND_INIT_ARRAY` should not show `(packed)` if OP1 is unused

Ouverte
#20,978 0 commentaires 1 réaction 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Bug Status: Needs Triage
Langage dominant
C
Étoiles
40.4k
Forks
8.2k
Merge moyen
2 j 15 h
PR mergées (30 j)
103

Description

Description

https://github.com/php/php-src/blob/0caebcd1966813dd8ac7546f8e94cb6c83a1d873/ext/standard/tests/array/array_map_foreach_optimization_001.phpt#L34

The ->extended_value (which holds the ZEND_ARRAY_NOT_PACKED flag) is completely ignored when OP1 is unused, making the (packed) indicator misleading, since the array will remain uninitalized.

The (packed) should either be removed or replaced by something like (uninit).

PHP Version
git master
Operating System

No response

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par le cas lié ext/standard/tests/array/array_map_foreach_optimization_001.phpt autour de la ligne 34 et examinez le dump des opcodes pour ZEND_INIT_ARRAY. Suivez le traitement du dump pour un OP1 inutilisé et vérifiez que l’indicateur (packed) ne représente plus incorrectement l’état du tableau non initialisé ; mettez à jour ou ajoutez une attente PHPT pour montrer la sortie corrigée.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
c, php
Domaine
compilers
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
48/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.