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.

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

Fix questa hjson file with needed options #25764

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 23 additions & 5 deletions hw/dv/tools/dvsim/questa.hjson
Original file line number Diff line number Diff line change
Expand Up @@ -2,26 +2,34 @@
// Licensed under the Apache License, Version 2.0, see LICENSE for details.
// SPDX-License-Identifier: Apache-2.0
{
build_cmd: "{job_prefix} {QUESTA_HOME}/questasim/linux_x86_64/qrun -optimize"
run_cmd: "{job_prefix} {QUESTA_HOME}/questasim/linux_x86_64/qrun -simulate"
build_cmd: "{job_prefix} {QUESTA_HOME}/linux_x86_64/qrun -optimize"
run_cmd: "{job_prefix} {QUESTA_HOME}/linux_x86_64/qrun -simulate"


build_opts: [ "-timescale 1ns/1ps",
"-outdir {build_dir}/qrun.out",
"-uvm -uvmhome {QUESTA_HOME}/questasim/verilog_src/uvm-1.2",
"+incdir+{QUESTA_HOME}/verilog_src/uvm-1.2/src",
"+incdir+{QUESTA_HOME}/verilog_src/uvm-1.2/src/mentor",
"{QUESTA_HOME}/verilog_src/uvm-1.2/src/uvm_pkg.sv",
"-uvm -uvmhome {QUESTA_HOME}/verilog_src/uvm-1.2",
"-mfcu",
"-sv",
"-svext=hierbinsof",
rswarbrick marked this conversation as resolved.
Show resolved Hide resolved
"-suppress 2744,2912,7070,7034,7033,2244,7041",
rswarbrick marked this conversation as resolved.
Show resolved Hide resolved
"-f {sv_flist}",
// List multiple tops for the simulation. Prepend each top level with `-top`.
"{eval_cmd} echo {sim_tops} | sed -E 's/(\\S+)/-top \\1/g'",
"-voptargs=\"+acc=nr\""
"-voptargs=\"-access=rw+/. -svext=cbos\""
]

run_opts: [ "-outdir {build_dir}/qrun.out",
"-sv_seed {seed}",
"-c",
"-sv_lib {QUESTA_HOME}/uvm-1.2/linux_x86_64/uvm_dpi",
// dv_macros.svh has a macro printing null using %0d
// format specifier, Questa throws an error on this
// we demote this error in Questa
"-suppress vsim-8323",
"-suppress 8323,7034,7033,7067,8111,16133,16132,7041",
rswarbrick marked this conversation as resolved.
Show resolved Hide resolved
// Questa forces all declared virtual interfaces to be allocated,
// even in classes that are not even created at runtime. The switch
// below demotes the associated error thrown.
Expand Down Expand Up @@ -84,6 +92,10 @@
name: questa_waves
is_sim_mode: 1
}
{
name: questa_waves_off
is_sim_mode: 1
}
// TODO Questa coverage only currently supported in the GUI with no merging
{
name: questa_cov
Expand All @@ -98,6 +110,12 @@
build_opts: []
run_opts: []
}
{
name: questa_gui_debug
is_sim_mode: 1
build_opts: []
run_opts: []
}
{
name: questa_xprop
is_sim_mode: 1
Expand Down
Loading