Skip to content

Commit ea26e9f

Browse files
r-barnesfacebook-github-bot
authored andcommitted
Fix unreachable-break issue in multipy/runtime/loader.cpp +1
Summary: LLVM has a warning `-Wunreachable-code-break` which identifies `break` statements that cannot be reached. These compromise readability, are misleading, and may identify bugs. This diff removes such statements. For questions/comments, contact r-barnes. - If you approve of this diff, please use the "Accept & Ship" button :-) Reviewed By: jasonjk-park Differential Revision: D76476288 fbshipit-source-id: 7c4274aee2e00bf9eedb5a6f30deb142daed55e8
1 parent be69fcd commit ea26e9f

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

multipy/runtime/loader.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1165,10 +1165,9 @@ struct __attribute__((visibility("hidden"))) CustomLibraryImpl
11651165
DEPLOY_ERROR(
11661166
"R_AARCH64_TLSDESC is not supported yet -- recompile with -mtls-dialect=trad. "
11671167
"https://github.com/pytorch/multipy/issues/64");
1168-
} break;
1168+
}
11691169
default:
11701170
DEPLOY_ERROR("unknown reloc type {} in \"{}\"", r_type, name_.c_str());
1171-
break;
11721171
}
11731172
}
11741173

0 commit comments

Comments
 (0)