Skip to content

Commit 2a9a721

Browse files
zacmustintensorflower-gardener
authored andcommitted
Fix clang-tidy errors in pjrt_c_api_helpers.cc.
PiperOrigin-RevId: 782047204
1 parent 8229468 commit 2a9a721

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

third_party/xla/xla/pjrt/c/BUILD

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -306,6 +306,7 @@ cc_library(
306306
"@com_google_absl//absl/container:inlined_vector",
307307
"@com_google_absl//absl/functional:any_invocable",
308308
"@com_google_absl//absl/log",
309+
"@com_google_absl//absl/log:check",
309310
"@com_google_absl//absl/status",
310311
"@com_google_absl//absl/status:statusor",
311312
"@com_google_absl//absl/strings",

third_party/xla/xla/pjrt/c/pjrt_c_api_helpers.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ limitations under the License.
2828
#include "absl/container/flat_hash_map.h"
2929
#include "absl/container/inlined_vector.h"
3030
#include "absl/functional/any_invocable.h"
31+
#include "absl/log/check.h"
3132
#include "absl/log/log.h"
3233
#include "absl/status/status.h"
3334
#include "absl/status/statusor.h"
@@ -72,8 +73,7 @@ PJRT_ClientDeleter MakeClientDeleter(const PJRT_Api* api) {
7273
destroy_args.extension_start = nullptr;
7374
destroy_args.client = client;
7475

75-
PJRT_Error* error = api->PJRT_Client_Destroy(&destroy_args);
76-
CHECK(error == nullptr);
76+
pjrt::LogFatalIfPjrtError(api->PJRT_Client_Destroy(&destroy_args), api);
7777
};
7878
}
7979

0 commit comments

Comments
 (0)