Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add kernel detection flag #1061

Merged
merged 5 commits into from
Jul 20, 2022
Merged

Add kernel detection flag #1061

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
bf7e4b1
Add kernel detection flag
TizianoDeMatteis Jul 19, 2022
0658aff
Fpga instrumentation test: use config flag
TizianoDeMatteis Jul 19, 2022
d17f519
Address review comments
TizianoDeMatteis Jul 20, 2022
4faa12e
Fix wrong config option name
TizianoDeMatteis Jul 20, 2022
112afe8
Merge branch 'master' into fpga_kernel_detection_option
TizianoDeMatteis Jul 20, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 3 additions & 1 deletion tests/fpga/fpga_instrumentation_test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
from dace.transformation.interstate import FPGATransformSDFG, InlineSDFG
import numpy as np
import re
from dace.config import set_temporary
TizianoDeMatteis marked this conversation as resolved.
Show resolved Hide resolved


def make_sdfg(make_tmp_local: bool):
Expand Down Expand Up @@ -114,7 +115,8 @@ def test_instrumentation_single():
@fpga_test()
def test_instrumentation_multiple():
sdfg = make_sdfg(False)
run_program(sdfg)
with set_temporary("compiler", "fpga", "concurrent_kernels_detection", value=True):
TizianoDeMatteis marked this conversation as resolved.
Show resolved Hide resolved
run_program(sdfg)
report = sdfg.get_latest_report()
# There should be five runtimes: One for each kernel, and two for the state
assert len(re.findall(r"[0-9\.]+\s+[0-9\.]+\s+[0-9\.]+\s+[0-9\.]+\s+", str(report))) == 6
Expand Down