15:37:03Bikepretty well. machine code disassembly is borked as i mentioned, but we're approaching parity with hir. and karlossz put in the loca call analysis so we're doing better on that front.
15:37:53BikeCoudl you look at kpoeck's pull request? https://github.com/clasp-developers/clasp/pull/1060
15:42:13drmeisterWhat's wrong with clasp_to_fixnum(T_sp thing)? I want something that will raise an error if T_sp is a hash-table or something.
15:43:09Bikeyou can do that with gc::As<Integer_sp>, no?
15:44:47Bikeor rather, the issue might just be that it's declared twice for some reason
15:59:44Bikekarlosz: did you not build clasp after your cleavir changes? we still have things using map-instructions. i can rewrite them, but did you not?
16:14:15karloszBike: i thought i did.... mustve gotten the wrong cleavir commit
16:14:42karloszsorry about that, could you rewrite them?
21:25:40drmeisterIs it better to have unsafe version of ELT and add the bounds checking after? Or a version of ELT for lists and another for vectors - both unsafe?
21:26:10Bikerow-major-aref also does no bounds checking, which is also a problem
21:26:17Bikeso it should probably be at that level, not elt