[PATCH] Move Python Init from InitializeForLLGS to Initialize

Tamas Berghammer tberghammer at google.com
Sun Mar 8 05:50:49 PDT 2015


Are you sure, that lldb-server don't use the ScriptInterpreterPython moduul?
If yes then it looks good to me, but I would suggest to add a reference counting to the [Initialize,Terminate]LLGS methods (possibly as a separate patch).

I think the behavior is different with cmake+ninja and with configure+make because one of them compiles with LLDB_DISABLE_PYTHON enabled while the other without it.


http://reviews.llvm.org/D8133

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list