Search
Thursday, 14th of January 2021, 10:06:12 UTC
16:25:53
karlosz
another thing - clone-ast copies the policy list like a billion times
16:26:50
karlosz
i'm not sure just copying all the slots is really the best thing to do
16:28:29
Bike
oh, the policy shouldn't be copied ever really.
16:28:44
karlosz
but the way clone-ast is written it will be
16:29:35
karlosz
i think separating references from variables at the ast-level is pretty orthogonal to the issue of map-ast having to keep a hash table
16:29:51
karlosz
but we should probably do it anyway for better policy and source like your old commit said
16:30:27
karlosz
all we really need to do to make map-ast not use the hash table is to make sure we don't visit any child more than once, which we can do by controlling the children method
16:30:53
karlosz
i.e. only function-ast and lexical-bind-ast have lexical-ast/lexical-variable as children
18:05:23
karlosz
i guess its pretty easy to solve the problem of unnecessarily making new conses just by checking whether the car and cdr remain the same
18:09:06
karlosz
Bike: could you merge the bignum dumping pr? i have clasp side changes queued up for the ast munging
18:31:22
karlosz
an interesting not about the ast interpreter - we don't actually need a linked list of frames, since the environment has already disappeared by the time we converted to an ast, so just a flat mapping will do
18:44:49
Bike
build seems to segfault now. hm.
18:45:18
karlosz
after merging the bignum?
18:45:35
karlosz
hm, i ran the ansi-tests
18:45:54
Bike
also, not a segfault, sigbus.
18:46:01
karlosz
like just a normal build, or with cando etc?
18:46:01
Bike
i'll make sure it worked before the merge
18:46:29
karlosz
i don't think i've ever seen a sigbus from clasp before
18:46:47
Bike
the actual error is UNDEFINED-FUNCTION (BUS-ERROR)
18:47:00
Bike
maybe it's the "Undefined instruction" error from infinite recursion? I dunno
18:47:20
karlosz
well the buildbot just finished building asdf
18:49:46
karlosz
where in the build are you getting the error?
18:50:12
karlosz
did you distclean and everything?
18:50:22
karlosz
buildbot is currently in the middle of building quicklisp
18:50:59
karlosz
you're building on bigmac right? that's pretty weird
19:09:53
Bike
works before the merge. very odd.
19:20:26
karlosz
yeah, given that the buildbot just succeeded on it too
19:20:36
karlosz
well a sigbus means something nefarious
20:36:24
kpoeck
i also get datum: UNDEFINED-FUNCTION (:NAME BUS-ERROR)
20:36:55
kpoeck
23: 25 iclasp-boehm 0x0000000105ba4178 _ZL7startupiPPcRbRiS2_ + 1304
20:42:36
kpoeck
i even have a backtrace in lldb
20:44:25
kpoeck
core::ltvc_read_bignum is in the backtrace
20:46:18
kpoeck
here all info: https://gist.github.com/kpoeck/e077bfac7c8685170fc76053cb02e9d7
21:13:44
karlosz
kpoeck: thanks a lot, i can look at that
21:13:51
karlosz
that's weird that it only happens sometimes
21:13:58
karlosz
or for certain people i guess
21:14:04
karlosz
clasp has been building fine on bigmac for me
21:14:26
kpoeck
I am still inlldb, if you need info from any frame?
21:15:33
karlosz
i think i've seen this error before
21:15:45
karlosz
but it went away, after what i assumed was just wiping out all the fasls
21:15:50
karlosz
maybe its more than that
21:16:32
karlosz
it's problematic that i can't reproduce though
21:17:57
karlosz
kpoeck: can you get the info around the frames that have to do with ltvc_read_bignum?
21:27:02
kpoeck
i added this to the gist
21:31:37
karlosz
i don't understand why it looks like we're in a lisp frame
21:32:57
karlosz
oh, i guess it must be because early in the bootstrap the bus error handler hasn't been defined yet
21:35:32
karlosz
it's dying in the allocator
21:35:51
karlosz
which means that at most the length would be corrupted
21:36:57
karlosz
also for whatever reason frame 47 doesn't show the size variable, which is extremely unhelpful
21:42:04
karlosz
oh here we go: (int64_t) length = -9223372036854775808
21:42:51
karlosz
that would certainly give a bus error
21:43:27
karlosz
which file is it loading when this error happens?
21:43:36
karlosz
is it predlib or some other file?
21:44:12
karlosz
Bike: if i go into your clasp directory and build will i be able to get the same error?
21:44:40
Bike
it seems pretty reproducible, sure. let me put it back at the commit... there you go
21:45:09
karlosz
still bizarre that it works for me but not for you
21:45:30
Bike
yeah, dunno what's going on there.
21:45:37
karlosz
i guess i should double check our wscripts are similar
21:46:18
karlosz
are there ny other build configuration things that might give us different build artifacts on the same machine?
21:48:22
karlosz
be back in about an hour to look at this more closely
21:51:16
kpoeck
it loads /Users/karstenpoeck/lisp/compiler/clasp-karsten/build/boehm/fasl/aclasp-boehm-image.fasp
Thursday, 14th of January 2021, 22:06:12 UTC