Skip to content

Commit 8e0454d

Browse files
r-barnesfacebook-github-bot
authored andcommitted
Remove unused-variable in multipy/runtime/interpreter/interpreter_impl.cpp +3
Summary: LLVM-15 has a warning `-Wunused-variable` which we treat as an error because it's so often diagnostic of a code issue. Unused variables can compromise readability or, worse, performance. This diff either (a) removes an unused variable and, possibly, it's associated code or (b) qualifies the variable with `[[maybe_unused]]`. #buildsonlynotests - Builds are sufficient - If you approve of this diff, please use the "Accept & Ship" button :-) Reviewed By: wuyuoss Differential Revision: D66143559 fbshipit-source-id: 5c3b7fe512f6fd29481a759d06291d8e930a51ff
1 parent fa01f56 commit 8e0454d

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

multipy/runtime/interpreter/interpreter_impl.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -360,7 +360,6 @@ struct __attribute__((visibility("hidden"))) ConcreteInterpreterObj
360360

361361
torch::deploy::Obj attr(const char* attribute) override {
362362
MULTIPY_SAFE_RETHROW {
363-
bool a = hasattr(attribute);
364363
py::object pyObj = getPyObject().attr(attribute);
365364
std::shared_ptr<ConcreteInterpreterObj> cObj =
366365
std::make_shared<ConcreteInterpreterObj>(pyObj, owningSession_);

0 commit comments

Comments
 (0)