search

openresty / stapxx

Simple macro language extentions to systemtap
698 stars 201 forks source link

support LJ_GC64 #33

Closed luobing2017 closed 7 years ago

luobing2017 commented 7 years ago

for LUAJIT_ENABLE_GC64

agentzh commented 7 years ago

@luobing2017 I think you should put luajit21-gc64 directory under samples/ instead, like samples/luajit21-gc64/*.sxx. These are still samples instead of being direct components of the stap++ compiler.

luobing2017 commented 7 years ago

@agentzh yes, other samples work fine and do not need modify

i will put luajit21-gc64 to samples/, so where can i put the new luajit.sxx? also under samples/luajit21-gc64/ or tapset/luajit21-gc64/?

agentzh commented 7 years ago

@luobing2017 The tapset module should be just renamed to luajit-gc64.sxx and it's fine to be under tapset/.

luobing2017 commented 7 years ago

@agentzh done, and i rename new luajit.sxx to luajit_gc64.sxx, not luajit-gc64.sxx, because @use luajit-gc64 parse error

agentzh commented 7 years ago

@luobing2017 Merged. Thanks!