Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

Commit 5ecf10d

Browse files
authored
[SYCL][FPGA][NFC] Change AOT tool requirement for FPGA emulator (#807)
'aoc' tool requires FPGA card and corresponding SDK to compile binaries for FPGA HW target. However, this test suite supports testing on FPGA emulator device only. 'opencl-aot' tool is used as a backend compiler for that target instead of 'aoc'. Remove 'aoc' tool from llvm-test-suite configuration and change AOT compilation tool requirement for tests that should be executed on FPGA emulator device. Signed-off-by: Mikhail Lychkov [email protected]
1 parent 52feaa5 commit 5ecf10d

15 files changed

+35
-27
lines changed

SYCL/AOT/accelerator.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
1-
//==--- accelerator.cpp - AOT compilation for fpga devices using aoc ------==//
1+
//=-- accelerator.cpp - compilation for fpga emulator dev using opencl-aot --=//
22
//
33
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
44
// See https://llvm.org/LICENSE.txt for license information.
55
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
66
//
77
//===---------------------------------------------------------------------===//
88

9-
// REQUIRES: aoc, accelerator
9+
// REQUIRES: opencl-aot, accelerator
1010

1111
// RUN: %clangxx -fsycl -fsycl-targets=spir64_fpga %S/Inputs/aot.cpp -o %t.out
1212
// RUN: %ACC_RUN_PLACEHOLDER %t.out

SYCL/AOT/multiple-devices.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
//
77
//===----------------------------------------------------------------------===//
88

9-
// REQUIRES: opencl-aot, ocloc, aoc, cpu, gpu, accelerator
9+
// REQUIRES: opencl-aot, ocloc, cpu, gpu, accelerator
1010
// UNSUPPORTED: cuda
1111
// CUDA is not compatible with SPIR.
1212

SYCL/Basic/context-with-multiple-devices.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// REQUIRES: accelerator, aoc
1+
// REQUIRES: accelerator, opencl-aot
22

33
// RUN: %clangxx -fsycl -fintelfpga -fsycl-unnamed-lambda %s -o %t2.out
44
// RUN: env CL_CONFIG_CPU_EMULATE_DEVICES=2 %t2.out

SYCL/Basic/fpga_tests/fpga_aocx.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
1-
//==----- fpga_aocx.cpp - AOT compilation for fpga using aoc with aocx -----==//
1+
//==---- fpga_aocx.cpp - AOT compilation for fpga emulator dev with aocx ---==//
22
//
33
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
44
// See https://llvm.org/LICENSE.txt for license information.
55
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
66
//
77
//===----------------------------------------------------------------------===//
88

9-
// REQUIRES: aoc, accelerator
9+
// REQUIRES: opencl-aot, accelerator
1010

1111
/// E2E test for AOCX creation/use/run for FPGA
1212
// Produce an archive with device (AOCX) image. To avoid appending objects to

SYCL/Basic/fpga_tests/fpga_aocx_win.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
1-
//==--- fpga_aocx_win.cpp - AOT compilation for fpga using aoc with aocx ---==//
1+
//==- fpga_aocx_win.cpp - AOT compilation for fpga emulator dev with aocx --==//
22
//
33
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
44
// See https://llvm.org/LICENSE.txt for license information.
55
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
66
//
77
//===----------------------------------------------------------------------===//
88

9-
// REQUIRES: aoc, accelerator
9+
// REQUIRES: opencl-aot, accelerator
1010
// REQUIRES: system-windows
1111

1212
/// E2E test for AOCX creation/use/run for FPGA

SYCL/Basic/fpga_tests/fpga_dsp_control.cpp

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,15 @@
1-
// REQUIRES: accelerator, aoc
2-
// RUN: %clangxx -fsycl -fintelfpga %s -o %t.out
3-
// RUN: %ACC_RUN_PLACEHOLDER %t.out
41
//==---------- fpga_dsp_control.cpp - SYCL FPGA DSP control test -----------==//
52
//
63
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
74
// See https://llvm.org/LICENSE.txt for license information.
85
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
96
//
107
//===----------------------------------------------------------------------===//
8+
9+
// REQUIRES: accelerator, opencl-aot
10+
// RUN: %clangxx -fsycl -fintelfpga %s -o %t.out
11+
// RUN: %ACC_RUN_PLACEHOLDER %t.out
12+
1113
#include <CL/sycl.hpp>
1214
#include <sycl/ext/intel/fpga_extensions.hpp>
1315

SYCL/Basic/fpga_tests/fpga_latency_control_lsu.cpp

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,15 @@
1-
// REQUIRES: aoc, accelerator
2-
// RUN: %clangxx -fsycl -fintelfpga %s -o %t.out
3-
// RUN: %ACC_RUN_PLACEHOLDER %t.out
41
//==- fpga_latency_control_lsu.cpp - SYCL FPGA latency control on LSU test -==//
52
//
63
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
74
// See https://llvm.org/LICENSE.txt for license information.
85
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
96
//
107
//===----------------------------------------------------------------------===//
8+
9+
// REQUIRES: opencl-aot, accelerator
10+
// RUN: %clangxx -fsycl -fintelfpga %s -o %t.out
11+
// RUN: %ACC_RUN_PLACEHOLDER %t.out
12+
1113
#include <CL/sycl.hpp>
1214
#include <sycl/ext/intel/fpga_extensions.hpp>
1315

SYCL/Basic/fpga_tests/fpga_latency_control_pipe.cpp

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,15 @@
1-
// REQUIRES: aoc, accelerator
2-
// RUN: %clangxx -fsycl -fintelfpga %s -o %t.out
3-
// RUN: %ACC_RUN_PLACEHOLDER %t.out
41
//== fpga_latency_control_pipe.cpp - SYCL FPGA latency control on pipe test ==//
52
//
63
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
74
// See https://llvm.org/LICENSE.txt for license information.
85
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
96
//
107
//===----------------------------------------------------------------------===//
8+
9+
// REQUIRES: opencl-aot, accelerator
10+
// RUN: %clangxx -fsycl -fintelfpga %s -o %t.out
11+
// RUN: %ACC_RUN_PLACEHOLDER %t.out
12+
1113
#include <CL/sycl.hpp>
1214
#include <sycl/ext/intel/fpga_extensions.hpp>
1315

SYCL/Basic/fpga_tests/fpga_lsu.cpp

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,14 @@
1-
// REQUIRES: accelerator, aoc
2-
// RUN: %clangxx -fsycl -fintelfpga %s -o %t.out
3-
// RUN: %ACC_RUN_PLACEHOLDER %t.out
41
//==----------------- fpga_lsu.cpp - SYCL FPGA LSU test --------------------==//
52
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
63
// See https://llvm.org/LICENSE.txt for license information.
74
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
85
//
96
//===----------------------------------------------------------------------===//
7+
8+
// REQUIRES: accelerator, opencl-aot
9+
// RUN: %clangxx -fsycl -fintelfpga %s -o %t.out
10+
// RUN: %ACC_RUN_PLACEHOLDER %t.out
11+
1012
#include <CL/sycl.hpp>
1113
#include <sycl/ext/intel/fpga_extensions.hpp>
1214

SYCL/Basic/fpga_tests/global_fpga_device_selector.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// REQUIRES: aoc, accelerator
1+
// REQUIRES: opencl-aot, accelerator
22

33
// RUN: %clangxx -fsycl -fintelfpga %s -o %t.out
44
// RUN: %ACC_RUN_PLACEHOLDER %t.out

SYCL/DeviceCodeSplit/aot-accelerator.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// REQUIRES: aoc, accelerator
1+
// REQUIRES: opencl-aot, accelerator
22

33
// RUN: %clangxx -fsycl -fsycl-device-code-split=per_source -fsycl-targets=spir64_fpga -I %S/Inputs -o %t.out %S/split-per-source-main.cpp %S/Inputs/split-per-source-second-file.cpp \
44
// RUN: -fsycl-dead-args-optimization

SYCL/DeviceLib/complex-fpga.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
1-
//==----- accelerator.cpp - AOT compilation for fpga devices using aoc ----==//
1+
//==----- accelerator.cpp - AOT compilation for fpga emulator devices -----==//
22
//
33
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
44
// See https://llvm.org/LICENSE.txt for license information.
55
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
66
//
77
//===------------------------------------------------------------------------===//
88
// UNSUPPORTED: windows
9-
// REQUIRES: aoc, accelerator
9+
// REQUIRES: opencl-aot, accelerator
1010

1111
// RUN: %clangxx -fsycl -fsycl-targets=spir64_fpga %S/std_complex_math_test.cpp -o %t.out
1212
// RUN: %ACC_RUN_PLACEHOLDER %t.out

SYCL/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ unavailable.
146146
* **cl_options** - CL command line options recognized (or not) by compiler;
147147
* **opencl_icd** - OpenCL ICD loader availability;
148148
* **aot_tool** - Ahead-of-time compilation tools availability;
149-
* **aoc**, **ocloc**, **opencl-aot** - Specific AOT tool availability;
149+
* **ocloc**, **opencl-aot** - Specific AOT tool availability;
150150
* **level_zero_dev_kit** - Level_Zero headers and libraries availability;
151151
* **gpu-intel-dg1** - Intel GPU DG1 availability;
152152
* **gpu-intel-pvc** - Intel GPU PVC availability;

SYCL/SpecConstants/2020/non_native/accelerator.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// REQUIRES: aoc, accelerator
1+
// REQUIRES: opencl-aot, accelerator
22

33
// RUN: %clangxx -fsycl -fsycl-targets=spir64_fpga %S/Inputs/common.cpp -o %t.out \
44
// RUN: -fsycl-dead-args-optimization

SYCL/lit.cfg.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -396,7 +396,7 @@
396396

397397
# Device AOT compilation tools aren't part of the SYCL project,
398398
# so they need to be pre-installed on the machine
399-
aot_tools = ["ocloc", "aoc", "opencl-aot"]
399+
aot_tools = ["ocloc", "opencl-aot"]
400400

401401
for aot_tool in aot_tools:
402402
if find_executable(aot_tool) is not None:

0 commit comments

Comments
 (0)