reactorlabs/rir

sequences starting with non-constant value

Offen

#789 geöffnet am 22.01.2020

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (19 Forks)github user discovery
enhancementgood first issuehelp wantedpir

Repository-Metriken

Stars
 (61 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 1m) (1 gemergte PR in 30 T)

Beschreibung

For some reason in https://github.com/reactorlabs/rir/blob/master/rir/src/ir/Compiler.cpp#L238 we only properly compile a simple for loop for (i in a:b) if a is a numerical constant.

This causes us to allocate lots of unnecessary vectors (e.g. in nbody_naive_2).

I suspect that this is a hack, exploiting the fact that a < b happens to error similarly to a:b if a is a number.

This should be fixable quite easily by emitting a check for the type of a ensuring that it is compatible with : first.

Contributor Guide

Research-Richtung
Überprüfe Compiler.cpp um Zeile 238, um die aktuelle Behandlung von `for (i in a:b)` zu verstehen. Identifiziere, wo die Typprüfung für `a` ausgegeben werden soll, um die Kompatibilität mit dem `:` Operator sicherzustellen. Implementiere dann die Typprüfung vor der Sequenzerzeugung.
Tech Stack
cpp
Domain
compilers
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85