python: Remove unused debug APIs
authorAndreas Sandberg <andreas.sandberg@arm.com>
Fri, 4 Sep 2020 16:17:28 +0000 (17:17 +0100)
committerAndreas Sandberg <andreas.sandberg@arm.com>
Mon, 7 Sep 2020 08:42:07 +0000 (08:42 +0000)
The following APIs are not exported from the _m5 namespace and not
used by any of the debug glue code:

 * m5.debug.findFlag
 * m5.debug.setDebugFlag
 * m5.debug.clearDebugFlag
 * m5.debug.dumpDebugFlags

All of them have a clean Python interface where flags are exported
using the m5.debug.flags dictionary. There is also an m5.debug.help
function that lists the available debug flags.

Remove the unused APIs to avoid confusion.

Change-Id: I74738451eb5874f83b135adaccd30a0c6b478996
Signed-off-by: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/34120
Reviewed-by: Jason Lowe-Power <power.jg@gmail.com>
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Jason Lowe-Power <power.jg@gmail.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/python/pybind11/debug.cc

index 69c497c1d5fd9cfa9e91b7d0989ae1129f566a37..84673f1f39752c750214449aa446f848932033a6 100644 (file)
@@ -83,10 +83,6 @@ pybind_init_debug(py::module &m_native)
     m_debug
         .def("getAllFlagsVersion", []() { return Debug::allFlagsVersion; })
         .def("allFlags", &Debug::allFlags, py::return_value_policy::reference)
-        .def("findFlag", &Debug::findFlag)
-        .def("setDebugFlag", &setDebugFlag)
-        .def("clearDebugFlag", &clearDebugFlag)
-        .def("dumpDebugFlags", &dumpDebugFlags)
 
         .def("schedBreak", &schedBreak)
         .def("setRemoteGDBPort", &setRemoteGDBPort)