Skip to content

Commit bfa7a43

Browse files
committed
fix crash
Signed-off-by: Xiaodong Ye <[email protected]>
1 parent e258f76 commit bfa7a43

File tree

1 file changed

+10
-7
lines changed

1 file changed

+10
-7
lines changed

tests/test-backend-ops.cpp

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -765,7 +765,8 @@ struct test_case {
765765
// pre-graph sentinel
766766
add_sentinel(ctx);
767767

768-
ggml_tensor * out = build_graph(ctx); if (op_name != nullptr && op_desc(out) != op_name) {
768+
ggml_tensor * out = build_graph(ctx);
769+
if (op_name != nullptr && op_desc(out) != op_name) {
769770
//printf(" %s: skipping\n", op_desc(out).c_str());
770771
ggml_free(ctx);
771772
return true;
@@ -789,7 +790,7 @@ struct test_case {
789790
// Create test result for unsupported operation
790791
test_result result;
791792
result.backend_name = ggml_backend_name(backend1);
792-
result.op_name = op_desc(out);
793+
result.op_name = op_name;
793794
result.op_params = vars();
794795
result.test_mode = "test";
795796
result.supported = false;
@@ -911,12 +912,13 @@ struct test_case {
911912

912913
ggml_backend_buffer_free(buf);
913914

914-
ggml_free(ctx); bool test_passed = ud.ok && cmp_ok;
915+
ggml_free(ctx);
915916

916917
// Create test result
918+
bool test_passed = ud.ok && cmp_ok;
917919
test_result result;
918920
result.backend_name = ggml_backend_name(backend1);
919-
result.op_name = op_desc(out);
921+
result.op_name = op_name;
920922
result.op_params = vars();
921923
result.test_mode = "test";
922924
result.supported = supported;
@@ -948,7 +950,8 @@ struct test_case {
948950
ggml_context_ptr ctx(ggml_init(params)); // smart ptr
949951
GGML_ASSERT(ctx);
950952

951-
ggml_tensor * out = build_graph(ctx.get()); if (op_name != nullptr && op_desc(out) != op_name) {
953+
ggml_tensor * out = build_graph(ctx.get());
954+
if (op_name != nullptr && op_desc(out) != op_name) {
952955
//printf(" %s: skipping\n", op_desc(out).c_str());
953956
return true;
954957
}
@@ -958,7 +961,7 @@ struct test_case {
958961
// Create test result for unsupported performance test
959962
test_result result;
960963
result.backend_name = ggml_backend_name(backend);
961-
result.op_name = op_desc(out);
964+
result.op_name = op_name;
962965
result.op_params = vars();
963966
result.test_mode = "perf";
964967
result.supported = false;
@@ -1063,7 +1066,7 @@ struct test_case {
10631066
// Create test result
10641067
test_result result;
10651068
result.backend_name = ggml_backend_name(backend);
1066-
result.op_name = op_desc(out);
1069+
result.op_name = op_name;
10671070
result.op_params = vars();
10681071
result.test_mode = "perf";
10691072
result.supported = true; // If we got this far, it's supported

0 commit comments

Comments
 (0)