This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Add JIT and IR tests
ClosedPublic

Authored by jdoerfert on May 15 2023, 3:44 PM.

Details

Summary

The OpenMP target JIT needs testing, so does the IR we actually generate
for the device. This is the initial commit with variations of an "empty
OpenMP kernel" that should all result in a empty IR kernel.

Diff Detail

Event Timeline

jdoerfert created this revision.May 15 2023, 3:44 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 15 2023, 3:44 PM
jdoerfert requested review of this revision.May 15 2023, 3:44 PM
tianshilei1992 added inline comments.May 15 2023, 3:47 PM
openmp/libomptarget/test/jit/empty_kernel_target.c
5

cat doesn't look the right way. Wonder if FileCheck allows the input to be from a file.

jdoerfert added inline comments.May 15 2023, 4:11 PM
openmp/libomptarget/test/jit/empty_kernel_target.c
5

One of many:
llvm/test/Transforms/Util/strip-nonlinetable-debuginfo-loops.ll:; RUN: cat %t | FileCheck %s

tianshilei1992 added inline comments.May 15 2023, 4:16 PM
openmp/libomptarget/test/jit/empty_kernel_target.c
5

Well, that's fair, but since FileCheck supports --input-file, I'd prefer to make it portable, though we only support Linux now.

jdoerfert added inline comments.May 15 2023, 4:37 PM
openmp/libomptarget/test/jit/empty_kernel_target.c
5

will change.

jdoerfert updated this revision to Diff 522396.May 15 2023, 5:56 PM
jdoerfert edited the summary of this revision. (Show Details)

update according to comments, more tests (consolidated)

This revision is now accepted and ready to land.May 17 2023, 11:51 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptMay 18 2023, 1:28 PM