Skip to content

Commit 2bf66f3

Browse files
committed
memcheck - clearer QF output NaN error msg
1 parent 77d04a1 commit 2bf66f3

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

backends/memcheck/ceed-memcheck-qfunction.c

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,14 +75,16 @@ static int CeedQFunctionApply_Memcheck(CeedQFunction qf, CeedInt Q, CeedVector *
7575
CeedCallBackend(CeedQFunctionGetKernelName(qf, &kernel_name));
7676
CeedCallBackend(CeedQFunctionGetFields(qf, NULL, NULL, NULL, &output_fields));
7777
for (CeedInt i = 0; i < num_out; i++) {
78-
CeedInt field_size;
78+
const char *field_name;
79+
CeedInt field_size;
7980

8081
// Note: need field size because vector may be longer than needed for output
8182
CeedCallBackend(CeedQFunctionFieldGetSize(output_fields[i], &field_size));
83+
CeedCallBackend(CeedQFunctionFieldGetName(output_fields[i], &field_name));
8284
for (CeedSize j = 0; j < field_size * (CeedSize)Q; j++) {
8385
CeedCheck(!isnan(impl->outputs[i][j]), ceed, CEED_ERROR_BACKEND,
84-
"QFunction output %" CeedInt_FMT " entry %" CeedSize_FMT " is NaN after restoring write-only access: %s:%s ", i, j, kernel_path,
85-
kernel_name);
86+
"QFunction output %" CeedInt_FMT " '%s' entry %" CeedSize_FMT " is NaN after restoring write-only access: %s:%s ", i, field_name, j,
87+
kernel_path, kernel_name);
8688
}
8789
CeedCallBackend(CeedVectorRestoreArray(V[i], &impl->outputs[i]));
8890
VALGRIND_DISCARD(output_block_ids[i]);

0 commit comments

Comments
 (0)