enhancementgood first issuehelp wantedpir
Métriques du dépôt
- Stars
- (61 étoiles)
- Métriques de merge PR
- (Merge moyen 1m) (1 PR mergée en 30 j)
Description
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.