Project

General

Profile

[logo] 
 
Home
News
Activity
About/Contact
Major Tools
  Dinotrace
  Verilator
  Verilog-mode
  Verilog-Perl
Other Tools
  BugVise
  CovVise
  Force-Gate-Sim
  Gspice
  IPC::Locker
  Rsvn
  SVN::S4
  Voneline
  WFH
General Info
  Papers

Issue #24

synopsys translate_off and synopsys translate_on

Added by cy wang about 11 years ago. Updated 10 months ago.

Status:
WillNotFix
Priority:
Low
Assignee:
-
Category:
-
% Done:

0%


Description

//synopsys translate_off
     non-synthesizable code
//synopsys translate on

I comment this region by hand.
If verilator can automatically do it, It will reduce manual effort.

History

#1 Updated by Wilson Snyder about 11 years ago

  • Status changed from New to WillNotFix

Absolutely not for religious reasons. translate_offs are horrors and I'm not going to encourage them; you should be using ifdefs instead. See for example my paper http://www.veripool.org/papers/TenIPEdits_SNUGBos07_paper.pdf

#2 Updated by Udi Finkelstein 10 months ago

Wilson Snyder wrote:

Absolutely not for religious reasons. translate_offs are horrors and I'm not going to encourage them; you should be using ifdefs instead. See for example my paper http://www.veripool.org/papers/TenIPEdits_SNUGBos07_paper.pdf

Sorry for reopening a long dead issue,

As much as I understand our reasoning, and agree with, sometimes, when dealing with work issues and some machine-generated vendor code, you cannot manually edit dozens of vendor files to replace '// synthesis translate on' with proper `ifdef.

As I need to support work code For this reason I have started my own branch that adds this feature: https://github.com/udif/verilator/tree/synthesis_off I plan to track the main verilator repository and keep merging new commits from time to time into my modified branch.

I'm leaving this message here so that anyone searching for translate_on/translate_off will hit it.

#3 Updated by Wilson Snyder 10 months ago

Ok. Note also verilog-perl can automatically convert translate comments to ifdefs.

Also available in: Atom