swiftlang/swift

[SR-340] Refactor SILParser::parseSILInstruction

Open

#42 962 ouverte le 22 déc. 2015

Voir sur GitHub
 (7 commentaires) (0 réactions) (1 assigné)Swift (10 719 forks)batch import
SILParsercompilergood first issuenfc

Métriques du dépôt

Stars
 (69 989 stars)
Métriques de merge PR
 (Merge moyen 8j 17h) (510 PRs mergées en 30 j)

Description
Previous ID SR-340
Radar None
Original Reporter @gottesmm
Type Bug
Status In Progress
Resolution
Votes 1
Component/s Compiler
Labels Bug, StarterBug
Assignee d066z (JIRA)
Priority Medium

md5: ee6df5ef735c9655c56ca6ce06200e7e

Issue Description:

parseSILInstruction is horrible and makes me cry every time I see it. It is a method that is ~1900 lines with a huge switch in it. We should refactor it into a visitor structure. In fact it is large enough that we should consider moving it into its own file if it is possible.

Guide contributeur

Direction de recherche
Familiarisez vous avec la classe SILParser, en particulier la méthode parseSILInstruction. Analysez la structure actuelle du switch case et proposez un modèle de visiteur pour diviser la grande méthode en méthodes visiteurs plus petites et plus faciles à maintenir. Envisagez de déplacer le visiteur dans un fichier séparé si possible.
Stack technique
swiftcpp
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
3-5 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitSwiftC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75