[BUG REPORT] Edalize does not support the use of vlogdefine parameters with no value

[amal-araweelo]

SystemVerilog allows you to use a macro name without setting it to a value, but this is not supported by Edalize:

Required Data type of the parameter. Valid values are bool, file, int, str. file is similar to str, but the value is treated as a path and converted to an absolute path

Even so, the VCS backend assumes that all vlogdefines do have a value:

...
                for k, v in vlog_defines.items():
                    _args.append(
                        "+define+{}={}".format(
                            k, self._param_value_str(v, str_quote_style='""')
                        )
                    )
...

[olofk]

I have looked at this before. It's possible to solve but it's a bit tricky. Can you work around it by just setting the define to 1 or something similar? That's typically what I do in these cases.

[amal-araweelo]

In the design or just the .core file? It would be nice not to have to change the rtl code to accommodate this bug, if possible. @olofk