github.com/moarvm/moarvm | IRC logs at colabti.org/irclogger/irclogger_logs/moarvm
Set by AlexDaniel on 12 June 2018.
Geth MoarVM: MasterDuke17++ created pull request #1287:
Remove commented out profiler code
14:57
Geth MoarVM: 03c9154e8c | MasterDuke17++ (committed using GitHub Web editor) | src/profiler/instrument.c
Remove commented out profiler code (#1287)

No longer used/needed after 269d307a85d1fdfa3752401c228d477d14b27c13 and a487900299788cc95a1bfb9462107d0e4550ecfc.
15:02
linkable6 MOARVM#1287 [closed]: github.com/MoarVM/MoarVM/pull/1287 Remove commented out profiler code
patrickb MasterDuke: Is there anything to look out for when doing a libtommath bump? I'd like to get github.com/MoarVM/MoarVM/issues/1279 fixed. 17:12
sena_kun patrickb, it must support musl. :D 17:14
MasterDuke patrickb: we have two commits on top of v1.2.0 that probably need to be brought along 17:15
i don't think they have a new release, that commit was just to their develop branch 17:16
so we could just cherry-pick it
patrickb MasterDuke: So cherry pick + push to MoarVM/libtommath and then a bump of the submodule should be fine? 17:18
MasterDuke pretty much. fyi, i think we're technically on a branch i created off of the upstream v1.2.0 tag (and then added some commits to), so maybe it makes sense to stay on that branch 18:04
patrickz MasterDuke: I created a PR for the libtommath bump: github.com/MoarVM/libtommath/pull/1 22:18