Search
20:18:58
stassats
re ir1-phases warning, i think i got rid of by doing better optimization
20:19:47
stassats
and making code slightly more general, i.e. propagate-let-args replaces ir1-optimize-mv-bind after a little bit of modification
20:20:40
stassats
although the resulting code is the same, but the IR coming out of ir1-phases is better
20:20:41
scymtym
i will have to look at the elt &rest thing before pushing the test changes
20:20:51
stassats
scymtym: it's just a safety 3 difference
20:22:02
scymtym
i can run it with a reduce range of optimization settings
20:22:15
scymtym
but it's still a bug, isn't it?
20:22:29
stassats
no, it works as expected
20:23:08
stassats
we convert elt to nth with safety below 3
20:23:30
scymtym
but why is the warning for ELT correct?
20:23:38
stassats
and of course, my ir1-optimize-mv-bind change doesn't survive cold-init
20:23:43
stassats
scymtym: which warning? it's an error
20:24:17
scymtym
oh, right, i read my own error message wrong :)
20:24:24
scymtym
it's signalling at runtime
20:25:09
scymtym
i will just filter (safety 3) for that test
20:25:45
stassats
the test author wouldn't mind, i'm sure
20:26:01
scymtym
maybe i respect our existing tests too much
20:37:14
stassats
cold-init is as helpful as ever
21:10:26
stassats
what would really simplify the compiler: only having multiple value combinations
21:11:13
stassats
or more equal treatment between the two
21:14:43
stassats
not there yet, an ansi-test fails
21:42:21
stassats
not entirely happy with the change, could do more