samcv | that's a long link | 00:20 | |
01:02
evalable6 joined
01:17
colomon joined
02:56
ilbot3 joined
04:10
evalable6 joined
05:07
evalable6 joined
05:59
evalable6 joined
07:32
evalable6 joined
08:07
domidumont joined
08:12
domidumont joined
|
|||
nine | 2 answers to my question from yesterday so far: we can't annotate the last instruction of the inlinee before merging because #1 we need it's index in the inlines table and more importantly #2 the last instruction can and probably will change as we replace the return statement by a set and goto | 08:48 | |
But maybe we still can. We could also move the end annotation when replacing the last instruction. | 08:54 | ||
A nicer way would be to do all the rewriting before the merging. But I don't know yet if that's even possible or if the rewriting needs information that's available only after the merge. Have to read more code... | 09:00 | ||
Or maybe it's much, much simpler. merge_graph does know exactly the last basic block of the inlinee. We don't change the list of basic blocks of the inlinee. We only change instructions. So the information stays correct. | 09:15 | ||
09:34
AlexDaniel joined
09:52
domidumont joined
10:20
evalable6 joined
11:03
nativecallable6 joined,
quotable6 joined,
releasable6 joined,
committable6 joined,
bloatable6 joined,
evalable6 joined,
benchable6 joined,
unicodable6 joined,
bisectable6 joined,
coverable6 joined,
greppable6 joined,
squashable6 joined,
statisfiable6 joined
|
|||
nine | Oooh...disabling eliminate_pointless_gotos improves things a lot. So that's where the MVM_SPESH_ANN_INLINE_END annotation goes missing | 11:25 | |
11:37
lizmat joined
13:17
evalable6 joined
13:21
domidumont joined
13:40
TimToady joined
13:45
dogbert17 joined
14:28
TimToady joined
|
|||
timotimo | sounds like it needs to consider some more gotos "not pointless" :) | 15:33 | |
15:57
domidumont joined
|
|||
jnthn | Well, or to move the annotation better :) | 15:57 | |
15:59
zakharyas joined
16:04
zakharyas joined
16:39
ggoebel joined
16:57
MasterDuke joined
19:23
evalable6 joined
19:43
unicodable6 joined,
benchable6 joined,
bloatable6 joined,
nativecallable6 joined,
greppable6 joined,
quotable6 joined,
evalable6 joined,
committable6 joined,
coverable6 joined,
bisectable6 joined,
releasable6 joined,
squashable6 joined,
statisfiable6 joined
20:15
MasterDuke joined
20:26
camelia joined
21:26
stmuk joined
21:38
zakharyas joined
23:16
ggoebel joined
|