[Openmp-commits] [PATCH] D42472: [OMPT] Also test ompt_get_task_info() in task_types.c testcase

[Joachim Protze via Phabricator via Openmp-commits]

protze.joachim added inline comments.


================
Comment at: runtime/test/ompt/callback.h:74
+  if(frame)
+    printf("%" PRIu64 ": task level %d: parallel_id=%" PRIu64 ", task_id=%" PRIu64 ", exit_frame=%p, reenter_frame=%p, task_type=%s=%d, thread_num=%d\n", ompt_get_thread_data()->value, level, exists_task ? task_parallel_data->value : 0, exists_task ? task_data->value : 0, frame->exit_frame, frame->enter_frame, buffer, task_type, thread_num);
 }
----------------
Hahnfeld wrote:
> I know we have been doing a poor job in the past, but this line is almost unreadable. Maybe we should start formatting all changes to tests with `clang-format` as well...
The problem with clang-format is, that it breaks the FileCheck patterns by enforcing line breaks for the comments


https://reviews.llvm.org/D42472