# HG changeset patch # User Dan Nicolaescu # Date 1246211565 0 # Node ID af77bf73dfe018929dc676b1619589115dcc981f # Parent fe1b52515c0ff31cb8ec3a8c13ccff26e25657d0 * verilog-mode.el (verilog-beg-of-statement) (verilog-endcomment-reason-re): Support unique case and priority case. (verilog-basic-complete-re): Support localparam lineup. (verilog-beg-of-statement-1): Fix for robustness, unique case. (verilog-set-auto-endcomments): Fix for unique case, always_comb commenting. (verilog-leap-to-case-head): Now support *nested* unique & priority case statements. (verilog-auto-lineup): Make just declarations the default (as it had been). (verilog-leap-to-case-head): Support priority/unique case statements. (verilog-auto-lineup): Rework to give users radio buttons to select the various styles of automatic lineup (verilog-error-regexp-alist): Rework to support the XEmacs style of error regular expressions from compilers, lint tools & simulators. Note that GNU Emacs has made it impossible for a mode to load such things. (electric-verilog-terminate-line, verilog-indent-declaration) (verilog-auto-wiure): Rework for radio button selection of auto-lineup selection of specification of auto lineup. (verilog-beg-of-statement-1): Redesign to support proper operation in additional code, based on testing with auto-lineup. (verilog-calculate-indent, assignments & declarations) (verilog-backward-token): Enhance to support auto-lineup of assignments & declarations. (verilog-in-directive-p, verilog-at-struct-p): New function for easy test of whether we are. (verilog-pretty-declarations, verilog-pretty-expr): Massive rework to support safe execution at almost anyline. (verilog-calc-1): Properly support indenting deep inside generate blocks. (verilog-init-font) Remove definition & use of verilog-init-font, as it is redundant with font-lock-defaults. (verilog-mode): Alter the definition of verilog-font-lock-defualts to avoid circular calls if syntax-ppss is a function (as is the case now in 22.x GNU Emacs) as that function would sometimes call itself, leading to (nearly) infinite recursion (verilog-ovm-begin-re, verilog-ovm-end-re) (verilog-ovm-statement-re, verilog-leap-to-head) (verilog-backward-token): Add support for OVM macros. Some are complete statements, and others open and close scopes like begin and end. (verilog-defun-level-not-generate-re, verilog-defun-level-re) (verilog-defun-level-generate-only-re): Really fix the defun-list compilation issue (verilog-calc-1) (verilog-beg-of-statement): Enhance support for coverpoint, constraint and cross statements (verilog-defun-level-list, verilog-generate-defun-level-list) (verilog-all-defun-level-list): Redo these specifications - it is too hard to support eval-when compile aggregation of lists also built at when-compile time. (verilog-defun-level-list): Place defconsts of variables used in building regular expressions which are built in eval-when-compile bodies in the same eval-when-compile body to facilitate compile without load. (verilog-beg-block-re-ordered): Support indenting virtual/protected tasks and functions. (verilog-defun-level-list,verilog-in-generate-region-p) (verilog-backward-ws&directives, verilog-calc-1): Speed up indentation of some module items (generate items). (verilog-forward-sexp, verilog-leap-to-head): Support stepping across virtual/protected tasks and functions. * verilog-mode.el (verilog-auto-arg, verilog-auto-arg-sort): Allow sorting AUTOARG lists. Suggested by Andrea Fedeli. (verilog-read-sub-decls-line): Fix AUTOWIRE signals getting lost in concatenations. Reported by Yishay Belkind. (verilog-auto-ascii-enum): Support one-hot state machines in AUTOASCIIENUM. Suggested by Lloyd Gomez. (verilog-auto-inst, verilog-auto-inst-port): Include interface modport in AUTOINST and add vl-modport for users. Reported by David Rogoff. (verilog-auto-inout-module, verilog-auto-inst) (verilog-decls-get-interfaces, verilog-insert-definition) (verilog-insert-one-definition, verilog-read-decls) (verilog-read-sub-decls, verilog-read-sub-decls-sig) (verilog-sig-modport, verilog-signals-combine-bus) (verilog-subdecls-get-interfaces): Fix expansion of SystemVerilog interfaces in AUTOINOUTMODULE, AUTOINOUTCOMP, and AUTOINST. Suggested by David Rogoff. (verilog-repair-open-comma): Fix non-insertion of comma when `DEFINE occurs in V2K argument list. Reported by Lane Brooks. (verilog-make-width-expression): Simplify [A-1:0] expression widths to just {A{1'b0}}. (verilog-mode): Cleanup checkdoc warnings. (verilog-auto-inout-module, verilog-signals-matching-dir-re): Add third optional regexp to AUTOINOUTMODULE to allow selecting only inputs/outputs or data type. Suggested by Vasu Kandadi. (next-error-last-buffer): Fix byte-compiler warning. (verilog-auto, verilog-auto-insert-lisp, verilog-auto-inst) (verilog-delete-auto): Add AUTOINSERTLISP to insert arbitrary lisp or shell command text during AUTO expansion. Suggested by Tad Truex. (verilog-read-sub-decls-expr, verilog-read-sub-decls-line) (verilog-read-sub-decls-sig, verilog-symbol-detick-text): Fix dotted nets {a.b,c.d} and excaped identifiers being mis-included in AUTOINOUT. Reported by Matthew Lovell. (verilog-read-always-signals-recurse): Fix AUTORESET "if (a<=b)" causing use of <= assignments. Reported by Alex Reed. (verilog-read-decls): Fix triand, trior, wand, wor to be recognized by AUTOWIRE. Reported by Spencer Isaacson. (verilog-extended-complete-re): Support import "DPI-C" functions. (verilog-read-always-signals-recurse): Fix AUTORESET of "x <= y[a+1:a+1]" to not include a in reset list. Reported by Dan Dever. (verilog-insert-date, verilog-insert-year) (verilog-sk-header-tmpl): Fix verilog-header inserting error on Windows systems. Reported by Michael Potts. (verilog-read-module-name): Fix AUTOINST when the child module declaration's name is a tick define. Reported by Elliot Mednick. (verilog-read-decls): Fix V2K parameter bit subscripts getting passed to next parameter's definition. Reported by Bruce T. (verilog-read-decls): Fix detecting "parameter int" when using AUTOINSTPARAM. Reported by Bruce T. (verilog-goto-defun): Fix goto not finding modules unless first perform a verilog-auto expansion. Suggested by Lawrence Butcher. (verilog-mode): Expand -f flag arguments on entry to mode so verilog-goto-defun will work. Reported by Lawrence Butcher. (verilog-getopt): Expand environment variables in -f file arguments. Suggested by Lawrence Butcher. (verilog-set-define): Fix "Symbol's value as variable is void" when reading enumerations. (verilog-auto-ascii-enum): Fix duplicate labels in AUTOASCIIENUM. Suggested by Stephen Peltan. (verilog-read-defines): Fix reading of enumerations in include files. Reported by Steve Peltan. diff -r fe1b52515c0f -r af77bf73dfe0 lisp/ChangeLog --- a/lisp/ChangeLog Sun Jun 28 16:09:46 2009 +0000 +++ b/lisp/ChangeLog Sun Jun 28 17:52:45 2009 +0000 @@ -1,3 +1,136 @@ +2009-06-28 Michael McNamara + + * verilog-mode.el (verilog-beg-of-statement) + (verilog-endcomment-reason-re): Support unique case and priority + case. + (verilog-basic-complete-re): Support localparam lineup. + (verilog-beg-of-statement-1): Fix for robustness, unique case. + (verilog-set-auto-endcomments): Fix for unique case, always_comb + commenting. + (verilog-leap-to-case-head): Now support *nested* unique & + priority case statements. + (verilog-auto-lineup): Make just declarations the default (as it + had been). + (verilog-leap-to-case-head): Support priority/unique case + statements. + (verilog-auto-lineup): Rework to give users radio buttons to + select the various styles of automatic lineup + (verilog-error-regexp-alist): Rework to support the XEmacs style + of error regular expressions from compilers, lint tools & + simulators. Note that GNU Emacs has made it impossible for a mode + to load such things. + (electric-verilog-terminate-line, verilog-indent-declaration) + (verilog-auto-wiure): Rework for radio button selection of + auto-lineup selection of specification of auto lineup. + (verilog-beg-of-statement-1): Redesign to support proper operation + in additional code, based on testing with auto-lineup. + (verilog-calculate-indent, assignments & declarations) + (verilog-backward-token): Enhance to support auto-lineup of + assignments & declarations. + (verilog-in-directive-p, verilog-at-struct-p): New function for + easy test of whether we are. + (verilog-pretty-declarations, verilog-pretty-expr): Massive rework + to support safe execution at almost anyline. + (verilog-calc-1): Properly support indenting deep inside generate + blocks. + (verilog-init-font) Remove definition & use of verilog-init-font, + as it is redundant with font-lock-defaults. + (verilog-mode): Alter the definition of verilog-font-lock-defualts + to avoid circular calls if syntax-ppss is a function (as is the + case now in 22.x GNU Emacs) as that function would sometimes call + itself, leading to (nearly) infinite recursion + (verilog-ovm-begin-re, verilog-ovm-end-re) + (verilog-ovm-statement-re, verilog-leap-to-head) + (verilog-backward-token): Add support for OVM macros. Some are + complete statements, and others open and close scopes like begin + and end. + (verilog-defun-level-not-generate-re, verilog-defun-level-re) + (verilog-defun-level-generate-only-re): Really fix the defun-list + compilation issue + (verilog-calc-1) (verilog-beg-of-statement): Enhance support for + coverpoint, constraint and cross statements + (verilog-defun-level-list, verilog-generate-defun-level-list) + (verilog-all-defun-level-list): Redo these specifications - it is + too hard to support eval-when compile aggregation of lists also + built at when-compile time. + (verilog-defun-level-list): Place defconsts of variables used in + building regular expressions which are built in eval-when-compile + bodies in the same eval-when-compile body to facilitate compile + without load. + (verilog-beg-block-re-ordered): Support indenting + virtual/protected tasks and functions. + (verilog-defun-level-list,verilog-in-generate-region-p) + (verilog-backward-ws&directives, verilog-calc-1): Speed up + indentation of some module items (generate items). + (verilog-forward-sexp, verilog-leap-to-head): Support stepping + across virtual/protected tasks and functions. + +2009-06-28 Wilson Snyder + + * verilog-mode.el (verilog-auto-arg, verilog-auto-arg-sort): Allow + sorting AUTOARG lists. Suggested by Andrea Fedeli. + (verilog-read-sub-decls-line): Fix AUTOWIRE signals getting lost + in concatenations. Reported by Yishay Belkind. + (verilog-auto-ascii-enum): Support one-hot state machines in + AUTOASCIIENUM. Suggested by Lloyd Gomez. + (verilog-auto-inst, verilog-auto-inst-port): Include interface + modport in AUTOINST and add vl-modport for users. Reported by + David Rogoff. + (verilog-auto-inout-module, verilog-auto-inst) + (verilog-decls-get-interfaces, verilog-insert-definition) + (verilog-insert-one-definition, verilog-read-decls) + (verilog-read-sub-decls, verilog-read-sub-decls-sig) + (verilog-sig-modport, verilog-signals-combine-bus) + (verilog-subdecls-get-interfaces): Fix expansion of SystemVerilog + interfaces in AUTOINOUTMODULE, AUTOINOUTCOMP, and AUTOINST. + Suggested by David Rogoff. + (verilog-repair-open-comma): Fix non-insertion of comma when + `DEFINE occurs in V2K argument list. Reported by Lane Brooks. + (verilog-make-width-expression): Simplify [A-1:0] expression + widths to just {A{1'b0}}. + (verilog-mode): Cleanup checkdoc warnings. + (verilog-auto-inout-module, verilog-signals-matching-dir-re): Add + third optional regexp to AUTOINOUTMODULE to allow selecting only + inputs/outputs or data type. Suggested by Vasu Kandadi. + (next-error-last-buffer): Fix byte-compiler warning. + (verilog-auto, verilog-auto-insert-lisp, verilog-auto-inst) + (verilog-delete-auto): Add AUTOINSERTLISP to insert arbitrary lisp + or shell command text during AUTO expansion. Suggested by Tad + Truex. + (verilog-read-sub-decls-expr, verilog-read-sub-decls-line) + (verilog-read-sub-decls-sig, verilog-symbol-detick-text): Fix + dotted nets {a.b,c.d} and excaped identifiers being mis-included + in AUTOINOUT. Reported by Matthew Lovell. + (verilog-read-always-signals-recurse): Fix AUTORESET "if (a<=b)" + causing use of <= assignments. Reported by Alex Reed. + (verilog-read-decls): Fix triand, trior, wand, wor to be + recognized by AUTOWIRE. Reported by Spencer Isaacson. + (verilog-extended-complete-re): Support import "DPI-C" functions. + (verilog-read-always-signals-recurse): Fix AUTORESET of "x <= + y[a+1:a+1]" to not include a in reset list. Reported by Dan + Dever. + (verilog-insert-date, verilog-insert-year) + (verilog-sk-header-tmpl): Fix verilog-header inserting error on + Windows systems. Reported by Michael Potts. + (verilog-read-module-name): Fix AUTOINST when the child module + declaration's name is a tick define. Reported by Elliot Mednick. + (verilog-read-decls): Fix V2K parameter bit subscripts getting + passed to next parameter's definition. Reported by Bruce T. + (verilog-read-decls): Fix detecting "parameter int" when using + AUTOINSTPARAM. Reported by Bruce T. + (verilog-goto-defun): Fix goto not finding modules unless first + perform a verilog-auto expansion. Suggested by Lawrence Butcher. + (verilog-mode): Expand -f flag arguments on entry to mode so + verilog-goto-defun will work. Reported by Lawrence Butcher. + (verilog-getopt): Expand environment variables in -f file + arguments. Suggested by Lawrence Butcher. + (verilog-set-define): Fix "Symbol's value as variable is void" + when reading enumerations. + (verilog-auto-ascii-enum): Fix duplicate labels in AUTOASCIIENUM. + Suggested by Stephen Peltan. + (verilog-read-defines): Fix reading of enumerations in include + files. Reported by Steve Peltan. + 2009-06-28 David De La Harpe Golden * files.el (trash-directory): Fix defcustom type. diff -r fe1b52515c0f -r af77bf73dfe0 lisp/progmodes/verilog-mode.el --- a/lisp/progmodes/verilog-mode.el Sun Jun 28 16:09:46 2009 +0000 +++ b/lisp/progmodes/verilog-mode.el Sun Jun 28 17:52:45 2009 +0000 @@ -54,7 +54,8 @@ ;; Verilog is a rapidly evolving language, and hence this mode is ;; under continuous development. Hence this is beta code, and likely -;; has bugs. Please report any and all bugs to me at mac@verilog.com. +;; has bugs. Please report any issues to the issue tracker at +;; http://www.veripool.org/verilog-mode ;; Please use verilog-submit-bug-report to submit a report; type C-c ;; C-b to invoke this and as a result I will have a much easier time ;; of reproducing the bug you find, and hence fixing it. @@ -102,7 +103,7 @@ ; verilog-auto-endcomments t ; verilog-minimum-comment-distance 40 ; verilog-indent-begin-after-if t -; verilog-auto-lineup '(all) +; verilog-auto-lineup 'declarations ; verilog-highlight-p1800-keywords nil ; verilog-linter "my_lint_shell_command" ; ) @@ -117,9 +118,9 @@ ;;; Code: ;; This variable will always hold the version number of the mode -(defconst verilog-mode-version "436" +(defconst verilog-mode-version "520" "Version of this Verilog mode.") -(defconst verilog-mode-release-date "2008-09-02-GNU" +(defconst verilog-mode-release-date "2009-06-12-GNU" "Release date of this Verilog mode.") (defconst verilog-mode-release-emacs t "If non-nil, this version of Verilog mode was released with Emacs itself.") @@ -265,7 +266,7 @@ (concat "\\<" (verilog-regexp-opt a t) "\\>"))) (defun verilog-easy-menu-filter (menu) - "Filter a easy-menu-define to support new features." + "Filter `easy-menu-define' MENU to support new features." (cond ((not (featurep 'xemacs)) menu) ;; GNU Emacs - passthru ;; Xemacs doesn't support :help. Strip it. @@ -299,10 +300,18 @@ (defun verilog-booleanp (value) "Return t if VALUE is boolean. - This implements GNU Emacs 22.1's `booleanp' function in earlier Emacs. - This function may be removed when Emacs 21 is no longer supported." +This implements GNU Emacs 22.1's `booleanp' function in earlier Emacs. +This function may be removed when Emacs 21 is no longer supported." (or (equal value t) (equal value nil))) +(defun verilog-insert-last-command-event () + "Insert the `last-command-event'." + (insert (if (featurep 'xemacs) + ;; XEmacs 21.5 doesn't like last-command-event + last-command-char + ;; And GNU Emacs 22 has obsoleted last-command-char + last-command-event))) + (defalias 'verilog-syntax-ppss (if (fboundp 'syntax-ppss) 'syntax-ppss (lambda (&optional pos) (parse-partial-sexp (point-min) (or pos (point)))))) @@ -390,6 +399,37 @@ ;; Note we don't use :safe, as that would break on Emacsen before 22.0. (put 'verilog-highlight-translate-off 'safe-local-variable 'verilog-booleanp) +(defcustom verilog-auto-lineup 'declarations + "*Type of statements to lineup across multiple lines. +If 'all' is selected, then all line ups described below are done. + +If 'declaration', then just declarations are lined up with any +preceding declarations, taking into account widths and the like, +so or example the code: + reg [31:0] a; + reg b; +would become + reg [31:0] a; + reg b; + +If 'assignment', then assignments are lined up with any preceding +assignments, so for example the code + a_long_variable <= b + c; + d = e + f; +would become + a_long_variable <= b + c; + d = e + f; + +In order to speed up editing, large blocks of statements are lined up +only when a \\[verilog-pretty-expr] is typed; and large blocks of declarations +are lineup only when \\[verilog-pretty-declarations] is typed." + + :type '(radio (const :tag "Line up Assignments and Declarations" all) + (const :tag "Line up Assignment statements" assignments ) + (const :tag "Line up Declarartions" declarations) + (function :tag "Other")) + :group 'verilog-mode-indent ) + (defcustom verilog-indent-level 3 "*Indentation of Verilog statements with respect to containing block." :group 'verilog-mode-indent @@ -512,47 +552,6 @@ :type 'integer) (put 'verilog-minimum-comment-distance 'safe-local-variable 'integerp) -(defcustom verilog-auto-lineup '(declaration) - "*Algorithm for lining up statements on multiple lines. - -If this list contains the symbol 'all', then all line ups described below -are done. - -If this list contains the symbol 'declaration', then declarations are lined up -with any preceding declarations, taking into account widths and the like, so -for example the code: - reg [31:0] a; - reg b; -would become - reg [31:0] a; - reg b; - -If this list contains the symbol 'assignment', then assignments are lined up -with any preceding assignments, so for example the code - a_long_variable = b + c; - d = e + f; -would become - a_long_variable = b + c; - d = e + f;" - -;; The following is not implemented: -;If this list contains the symbol 'case', then case items are lined up -;with any preceding case items, so for example the code -; case (a) begin -; a_long_state : a = 3; -; b: a = 4; -; endcase -;would become -; case (a) begin -; a_long_state : a = 3; -; b : a = 4; -; endcase -; - - :group 'verilog-mode-indent - :type 'list) -(put 'verilog-auto-lineup 'safe-local-variable 'listp) - (defcustom verilog-highlight-p1800-keywords nil "*True means highlight words newly reserved by IEEE-1800. These will appear in `verilog-font-lock-p1800-face' in order to gently @@ -566,8 +565,8 @@ (defcustom verilog-highlight-grouping-keywords nil "*True means highlight grouping keywords 'begin' and 'end' more dramatically. -If false, these words are in the font-lock-type-face; if True then they are in -`verilog-font-lock-ams-face'. Some find that special highlighting on these +If false, these words are in the `font-lock-type-face'; if True then they are in +`verilog-font-lock-ams-face'. Some find that special highlighting on these grouping constructs allow the structure of the code to be understood at a glance." :group 'verilog-mode-indent :type 'boolean) @@ -630,38 +629,42 @@ (defvar verilog-error-regexp-add-didit nil) (defvar verilog-error-regexp nil) -(setq verilog-error-regexp-add-didit nil - verilog-error-regexp - '( - ; SureLint -;; ("[^\n]*\\[\\([^:]+\\):\\([0-9]+\\)\\]" 1 2) - ; Most SureFire tools - ("\\(WARNING\\|ERROR\\|INFO\\)[^:]*: \\([^,]+\\), \\(line \\|\\)\\([0-9]+\\):" 2 4 ) - ("\ +;;; Compile support +(require 'compile) +(make-variable-buffer-local 'compilation-error-regexp-systems-list) +(defvar compilation-error-regexp-alist) ; in case not +(make-variable-buffer-local 'compilation-error-regexp-alist) + +; List of regexps for Verilog compilers, like verilint. See compilation-error-regexp-alist +; for the formatting. +(defvar verilog-error-regexp-alist + '(verilog +; SureLint + ("[^\n]*\\[\\([^:]+\\):\\([0-9]+\\)\\]" 1 2) + ("\\(WARNING\\|ERROR\\|INFO\\)[^:]*: \\([^,]+\\), \\(line \\|\\)\\([0-9]+\\):" 2 4 ) + ("\ \\([a-zA-Z]?:?[^:( \t\n]+\\)[:(][ \t]*\\([0-9]+\\)\\([) \t]\\|\ :\\([^0-9\n]\\|\\([0-9]+:\\)\\)\\)" 1 2 5) - ; xsim - ; Error! in file /homes/mac/Axis/Xsim/test.v at line 13 [OBJ_NOT_DECLARED] - ("\\(Error\\|Warning\\).*in file (\\([^ \t]+\\) at line *\\([0-9]+\\))" 2 3) - ; vcs - ("\\(Error\\|Warning\\):[^(]*(\\([^ \t]+\\) line *\\([0-9]+\\))" 2 3) - ("Warning:.*(port.*(\\([^ \t]+\\) line \\([0-9]+\\))" 1 2) - ("\\(Error\\|Warning\\):[\n.]*\\([^ \t]+\\) *\\([0-9]+\\):" 2 3) - ("syntax error:.*\n\\([^ \t]+\\) *\\([0-9]+\\):" 1 2) - ; Verilator - ("%?\\(Error\\|Warning\\)\\(-[^:]+\\|\\):[\n ]*\\([^ \t:]+\\):\\([0-9]+\\):" 3 4) - ("%?\\(Error\\|Warning\\)\\(-[^:]+\\|\\):[\n ]*\\([^ \t:]+\\):\\([0-9]+\\):" 3 4) - ; vxl - ("\\(Error\\|Warning\\)!.*\n?.*\"\\([^\"]+\\)\", \\([0-9]+\\)" 2 3) - ("([WE][0-9A-Z]+)[ \t]+\\([^ \t\n,]+\\)[, \t]+\\([0-9]+\\):.*$" 1 2) ; vxl - ("([WE][0-9A-Z]+)[ \t]+\\([^ \t\n,]+\\)[, \t]+line[ \t]+\\([0-9]+\\):.*$" 1 2) - ; nc-verilog - (".*\\*[WE],[0-9A-Z]+ (\\([^ \t,]+\\),\\([0-9]+\\)|" 1 2) - ; Leda - ("In file \\([^ \t]+\\)[ \t]+line[ \t]+\\([0-9]+\\):\n[^\n]*\n[^\n]*\n\\[\\(Warning\\|Error\\|Failure\\)\\][^\n]*" 1 2) - ) -; "*List of regexps for Verilog compilers, like verilint. See compilation-error-regexp-alist for the formatting." -) +; xsim +; Error! in file /homes/mac/Axis/Xsim/test.v at line 13 [OBJ_NOT_DECLARED] + ("\\(Error\\|Warning\\).*in file (\\([^ \t]+\\) at line *\\([0-9]+\\))" 2 3) +; vcs + ("\\(Error\\|Warning\\):[^(]*(\\([^ \t]+\\) line *\\([0-9]+\\))" 2 3) + ("Warning:.*(port.*(\\([^ \t]+\\) line \\([0-9]+\\))" 1 2) + ("\\(Error\\|Warning\\):[\n.]*\\([^ \t]+\\) *\\([0-9]+\\):" 2 3) + ("syntax error:.*\n\\([^ \t]+\\) *\\([0-9]+\\):" 1 2) +; Verilator + ("%?\\(Error\\|Warning\\)\\(-[^:]+\\|\\):[\n ]*\\([^ \t:]+\\):\\([0-9]+\\):" 3 4) + ("%?\\(Error\\|Warning\\)\\(-[^:]+\\|\\):[\n ]*\\([^ \t:]+\\):\\([0-9]+\\):" 3 4) +; verilog-xl + ("\\(Error\\|Warning\\)!.*\n?.*\"\\([^\"]+\\)\", \\([0-9]+\\)" 2 3) + ("([WE][0-9A-Z]+)[ \t]+\\([^ \t\n,]+\\)[, \t]+\\([0-9]+\\):.*$" 1 2) ; vxl + ("([WE][0-9A-Z]+)[ \t]+\\([^ \t\n,]+\\)[, \t]+line[ \t]+\\([0-9]+\\):.*$" 1 2) +; nc-verilog + (".*\\*[WE],[0-9A-Z]+ (\\([^ \t,]+\\),\\([0-9]+\\)|" 1 2) +; Leda + ("In file \\([^ \t]+\\)[ \t]+line[ \t]+\\([0-9]+\\):\n[^\n]*\n[^\n]*\n\\[\\(Warning\\|Error\\|Failure\\)\\][^\n]*" 1 2) + )) (defvar verilog-error-font-lock-keywords '( @@ -834,6 +837,16 @@ :type 'string) (put 'verilog-assignment-delay 'safe-local-variable 'stringp) +(defcustom verilog-auto-arg-sort nil + "*If set, AUTOARG will sort signal names, rather than leave them in +declaration order. Declaration order is advantageous with order based +instantiations and is the default for backward compatibility. Sorted order +reduces changes when declarations are moved around in a file, and it's bad +practice to rely on order based instantiations anyhow." + :group 'verilog-mode-auto + :type 'boolean) +(put 'verilog-auto-arg-sort 'safe-local-variable 'verilog-booleanp) + (defcustom verilog-auto-inst-param-value nil "*If set, AUTOINST will replace parameters with the parameter value. If nil, leave parameters as symbolic names. @@ -1153,6 +1166,8 @@ :help "Help on AUTOINOUTCOMP - copying complemented i/o from another file"] ["AUTOINOUTMODULE" (describe-function 'verilog-auto-inout-module) :help "Help on AUTOINOUTMODULE - copying i/o from another file"] + ["AUTOINSERTLISP" (describe-function 'verilog-auto-insert-lisp) + :help "Help on AUTOINSERTLISP - insert text from a lisp function"] ["AUTOINOUT" (describe-function 'verilog-auto-inout) :help "Help on AUTOINOUT - adding inouts from cells"] ["AUTOINPUT" (describe-function 'verilog-auto-input) @@ -1408,26 +1423,47 @@ "\\b__FILE__\\b" (file-name-nondirectory (buffer-file-name)) t t compile-command)))) -;; Following code only gets called from compilation-mode-hook. -(defvar compilation-error-regexp-alist) - -(defun verilog-error-regexp-add () - "Add the messages to the `compilation-error-regexp-alist'. +;; Following code only gets called from compilation-mode-hook on XEmacs to add error handling. +;; There is no way to add this on the fly to Emacs; instead we must update compile.el +(if (featurep 'xemacs) + (defun verilog-error-regexp-add-xemacs () + "Teach XEmacs about verilog errors. Called by `compilation-mode-hook'. This allows \\[next-error] to find the errors." - (if (not verilog-error-regexp-add-didit) - (progn - (setq verilog-error-regexp-add-didit t) - (setq-default compilation-error-regexp-alist - (append verilog-error-regexp - (default-value 'compilation-error-regexp-alist))) - ;; Could be buffer local at this point; maybe also in let; change all three - (setq compilation-error-regexp-alist - (default-value 'compilation-error-regexp-alist)) - (set (make-local-variable 'compilation-error-regexp-alist) - (default-value 'compilation-error-regexp-alist))))) - -(add-hook 'compilation-mode-hook 'verilog-error-regexp-add) + (interactive) + (if 't ; (not verilog-error-regexp-add-didit) + (progn + (if (or (equal compilation-error-regexp-systems-list 'all) + (not (member 'verilog compilation-error-regexp-systems-list))) + (setq compilation-error-regexp-systems-list + (if (listp compilation-error-regexp-systems-list) + (nconc compilation-error-regexp-systems-list 'verilog) + 'verilog))) + (if (not (assoc 'verilog compilation-error-regexp-alist-alist)) + (setcdr compilation-error-regexp-alist-alist + (cons verilog-error-regexp-alist + (cdr compilation-error-regexp-alist-alist)))) + ;; Need to re-run compilation-error-regexp builder + (compilation-build-compilation-error-regexp-alist)))) + ) +(defun verilog-error-regexp-add-emacs () + "Tell Emacs compile that we are Verilog. +Called by `compilation-mode-hook'. This allows \\[next-error] to +find the errors." + (interactive) +;; Turned off because there seems no way to do this outside of compile.el +;; +;; (if (or (equal compilation-error-regexp-alist 'all) +;; (not (member 'verilog compilation-error-regexp-alist))) +;; (setq compilation-error-regexp-alist +;; (if (listp compilation-error-regexp-alist) +;; (append '(verilog) compilation-error-regexp-alist) +;; '(verilog) ))) + ) + +(if (featurep 'xemacs) + (add-hook 'compilation-mode-hook 'verilog-error-regexp-add-xemacs) + (add-hook 'compilation-mode-hook 'verilog-error-regexp-add-emacs)) (defconst verilog-directive-re ;; "`case" "`default" "`define" "`define" "`else" "`endfor" "`endif" @@ -1436,6 +1472,9 @@ ;; "`time_scale" "`undef" "`while" "\\<`\\(case\\|def\\(ault\\|ine\\(\\)?\\)\\|e\\(lse\\|nd\\(for\\|if\\|protect\\|switch\\|while\\)\\)\\|for\\(mat\\)?\\|i\\(f\\(def\\|ndef\\)?\\|nclude\\)\\|let\\|protect\\|switch\\|time\\(_scale\\|scale\\)\\|undef\\|while\\)\\>") +(defconst verilog-directive-re-1 + (concat "[ \t]*" verilog-directive-re)) + (defconst verilog-directive-begin "\\<`\\(for\\|i\\(f\\|fdef\\|fndef\\)\\|switch\\|while\\)\\>") @@ -1445,14 +1484,138 @@ (defconst verilog-directive-end "`\\(endfor\\|endif\\|endswitch\\|endwhile\\)\\>") -(defconst verilog-directive-re-1 - (concat "[ \t]*" verilog-directive-re)) +(defconst verilog-ovm-begin-re + (eval-when-compile + (verilog-regexp-opt + '( + "`ovm_component_utils_begin" + "`ovm_field_utils_begin" + "`ovm_object_utils_begin" + "`ovm_sequence_utils_begin" + "`ovm_sequencer_utils_begin" + ) nil ))) + +(defconst verilog-ovm-end-re + (eval-when-compile + (verilog-regexp-opt + '( + "`ovm_component_utils_end" + "`ovm_field_utils_end" + "`ovm_object_utils_end" + "`ovm_sequence_utils_end" + "`ovm_sequencer_utils_end" + ) nil ))) + +(defconst verilog-ovm-statement-re + (eval-when-compile + (verilog-regexp-opt + '( + ;; Statements + "`DUT_ERROR" + "`MESSAGE" + "`dut_error" + "`message" + "`ovm_analysis_imp_decl" + "`ovm_blocking_get_imp_decl" + "`ovm_blocking_get_peek_imp_decl" + "`ovm_blocking_master_imp_decl" + "`ovm_blocking_peek_imp_decl" + "`ovm_blocking_put_imp_decl" + "`ovm_blocking_slave_imp_decl" + "`ovm_blocking_transport_imp_decl" + "`ovm_component_registry" + "`ovm_component_registry_param" + "`ovm_component_utils" + "`ovm_create" + "`ovm_create_seq" + "`ovm_declare_sequence_lib" + "`ovm_do" + "`ovm_do_seq" + "`ovm_do_seq_with" + "`ovm_do_with" + "`ovm_error" + "`ovm_fatal" + "`ovm_field_aa_int_byte" + "`ovm_field_aa_int_byte_unsigned" + "`ovm_field_aa_int_int" + "`ovm_field_aa_int_int_unsigned" + "`ovm_field_aa_int_integer" + "`ovm_field_aa_int_integer_unsigned" + "`ovm_field_aa_int_key" + "`ovm_field_aa_int_longint" + "`ovm_field_aa_int_longint_unsigned" + "`ovm_field_aa_int_shortint" + "`ovm_field_aa_int_shortint_unsigned" + "`ovm_field_aa_int_string" + "`ovm_field_aa_object_int" + "`ovm_field_aa_object_string" + "`ovm_field_aa_string_int" + "`ovm_field_aa_string_string" + "`ovm_field_array_int" + "`ovm_field_array_object" + "`ovm_field_array_string" + "`ovm_field_enum" + "`ovm_field_event" + "`ovm_field_int" + "`ovm_field_object" + "`ovm_field_queue_int" + "`ovm_field_queue_object" + "`ovm_field_queue_string" + "`ovm_field_sarray_int" + "`ovm_field_string" + "`ovm_field_utils" + "`ovm_file" + "`ovm_get_imp_decl" + "`ovm_get_peek_imp_decl" + "`ovm_info" + "`ovm_info1" + "`ovm_info2" + "`ovm_info3" + "`ovm_info4" + "`ovm_line" + "`ovm_master_imp_decl" + "`ovm_msg_detail" + "`ovm_non_blocking_transport_imp_decl" + "`ovm_nonblocking_get_imp_decl" + "`ovm_nonblocking_get_peek_imp_decl" + "`ovm_nonblocking_master_imp_decl" + "`ovm_nonblocking_peek_imp_decl" + "`ovm_nonblocking_put_imp_decl" + "`ovm_nonblocking_slave_imp_decl" + "`ovm_object_registry" + "`ovm_object_registry_param" + "`ovm_object_utils" + "`ovm_peek_imp_decl" + "`ovm_phase_func_decl" + "`ovm_phase_task_decl" + "`ovm_print_aa_int_object" + "`ovm_print_aa_string_int" + "`ovm_print_aa_string_object" + "`ovm_print_aa_string_string" + "`ovm_print_array_int" + "`ovm_print_array_object" + "`ovm_print_array_string" + "`ovm_print_object_queue" + "`ovm_print_queue_int" + "`ovm_print_string_queue" + "`ovm_put_imp_decl" + "`ovm_rand_send" + "`ovm_rand_send_with" + "`ovm_send" + "`ovm_sequence_utils" + "`ovm_slave_imp_decl" + "`ovm_transport_imp_decl" + "`ovm_update_sequence_lib" + "`ovm_update_sequence_lib_and_item" + "`ovm_warning" + "`static_dut_error" + "`static_message") nil ))) + ;; ;; Regular expressions used to calculate indent, etc. ;; (defconst verilog-symbol-re "\\<[a-zA-Z_][a-zA-Z_0-9.]*\\>") -(defconst verilog-case-re "\\(\\\\|\\\\)") ;; Want to match ;; aa : ;; aa,bb : @@ -1477,7 +1640,14 @@ "\\(\\\\)\\|" ; 9 "\\(\\\\)\\|" ; 10 "\\(\\\\)\\|" ; 11 - "\\(\\\\)" ; 12 + "\\(\\\\)\\|" ; 12 + ;; OVM + "\\(\\<`ovm_component_utils_end\\>\\)\\|" + "\\(\\<`ovm_field_utils_end\\>\\)\\|" + "\\(\\<`ovm_object_utils_end\\>\\)\\|" + "\\(\\<`ovm_sequence_utils_end\\>\\)\\|" + "\\(\\<`ovm_sequencer_utils_end\\>\\)" + )) (defconst verilog-auto-end-comment-lines-re @@ -1553,28 +1723,35 @@ "endprogram" "endsequence" "endclocking" + ;; OVM + "`ovm_component_utils_end" + "`ovm_field_utils_end" + "`ovm_object_utils_end" + "`ovm_sequence_utils_end" + "`ovm_sequencer_utils_end" + )))) (defconst verilog-endcomment-reason-re ;; Parenthesis indicate type of keyword found (concat - "\\(\\\\)\\|" - "\\(\\\\)\\|" + "\\(\\\\)\\|" ; 1 + "\\(\\\\)\\|" ; 2 + "\\(\\\\s-+\\\\)\\|" ; 3 + "\\(\\\\(\[ \t\]*@\\)?\\)\\|" ; 4 + "\\(\\\\(\[ \t\]*@\\)?\\)\\|" ; 5 + "\\(\\\\(\[ \t\]*@\\)?\\)\\|" ; 6 + "\\(\\\\)\\|" ; 7 + "\\(\\\\(\[ \t\]*@\\)?\\)\\|" "\\(\\\\)\\|" "\\(\\\\)\\|" - "\\(\\\\)\\|" - "\\(\\.*\\\\)\\|" - "\\(\\\\)\\|" - "\\(\\\\)\\|" + "\\(\\\\)\\|" + "\\(\\\\)\\|" "\\(\\\\)\\|" "\\(\\\\)\\|" "\\(\\\\)\\|" "\\(\\\\)\\|" - "\\(\\\\(\[ \t\]*@\\)?\\)\\|" - "\\(\\\\(\[ \t\]*@\\)?\\)\\|" - "\\(\\\\(\[ \t\]*@\\)?\\)\\|" - "\\(\\\\(\[ \t\]*@\\)?\\)\\|" "\\(@\\)\\|" "\\(\\\\)\\|" "\\(\\\\)\\|" @@ -1599,11 +1776,17 @@ "specify" "table" "task" + ;;; OVM + "`ovm_component_utils_begin" + "`ovm_field_utils_begin" + "`ovm_object_utils_begin" + "`ovm_sequence_utils_begin" + "`ovm_sequencer_utils_begin" + )))) ;; These are the same words, in a specific order in the regular ;; expression so that matching will work nicely for ;; verilog-forward-sexp and verilog-calc-indent - (defconst verilog-beg-block-re-ordered ( concat "\\(\\\\)" ;1 "\\|\\(\\\\|\\(\\\\)" ; 2,3 @@ -1612,13 +1795,17 @@ "\\|\\(\\\\)" ;7 "\\|\\(\\\\)" ;8 "\\|\\(\\\\)" ;9 - "\\|\\(\\\\)" ;10 - "\\|\\(\\(\\(\\\\s-+\\)\\|\\(\\\\s-+\\)\\)*\\\\)" ;11 - "\\|\\(\\\\)" ;15 - "\\|\\(\\\\)" ;16 - "\\|\\(\\(\\(\\\\s-+\\)\\|\\(\\\\s-+\\)\\)*\\\\)" ;17 - "\\|\\(\\<\\(rand\\)?sequence\\>\\)" ;21 - "\\|\\(\\\\)" ;22 + "\\|\\(\\(\\(\\\\s-+\\)\\|\\(\\\\s-+\\)\\)*\\\\)" ;10 + "\\|\\(\\\\)" ;14 + "\\|\\(\\(\\(\\\\s-+\\)\\|\\(\\\\s-+\\)\\)*\\\\)" ;15 + "\\|\\(\\\\)" ;18 + "\\|\\(\\\\)" ;16 20 + "\\|\\(\\(\\(\\\\s-+\\)\\|\\(\\\\s-+\\)\\)*\\\\)" ;17 21 + "\\|\\(\\<\\(rand\\)?sequence\\>\\)" ;21 25 + "\\|\\(\\\\)" ;22 27 + "\\|\\(\\<`ovm_[a-z_]+_begin\\>\\)" ;28 + ;; + )) (defconst verilog-end-block-ordered-rry @@ -1731,7 +1918,7 @@ (defconst verilog-behavioral-block-beg-re (eval-when-compile (verilog-regexp-words `("initial" "final" "always" "always_comb" "always_latch" "always_ff" "function" "task")))) - +(defconst verilog-coverpoint-re "\\w+\\s*:\\s*\\(coverpoint\\|cross\\constraint\\)" ) (defconst verilog-indent-re (eval-when-compile (verilog-regexp-words @@ -1774,21 +1961,40 @@ "`switch" "`endswitch" "`timescale" "`time_scale" + ;; OVM Begin tokens + "`ovm_component_utils_begin" + "`ovm_field_utils_begin" + "`ovm_object_utils_begin" + "`ovm_sequence_utils_begin" + "`ovm_sequencer_utils_begin" + ;; OVM End tokens + "`ovm_component_utils_end" + "`ovm_field_utils_end" + "`ovm_object_utils_end" + "`ovm_sequence_utils_end" + "`ovm_sequencer_utils_end" )))) +(defconst verilog-defun-level-not-generate-re + (eval-when-compile + (verilog-regexp-words + `( "module" "macromodule" "primitive" "class" "program" + "interface" "package" "config")))) + (defconst verilog-defun-level-re (eval-when-compile (verilog-regexp-words - `( - "module" "macromodule" "primitive" "class" "program" "initial" "final" "always" "always_comb" - "always_ff" "always_latch" "endtask" "endfunction" "interface" "package" - "config")))) - -(defconst verilog-defun-level-not-generate-re + (append + `( "module" "macromodule" "primitive" "class" "program" + "interface" "package" "config") + `( "initial" "final" "always" "always_comb" "always_ff" + "always_latch" "endtask" "endfunction" ))))) + +(defconst verilog-defun-level-generate-only-re (eval-when-compile (verilog-regexp-words - `( - "module" "macromodule" "primitive" "class" "program" "interface" "package" "config")))) + `( "initial" "final" "always" "always_comb" "always_ff" + "always_latch" "endtask" "endfunction" )))) (defconst verilog-cpp-level-re (eval-when-compile @@ -1801,6 +2007,7 @@ (defconst verilog-extended-complete-re (concat "\\(\\\\|\\\\)" "\\|\\(\\\\s-+\\)*\\(\\\\|\\\\|\\\\)" + "\\|\\(\\\\s-+\\)?\"DPI-C\"\\s-+\\(function\\>\\|task\\>\\)" "\\|" verilog-extended-case-re )) (defconst verilog-basic-complete-re (eval-when-compile @@ -1808,7 +2015,7 @@ `( "always" "assign" "always_latch" "always_ff" "always_comb" "constraint" "import" "initial" "final" "module" "macromodule" "repeat" "randcase" "while" - "if" "for" "forever" "foreach" "else" "parameter" "do" + "if" "for" "forever" "foreach" "else" "parameter" "do" "localparam" )))) (defconst verilog-complete-reg (concat @@ -1821,7 +2028,7 @@ verilog-end-block-re "\\)")) (defconst verilog-endcase-re - (concat verilog-case-re "\\|" + (concat verilog-extended-case-re "\\|" "\\(endcase\\)\\|" verilog-defun-re )) @@ -2260,7 +2467,7 @@ ) ((match-end 4) ;; might be "disable fork" - (if (or + (if (or (looking-at verilog-disable-fork-re) (and (looking-at "fork") (progn @@ -2268,7 +2475,7 @@ (looking-at verilog-disable-fork-re)))) (progn (goto-char (match-end 0)) - (forward-word) + (forward-word 1) (setq reg nil)) (progn ;; Search forward for matching join @@ -2276,7 +2483,7 @@ ((match-end 6) ;; Search forward for matching endclass (setq reg "\\(\\\\)\\|\\(\\\\)" )) - + ((match-end 7) ;; Search forward for matching endtable (setq reg "\\" ) @@ -2289,33 +2496,37 @@ (setq reg "\\" ) (setq nest 'no)) ((match-end 10) - ;; Search forward for matching endtask - (setq reg "\\" ) + ;; Search forward for matching endfunction + (setq reg "\\" ) (setq nest 'no)) - ((match-end 11) + ((match-end 14) ;; Search forward for matching endtask (setq reg "\\" ) (setq nest 'no)) ((match-end 15) + ;; Search forward for matching endtask + (setq reg "\\" ) + (setq nest 'no)) + ((match-end 19) ;; Search forward for matching endgenerate (setq reg "\\(\\\\)\\|\\(\\\\)" )) - ((match-end 16) + ((match-end 20) ;; Search forward for matching endgroup (setq reg "\\(\\\\)\\|\\(\\\\)" )) - ((match-end 17) + ((match-end 21) ;; Search forward for matching endproperty (setq reg "\\(\\\\)\\|\\(\\\\)" )) - ((match-end 18) + ((match-end 25) ;; Search forward for matching endsequence (setq reg "\\(\\<\\(rand\\)?sequence\\>\\)\\|\\(\\\\)" ) (setq md 3)) ; 3 to get to endsequence in the reg above - ((match-end 19) + ((match-end 27) ;; Search forward for matching endclocking (setq reg "\\(\\\\)\\|\\(\\\\)" ))) (if (and reg (forward-word 1)) (catch 'skip - (if (eq nest 'yes) + (if (eq nest 'yes) (let ((depth 1)) (while (verilog-re-search-forward reg nil 'move) (cond @@ -2327,7 +2538,7 @@ (setq depth (1+ depth)))))) (if (verilog-re-search-forward reg nil 'move) (throw 'skip 1)))))) - + ((looking-at (concat "\\(\\<\\(macro\\)?module\\>\\)\\|" "\\(\\\\)\\|" @@ -2362,33 +2573,6 @@ (defun verilog-declaration-beg () (verilog-re-search-backward verilog-declaration-re (bobp) t)) -(defun verilog-font-lock-init () - "Initialize fontification." - ;; highlight keywords and standardized types, attributes, enumeration - ;; values, and subprograms - (setq verilog-font-lock-keywords-3 - (append verilog-font-lock-keywords-2 - (when verilog-highlight-translate-off - (list - ;; Fontify things in translate off regions - '(verilog-match-translate-off - (0 'verilog-font-lock-translate-off-face prepend)))))) - ;; FIXME: This XEmacs setting is redundant with the setting done later - ;; for Emacs (because XEmacs obeys Emacs's setting as well). - (put 'verilog-mode 'font-lock-defaults - '((verilog-font-lock-keywords - verilog-font-lock-keywords-1 - verilog-font-lock-keywords-2 - verilog-font-lock-keywords-3) - nil ; nil means highlight strings & comments as well as keywords - nil ; nil means keywords must match case - nil ; syntax table handled elsewhere - ;; Function to move to beginning of reasonable region to highlight - verilog-beg-of-defun))) - -;; initialize fontification for Verilog Mode -(verilog-font-lock-init) - ;; ;; ;; Mode @@ -2460,7 +2644,7 @@ will be inserted. Setting this variable to zero results in every end acquiring a comment; the default avoids too many redundant comments in tight quarters. - `verilog-auto-lineup' (default `(all)) + `verilog-auto-lineup' (default 'declarations) List of contexts where auto lineup of code should be done. Variables controlling other actions: @@ -2566,13 +2750,13 @@ verilog-font-lock-keywords-2 verilog-font-lock-keywords-3) nil nil nil - ,(if (functionp 'syntax-ppss) - ;; verilog-beg-of-defun uses syntax-ppss, and syntax-ppss uses - ;; font-lock-beginning-of-syntax-function, so - ;; font-lock-beginning-of-syntax-function, can't use + ,(if (functionp 'syntax-ppss) + ;; verilog-beg-of-defun uses syntax-ppss, and syntax-ppss uses + ;; font-lock-beginning-of-syntax-function, so + ;; font-lock-beginning-of-syntax-function, can't use ;; verilog-beg-of-defun. - nil - 'verilog-beg-of-defun))) + nil + 'verilog-beg-of-defun))) ;;------------------------------------------------------------ ;; now hook in 'verilog-colorize-include-files (eldo-mode.el&spice-mode.el) ;; all buffer local: @@ -2600,8 +2784,6 @@ ;; Stuff for autos (add-hook 'write-contents-hooks 'verilog-auto-save-check) ; already local -;; (verilog-auto-reeval-locals t) ; Save locals in case user changes them -;; (verilog-getopt-flags) (run-hooks 'verilog-mode-hook)) @@ -2651,9 +2833,9 @@ 't))) ;; see if we should line up assignments (progn - (if (or (memq 'all verilog-auto-lineup) - (memq 'assignments verilog-auto-lineup)) - (verilog-pretty-expr)) + (if (or (eq 'all verilog-auto-lineup) + (eq 'assignments verilog-auto-lineup)) + (verilog-pretty-expr t "\\(<\\|:\\)?=" )) (newline)) (forward-line 1)) ;; Indent next line @@ -2670,7 +2852,7 @@ (defun electric-verilog-semi () "Insert `;' character and reindent the line." (interactive) - (insert last-command-event) + (verilog-insert-last-command-event) (if (or (verilog-in-comment-or-string-p) (verilog-in-escaped-name-p)) @@ -2695,7 +2877,7 @@ (defun electric-verilog-colon () "Insert `:' and do all indentations except line indent on this line." (interactive) - (insert last-command-event) + (verilog-insert-last-command-event) ;; Do nothing if within string. (if (or (verilog-within-string) @@ -2714,7 +2896,7 @@ ;;(defun electric-verilog-equal () ;; "Insert `=', and do indentation if within block." ;; (interactive) -;; (insert last-command-event) +;; (verilog-insert-last-command-event) ;; Could auto line up expressions, but not yet ;; (if (eq (car (verilog-calculate-indent)) 'block) ;; (let ((verilog-tab-always-indent nil)) @@ -2724,12 +2906,10 @@ (defun electric-verilog-tick () "Insert back-tick, and indent to column 0 if this is a CPP directive." (interactive) - (insert last-command-event) + (verilog-insert-last-command-event) (save-excursion - (if (progn - (beginning-of-line) - (looking-at verilog-directive-re-1)) - (verilog-indent-line)))) + (if (verilog-in-directive-p) + (verilog-indent-line)))) (defun electric-verilog-tab () "Function called when TAB is pressed in Verilog mode." @@ -2767,7 +2947,7 @@ (skip-chars-backward " \t") (kill-region (point) oldpnt))))))) (t (progn (insert "\t"))))) - + ;; @@ -3025,6 +3205,7 @@ (not (or (bolp) (= (preceding-char) ?\;) + (looking-at "\\w+\\W*:\\W*\\(coverpoint\\|cross\\|constraint\\)") (not (or (looking-at "\\<") (forward-word -1))) @@ -3047,55 +3228,85 @@ (defun verilog-beg-of-statement-1 () "Move backward to beginning of statement." (interactive) + (if (verilog-in-comment-p) + (verilog-backward-syntactic-ws)) (let ((pt (point))) - - (while (and (not (looking-at verilog-complete-reg)) - (setq pt (point)) - (verilog-backward-token) - (not (looking-at verilog-complete-reg)) - (verilog-backward-syntactic-ws) - (setq pt (point)) - (not (bolp)) - (not (= (preceding-char) ?\;)))) - (goto-char pt) - (verilog-forward-ws&directives))) + (catch 'done + (while (not (looking-at verilog-complete-reg)) + (setq pt (point)) + (verilog-backward-syntactic-ws) + (if (or (bolp) + (= (preceding-char) ?\;)) + (progn + (goto-char pt) + (throw 'done t)) + (verilog-backward-token)))) + (verilog-forward-syntactic-ws))) +; +; (while (and +; (not (looking-at verilog-complete-reg)) +; (not (bolp)) +; (not (= (preceding-char) ?\;))) +; (verilog-backward-token) +; (verilog-backward-syntactic-ws) +; (setq pt (point))) +; (goto-char pt) +; ;(verilog-forward-syntactic-ws) (defun verilog-end-of-statement () "Move forward to end of current statement." (interactive) (let ((nest 0) pos) - (or (looking-at verilog-beg-block-re) - ;; Skip to end of statement - (setq pos (catch 'found - (while t - (forward-sexp 1) - (verilog-skip-forward-comment-or-string) - (cond ((looking-at "[ \t]*;") - (skip-chars-forward "^;") - (forward-char 1) - (throw 'found (point))) - ((save-excursion - (forward-sexp -1) - (looking-at verilog-beg-block-re)) - (goto-char (match-beginning 0)) - (throw 'found nil)) - ((looking-at "[ \t]*)") - (throw 'found (point))) - ((eobp) - (throw 'found (point)))))))) - (if (not pos) - ;; Skip a whole block - (catch 'found - (while t - (verilog-re-search-forward verilog-end-statement-re nil 'move) - (setq nest (if (match-end 1) - (1+ nest) - (1- nest))) - (cond ((eobp) - (throw 'found (point))) - ((= 0 nest) - (throw 'found (verilog-end-of-statement)))))) - pos))) + (cond + ((verilog-in-directive-p) + (forward-line 1) + (backward-char 1)) + + ((looking-at verilog-beg-block-re) + (verilog-forward-sexp)) + + ((equal (char-after) ?\}) + (forward-char)) + + ;; Skip to end of statement + ((condition-case nil + (setq pos + (catch 'found + (while t + (forward-sexp 1) + (verilog-skip-forward-comment-or-string) + (if (eolp) + (forward-line 1)) + (cond ((looking-at "[ \t]*;") + (skip-chars-forward "^;") + (forward-char 1) + (throw 'found (point))) + ((save-excursion + (forward-sexp -1) + (looking-at verilog-beg-block-re)) + (goto-char (match-beginning 0)) + (throw 'found nil)) + ((looking-at "[ \t]*)") + (throw 'found (point))) + ((eobp) + (throw 'found (point))) + ))) + + ) + (error nil)) + (if (not pos) + ;; Skip a whole block + (catch 'found + (while t + (verilog-re-search-forward verilog-end-statement-re nil 'move) + (setq nest (if (match-end 1) + (1+ nest) + (1- nest))) + (cond ((eobp) + (throw 'found (point))) + ((= 0 nest) + (throw 'found (verilog-end-of-statement)))))) + pos))))) (defun verilog-in-case-region-p () "Return true if in a case region. @@ -3140,17 +3351,20 @@ "Return true if in a generate region. More specifically, after a generate and before an endgenerate." (interactive) - (let ((lim (save-excursion (verilog-beg-of-defun) (point))) - (nest 1)) + (let ((nest 1)) (save-excursion - (while (and - (/= nest 0) - (verilog-re-search-backward "\\<\\(generate\\)\\|\\(endgenerate\\)\\>" lim 'move) - (cond - ((match-end 1) ; generate - (setq nest (1- nest))) - ((match-end 2) ; endgenerate - (setq nest (1+ nest))))))) + (catch 'done + (while (and + (/= nest 0) + (verilog-re-search-backward + "\\<\\(module\\)\\|\\(generate\\)\\|\\(endgenerate\\)\\>" nil 'move) + (cond + ((match-end 1) ; module - we have crawled out + (throw 'done 1)) + ((match-end 2) ; generate + (setq nest (1- nest))) + ((match-end 3) ; endgenerate + (setq nest (1+ nest)))))))) (= nest 0) )) ; return nest (defun verilog-in-fork-region-p () @@ -3337,18 +3551,11 @@ ((looking-at "\\") (setq str "randcase") (setq err nil)) - ((match-end 0) - (goto-char (match-end 1)) - (if nil - (let (s f) - (setq s (match-beginning 1)) - (setq f (progn (end-of-line) - (point))) - (setq str (buffer-substring s f))) - (setq err nil)) - (setq str (concat (buffer-substring (match-beginning 1) (match-end 1)) - " " - (verilog-get-expr)))))) + ((looking-at "\\(\\(unique\\s-+\\|priority\\s-+\\)?case[xz]?\\)") + (goto-char (match-end 0)) + (setq str (concat (match-string 0) " " (verilog-get-expr))) + (setq err nil)) + )) (end-of-line) (if kill-existing-comment (verilog-kill-existing-comment)) @@ -3405,11 +3612,10 @@ (setq str "")) ((looking-at verilog-endcomment-reason-re) (setq there (match-end 0)) - (setq cntx (concat - (buffer-substring (match-beginning 0) (match-end 0)) " ")) + (setq cntx (concat (match-string 0) " ")) (cond (;- begin - (match-end 2) + (match-end 1) (setq err nil) (save-excursion (if (and (verilog-continued-line) @@ -3418,13 +3624,11 @@ (goto-char (match-end 0)) (setq there (point)) (setq str - (concat " // " - (buffer-substring (match-beginning 0) (match-end 0)) " " - (verilog-get-expr)))) + (concat " // " (match-string 0) " " (verilog-get-expr)))) (setq str "")))) (;- else - (match-end 4) + (match-end 2) (let ((nest 0) ( reg "\\(\\\\)\\|\\(\\\\)\\|\\(\\\\)")) (catch 'skip @@ -3445,7 +3649,7 @@ (throw 'skip 1))))))))) (;- end else - (match-end 5) + (match-end 3) (goto-char there) (let ((nest 0) (reg "\\(\\\\)\\|\\(\\\\)\\|\\(\\\\)")) @@ -3465,6 +3669,12 @@ (setq str (verilog-get-expr)) (setq str (concat " // else: !if" str )) (throw 'skip 1))))))))) + (; always_comb, always_ff, always_latch + (or (match-end 4) (match-end 5) (match-end 6)) + (goto-char (match-end 0)) + (setq there (point)) + (setq err nil) + (setq str (concat " // " cntx ))) (;- task/function/initial et cetera t @@ -3472,8 +3682,7 @@ (goto-char (match-end 0)) (setq there (point)) (setq err nil) - (setq str (verilog-get-expr)) - (setq str (concat " // " cntx str ))) + (setq str (concat " // " cntx (verilog-get-expr)))) (;-- otherwise... (setq str " // auto-endcomment confused ")))) @@ -3782,6 +3991,7 @@ (t (error "Linter name not set"))))) (defvar compilation-last-buffer) +(defvar next-error-last-buffer) (defun verilog-surelint-off () "Convert a SureLint warning line into a disable statement. @@ -4010,17 +4220,23 @@ (throw 'nesting 'comment)) ;; if we have a directive, done. - (if (save-excursion (beginning-of-line) (looking-at verilog-directive-re-1)) + (if (save-excursion (beginning-of-line) + (and (looking-at verilog-directive-re-1) + (not (looking-at "[ \t]*`ovm_")))) (throw 'nesting 'directive)) + ;; indent structs as if there were module level + (if (verilog-in-struct-p) + (throw 'nesting 'block)) ;; unless we are in the newfangled coverpoint or constraint blocks ;; if we are in a parenthesized list, and the user likes to indent these, return. (if (and - verilog-indent-lists - (not (verilog-in-coverage)) - (verilog-in-paren)) + verilog-indent-lists + (verilog-in-paren) + (not (verilog-in-coverage-p)) + ) (progn (setq par 1) - (throw 'nesting 'block))) + (throw 'nesting 'block))) ;; See if we are continuing a previous line (while t @@ -4143,7 +4359,8 @@ (setq type 'cpp)) (if (and verilog-indent-lists - (not (verilog-in-coverage)) + (not(or (verilog-in-coverage-p) + (verilog-in-struct-p))) (verilog-in-paren)) (setq depth 1) (cond @@ -4159,38 +4376,44 @@ (defun verilog-calc-1 () (catch 'nesting - (while (verilog-re-search-backward (concat "\\({\\|}\\|" verilog-indent-re "\\)") nil 'move) - (cond - ((equal (char-after) ?\{) - (if (verilog-at-constraint-p) - (throw 'nesting 'block))) - ((equal (char-after) ?\}) - - (let ((there (verilog-at-close-constraint-p))) - (if there (goto-char there)))) - - ((looking-at verilog-beg-block-re-ordered) - (cond - ((match-end 2) ; *sigh* could be "unique case" or "priority casex" - (let ((here (point))) - (verilog-beg-of-statement) - (if (looking-at verilog-extended-case-re) - (throw 'nesting 'case) - (goto-char here))) - (throw 'nesting 'case)) - - ((match-end 4) ; *sigh* could be "disable fork" - (let ((here (point))) - (verilog-beg-of-statement) - (if (looking-at verilog-disable-fork-re) - t ; is disable fork, this is a normal statement - (progn ; or is fork, starts a new block - (goto-char here) - (throw 'nesting 'block))))) - - - ;; need to consider typedef struct here... - ((looking-at "\\") + (let ((re (concat "\\({\\|}\\|" verilog-indent-re "\\)"))) + (while (verilog-re-search-backward re nil 'move) + (catch 'continue + (cond + ((equal (char-after) ?\{) + (if (verilog-at-constraint-p) + (throw 'nesting 'block))) + + ((equal (char-after) ?\}) + (let ((there (verilog-at-close-constraint-p))) + (if there ;; we are at the } that closes a constraing. Find the { that opens it + (progn + (forward-char 1) + (backward-list 1) + (verilog-beg-of-statement))))) + + ((looking-at verilog-beg-block-re-ordered) + (cond + ((match-end 2) ; *sigh* could be "unique case" or "priority casex" + (let ((here (point))) + (verilog-beg-of-statement) + (if (looking-at verilog-extended-case-re) + (throw 'nesting 'case) + (goto-char here))) + (throw 'nesting 'case)) + + ((match-end 4) ; *sigh* could be "disable fork" + (let ((here (point))) + (verilog-beg-of-statement) + (if (looking-at verilog-disable-fork-re) + t ; is disable fork, this is a normal statement + (progn ; or is fork, starts a new block + (goto-char here) + (throw 'nesting 'block))))) + + + ;; need to consider typedef struct here... + ((looking-at "\\") ; *sigh* These words have an optional prefix: ; extern {virtual|protected}? function a(); ; typedef class foo; @@ -4199,43 +4422,46 @@ ; property ; ... ; endfunction - (verilog-beg-of-statement) - (if (looking-at verilog-beg-block-re-ordered) - (throw 'nesting 'block) - (throw 'nesting 'defun))) - - ((looking-at "\\") - ; *sigh* + (verilog-beg-of-statement) + (if (looking-at verilog-beg-block-re-ordered) + (throw 'nesting 'block) + (throw 'nesting 'defun))) + + ((looking-at "\\") + ; *sigh* ; {assert|assume|cover} property (); are complete ; but ; property ID () ... needs end_property - (verilog-beg-of-statement) - (if (looking-at "\\(assert\\|assume\\|cover\\)\\s-+property\\>") - (throw 'nesting 'statement) ; We don't need an endproperty for these - (throw 'nesting 'block) ;We still need a endproperty - )) - - (t (throw 'nesting 'block)))) - - ((looking-at verilog-end-block-re) - (verilog-leap-to-head) - (if (verilog-in-case-region-p) - (progn - (verilog-leap-to-case-head) - (if (looking-at verilog-case-re) - (throw 'nesting 'case))))) - - ((looking-at (if (verilog-in-generate-region-p) - verilog-defun-level-not-generate-re - verilog-defun-level-re)) - (throw 'nesting 'defun)) - - ((looking-at verilog-cpp-level-re) - (throw 'nesting 'cpp)) - - ((bobp) - (throw 'nesting 'cpp)))) - (throw 'nesting 'cpp))) + (verilog-beg-of-statement) + (if (looking-at "\\(assert\\|assume\\|cover\\)\\s-+property\\>") + (throw 'nesting 'statement) ; We don't need an endproperty for these + (throw 'nesting 'block) ;We still need a endproperty + )) + + (t (throw 'nesting 'block)))) + + ((looking-at verilog-end-block-re) + (verilog-leap-to-head) + (if (verilog-in-case-region-p) + (progn + (verilog-leap-to-case-head) + (if (looking-at verilog-extended-case-re) + (throw 'nesting 'case))))) + + ((looking-at verilog-defun-level-re) + (if (looking-at verilog-defun-level-generate-only-re) + (if (verilog-in-generate-region-p) + (throw 'continue 'foo) ; always block in a generate + (throw 'nesting 'defun)) + (throw 'nesting 'defun))) + + ((looking-at verilog-cpp-level-re) + (throw 'nesting 'cpp)) + + ((bobp) + (throw 'nesting 'cpp))))) + + (throw 'nesting 'cpp)))) (defun verilog-calculate-indent-directive () "Return indentation level for directive. @@ -4271,11 +4497,19 @@ (defun verilog-leap-to-case-head () (let ((nest 1)) (while (/= 0 nest) - (verilog-re-search-backward "\\(\\\\|\\[^:]\\)\\|\\(\\\\)" nil 'move) + (verilog-re-search-backward + (concat + "\\(\\\\|\\(\\\\)" + "\\|\\(\\\\)" ) + nil 'move) (cond ((match-end 1) + (let ((here (point))) + (verilog-beg-of-statement) + (unless (looking-at verilog-extended-case-re) + (goto-char here))) (setq nest (1- nest))) - ((match-end 2) + ((match-end 3) (setq nest (1+ nest))) ((bobp) (ding 't) @@ -4294,27 +4528,32 @@ (setq reg (concat "\\(\\\\)\\|\\(\\\\)\\|" "\\(\\\\)\\|\\(\\\\)" ))) ((looking-at "\\") - ;; 9: Search back for matching task + ;; 2: Search back for matching task (setq reg "\\(\\\\)\\|\\(\\(\\(\\\\s-+\\)\\|\\(\\\\s-+\\)\\)+\\\\)") (setq nesting 'no)) ((looking-at "\\") - ;; 2: Search back for matching case - (setq reg "\\(\\\\|\\\\)\\|\\(\\\\)" )) + (catch 'nesting + (verilog-leap-to-case-head) ) + (setq reg nil) ; to force skip + ) + ((looking-at "\\") - ;; 3: Search back for matching fork + ;; 4: Search back for matching fork (setq reg "\\(\\\\)\\|\\(\\\\)" )) ((looking-at "\\") - ;; 4: Search back for matching class + ;; 5: Search back for matching class (setq reg "\\(\\\\)\\|\\(\\\\)" )) ((looking-at "\\") - ;; 5: Search back for matching table + ;; 6: Search back for matching table (setq reg "\\(\\\\)\\|\\(\\\\)" )) ((looking-at "\\") - ;; 6: Search back for matching specify + ;; 7: Search back for matching specify (setq reg "\\(\\\\)\\|\\(\\\\)" )) ((looking-at "\\") - ;; 7: Search back for matching function - (setq reg "\\(\\\\)\\|\\(\\\\)" )) + ;; 8: Search back for matching function + (setq reg "\\(\\\\)\\|\\(\\(\\(\\\\s-+\\)\\|\\(\\\\s-+\\)\\)+\\\\)") + (setq nesting 'no)) + ;;(setq reg "\\(\\\\)\\|\\(\\\\)" )) ((looking-at "\\") ;; 8: Search back for matching generate (setq reg "\\(\\\\)\\|\\(\\\\)" )) @@ -4324,6 +4563,9 @@ ((looking-at "\\") ;; 11: Search back for matching property (setq reg "\\(\\\\)\\|\\(\\\\)" )) + ((looking-at verilog-ovm-end-re) + ;; 12: Search back for matching sequence + (setq reg (concat "\\(" verilog-ovm-begin-re "\\|" verilog-ovm-end-re "\\)"))) ((looking-at "\\") ;; 12: Search back for matching interface (setq reg "\\(\\\\)\\|\\(\\\\)" )) @@ -4364,7 +4606,7 @@ ))) ;no nesting (if (and - (verilog-re-search-backward reg nil 'move) + (verilog-re-search-backward reg nil 'move) (match-end 1)) ; task -> could be virtual and/or protected (progn (verilog-beg-of-statement) @@ -4390,7 +4632,7 @@ continued)) (defun verilog-backward-token () - "Step backward token, returning true if we are now at an end of line token." + "Step backward token, returing true if nil if continued line." (interactive) (verilog-backward-syntactic-ws) (cond @@ -4404,13 +4646,25 @@ (= (preceding-char) ?\}) (progn (backward-char) - (verilog-at-close-constraint-p))) + (not(verilog-at-close-constraint-p)))) (;-- constraint foo { a = b } ; is a complete statement. *sigh* (= (preceding-char) ?\{) (progn (backward-char) (not (verilog-at-constraint-p)))) + (;" string " + (= (preceding-char) ?\") + (backward-char) + (verilog-skip-backward-comment-or-string) + nil) + + (; [3:4] + (= (preceding-char) ?\]) + (backward-char) + (verilog-backward-open-bracket) + t) + (;-- Could be 'case (foo)' or 'always @(bar)' which is complete ; also could be simply '@(foo)' ; or foo u1 #(a=8) @@ -4424,8 +4678,15 @@ (let ((back (point))) (forward-word -1) (cond + ;;XX ((looking-at "\\<\\(always\\(_latch\\|_ff\\|_comb\\)?\\|case\\(\\|[xz]\\)\\|for\\(\\|each\\|ever\\)\\|i\\(f\\|nitial\\)\\|repeat\\|while\\)\\>") (not (looking-at "\\\\|\\[^:]"))) + ((looking-at verilog-ovm-statement-re) + nil) + ((looking-at verilog-ovm-begin-re) + t) + ((looking-at verilog-ovm-end-re) + t) (t (goto-char back) (cond @@ -4521,7 +4782,8 @@ (save-excursion (beginning-of-line) (cond - ((verilog-within-translate-off) + ((and verilog-highlight-translate-off + (verilog-within-translate-off)) (verilog-back-to-start-translate-off (point-min))) ((looking-at verilog-directive-re-1) (point)) @@ -4588,13 +4850,29 @@ (if (equal (char-after (point) ) ?\\ ) t nil))) +(defun verilog-in-directive-p () + "Return true if in a star or // comment." + (save-excursion + (beginning-of-line) + (looking-at verilog-directive-re-1))) (defun verilog-in-paren () "Return true if in a parenthetical expression." (let ((state (save-excursion (verilog-syntax-ppss)))) (> (nth 0 state) 0 ))) -(defun verilog-in-coverage () +(defun verilog-in-struct-p () + "Return true if in a struct declaration." + (interactive) + (save-excursion + (if (verilog-in-paren) + (progn + (backward-up-list 1) + (verilog-at-struct-p) + ) + nil))) + +(defun verilog-in-coverage-p () "Return true if in a constraint or coverpoint expression." (interactive) (save-excursion @@ -4630,6 +4908,14 @@ ;; not nil)) +(defun verilog-at-struct-p () + "If at the { of a struct, return true, moving point to struct." + (save-excursion + (if (and (equal (char-after) ?\{) + (verilog-backward-token)) + (looking-at "\\") + nil))) + (defun verilog-parenthesis-depth () "Return non zero if in parenthetical-expression." (save-excursion (nth 1 (verilog-syntax-ppss)))) @@ -4806,7 +5092,7 @@ (current-column)))) (indent-line-to val) )) - + (;-- Handle the ends (or (looking-at verilog-end-block-re ) @@ -4869,7 +5155,7 @@ (cond ((looking-at verilog-named-block-re) (current-column)) - ((and (not (looking-at verilog-case-re)) + ((and (not (looking-at verilog-extended-case-re)) (looking-at "^[^:;]+[ \t]*:")) (verilog-re-search-forward ":" nil t) (skip-chars-forward " \t") @@ -4937,101 +5223,111 @@ ;; (defun verilog-pretty-declarations (&optional quiet) - "Line up declarations around point." + "Line up declarations around point. +Be verbose about progress unless optional QUIET set." (interactive) (save-excursion (if (progn - (verilog-beg-of-statement-1) - (looking-at verilog-declaration-re)) - (let* ((m1 (make-marker)) - (e) (r) - (here (point)) - ;; Start of declaration range - (start - (progn - (verilog-beg-of-statement-1) - (while (looking-at verilog-declaration-re) - (beginning-of-line) - (setq e (point)) - (verilog-backward-syntactic-ws) - (backward-char) - (verilog-beg-of-statement-1)) ;Ack, need to grok `define - e)) - ;; End of declaration range - (end - (progn - (goto-char here) - (verilog-end-of-statement) - (setq e (point)) ;Might be on last line - (verilog-forward-syntactic-ws) - (while (looking-at verilog-declaration-re) - (beginning-of-line) - (verilog-end-of-statement) - (setq e (point)) - (verilog-forward-syntactic-ws)) - e)) - (edpos (set-marker (make-marker) end)) - (ind) - (base-ind - (progn - (goto-char start) - (verilog-do-indent (verilog-calculate-indent)) - (verilog-forward-ws&directives) - (current-column)))) - (goto-char end) - (goto-char start) - (if (and (not quiet) - (> (- end start) 100)) - (message "Lining up declarations..(please stand by)")) - ;; Get the beginning of line indent first - (while (progn (setq e (marker-position edpos)) - (< (point) e)) - (cond - ( (save-excursion (skip-chars-backward " \t") - (bolp)) - (verilog-forward-ws&directives) - (indent-line-to base-ind) - (verilog-forward-ws&directives) - (verilog-re-search-forward "[ \t\n\f]" e 'move)) - (t - (just-one-space) - (verilog-re-search-forward "[ \t\n\f]" e 'move))) - ;;(forward-line) - ) - ;; Now find biggest prefix - (setq ind (verilog-get-lineup-indent start edpos)) - ;; Now indent each line. - (goto-char start) - (while (progn (setq e (marker-position edpos)) - (setq r (- e (point))) - (> r 0)) - (setq e (point)) - (unless quiet (message "%d" r)) - (cond - ((or (and verilog-indent-declaration-macros - (looking-at verilog-declaration-re-1-macro)) - (looking-at verilog-declaration-re-1-no-macro)) - (let ((p (match-end 0))) - (set-marker m1 p) - (if (verilog-re-search-forward "[[#`]" p 'move) - (progn - (forward-char -1) - (just-one-space) - (goto-char (marker-position m1)) - (just-one-space) - (indent-to ind)) - (progn - (just-one-space) - (indent-to ind))))) - ((verilog-continued-line-1 start) - (goto-char e) - (indent-line-to ind)) - (t ; Must be comment or white space - (goto-char e) - (verilog-forward-ws&directives) - (forward-line -1))) - (forward-line 1)) - (unless quiet (message "")))))) + (verilog-beg-of-statement-1) + (and (not (verilog-in-directive-p)) ;; could have `define input foo + (not (verilog-parenthesis-depth)) ;; could be in a #(param block ) + (looking-at verilog-declaration-re))) + (let* ((m1 (make-marker)) + (e (point)) + (r) + (here (point)) + ;; Start of declaration range + (start + (progn + (verilog-beg-of-statement-1) + (while (and (looking-at verilog-declaration-re) + (not (bobp))) + (skip-chars-backward " \t") + (setq e (point)) + (beginning-of-line) + (verilog-backward-syntactic-ws) + (backward-char) + (verilog-beg-of-statement-1)) + e)) + ;; End of declaration range + (end + (progn + (goto-char here) + (verilog-end-of-statement) + (setq e (point)) ;Might be on last line + (verilog-forward-syntactic-ws) + (while (looking-at verilog-declaration-re) + ;;(beginning-of-line) + (verilog-end-of-statement) + (setq e (point)) + (verilog-forward-syntactic-ws)) + e)) + (edpos (set-marker (make-marker) end)) + (ind) + (base-ind + (progn + (goto-char start) + (verilog-do-indent (verilog-calculate-indent)) + (verilog-forward-ws&directives) + (current-column)))) + (goto-char start) + (if (and (not quiet) + (> (- end start) 100)) + (message "Lining up declarations..(please stand by)")) + ;; Get the beginning of line indent first + (while (progn (setq e (marker-position edpos)) + (< (point) e)) + (cond + ( (save-excursion (skip-chars-backward " \t") + (bolp)) + (verilog-forward-ws&directives) + (indent-line-to base-ind) + (verilog-forward-ws&directives) + (verilog-re-search-forward "[ \t\n\f]" e 'move)) + (t + (just-one-space) + (verilog-re-search-forward "[ \t\n\f]" e 'move))) + ;;(forward-line) + ) + ;; Now find biggest prefix + (setq ind (verilog-get-lineup-indent start edpos)) + ;; Now indent each line. + (goto-char start) + (while (progn (setq e (marker-position edpos)) + (setq r (- e (point))) + (> r 0)) + (setq e (point)) + (unless quiet (message "%d" r)) + (verilog-indent-line) + (cond + ((or (and verilog-indent-declaration-macros + (looking-at verilog-declaration-re-2-macro)) + (looking-at verilog-declaration-re-2-no-macro)) + (let ((p (match-end 0))) + (set-marker m1 p) + (if (verilog-re-search-forward "[[#`]" p 'move) + (progn + (forward-char -1) + (just-one-space) + (goto-char (marker-position m1)) + (just-one-space) + (indent-to ind)) + (progn + (just-one-space) + (indent-to ind))))) + ((verilog-continued-line-1 start) + (goto-char e) + (indent-line-to ind)) + ((verilog-in-struct-p) + ;; could have a declaration of a user defined item + (goto-char e) + (verilog-end-of-statement)) + (t ; Must be comment or white space + (goto-char e) + (verilog-forward-ws&directives) + (forward-line -1))) + (forward-line 1)) + (unless quiet (message "")))))) (defun verilog-pretty-expr (&optional quiet myre) "Line up expressions around point, or optional regexp MYRE." @@ -5040,89 +5336,95 @@ (if (or (eq myre nil) (string-equal myre "")) (setq myre "\\(<\\|:\\)?=")) - (setq myre (concat "\\(^[^;#<=>]*\\)\\(" myre "\\)")) + (setq myre (concat "\\(^[^;#:<=>]*\\)\\(" myre "\\)")) (let ((rexp(concat "^\\s-*" verilog-complete-reg))) (beginning-of-line) (if (and (not (looking-at rexp )) - (looking-at myre)) - (let* ((here (point)) - (e) (r) - (start - (progn - (beginning-of-line) - (setq e (point)) - (verilog-backward-syntactic-ws) - (beginning-of-line) - (while (and (not (looking-at rexp )) - (looking-at myre) - (not (bobp)) - ) - (setq e (point)) - (verilog-backward-syntactic-ws) - (beginning-of-line) - ) ;Ack, need to grok `define - e)) - (end - (progn - (goto-char here) - (end-of-line) - (setq e (point)) ;Might be on last line - (verilog-forward-syntactic-ws) - (beginning-of-line) - (while (and (not (looking-at rexp )) - (looking-at myre)) - (end-of-line) - (setq e (point)) - (verilog-forward-syntactic-ws) - (beginning-of-line) - ) - e)) - (edpos (set-marker (make-marker) end)) - (ind) - ) - (goto-char start) - (verilog-do-indent (verilog-calculate-indent)) - (if (and (not quiet) - (> (- end start) 100)) - (message "Lining up expressions..(please stand by)")) - - ;; Set indent to minimum throughout region - (while (< (point) (marker-position edpos)) - (beginning-of-line) - (verilog-just-one-space myre) - (end-of-line) - (verilog-forward-syntactic-ws) - ) - - ;; Now find biggest prefix - (setq ind (verilog-get-lineup-indent-2 myre start edpos)) - - ;; Now indent each line. - (goto-char start) - (while (progn (setq e (marker-position edpos)) - (setq r (- e (point))) - (> r 0)) - (setq e (point)) - (if (not quiet) (message "%d" r)) - (cond - ((looking-at myre) - (goto-char (match-end 1)) - (if (not (verilog-parenthesis-depth)) ;; ignore parenthsized exprs - (if (eq (char-after) ?=) - (indent-to (1+ ind)) ; line up the = of the <= with surrounding = - (indent-to ind) - ))) - ((verilog-continued-line-1 start) - (goto-char e) - (indent-line-to ind)) - (t ; Must be comment or white space - (goto-char e) - (verilog-forward-ws&directives) - (forward-line -1)) - ) - (forward-line 1)) - (unless quiet (message "")) - ))))) + (looking-at myre) + (save-excursion + (goto-char (match-beginning 2)) + (not (verilog-in-comment-or-string-p)))) + (let* ((here (point)) + (e) (r) + (start + (progn + (beginning-of-line) + (setq e (point)) + (verilog-backward-syntactic-ws) + (beginning-of-line) + (while (and (not (looking-at rexp )) + (looking-at myre) + (not (bobp)) + ) + (setq e (point)) + (verilog-backward-syntactic-ws) + (beginning-of-line) + ) ;Ack, need to grok `define + e)) + (end + (progn + (goto-char here) + (end-of-line) + (setq e (point)) ;Might be on last line + (verilog-forward-syntactic-ws) + (beginning-of-line) + (while (and + (not (looking-at rexp )) + (looking-at myre) + (progn + (end-of-line) + (not (eq e (point))))) + (setq e (point)) + (verilog-forward-syntactic-ws) + (beginning-of-line) + ) + e)) + (edpos (set-marker (make-marker) end)) + (ind) + ) + (goto-char start) + (verilog-do-indent (verilog-calculate-indent)) + (if (and (not quiet) + (> (- end start) 100)) + (message "Lining up expressions..(please stand by)")) + + ;; Set indent to minimum throughout region + (while (< (point) (marker-position edpos)) + (beginning-of-line) + (verilog-just-one-space myre) + (end-of-line) + (verilog-forward-syntactic-ws) + ) + + ;; Now find biggest prefix + (setq ind (verilog-get-lineup-indent-2 myre start edpos)) + + ;; Now indent each line. + (goto-char start) + (while (progn (setq e (marker-position edpos)) + (setq r (- e (point))) + (> r 0)) + (setq e (point)) + (if (not quiet) (message "%d" r)) + (cond + ((looking-at myre) + (goto-char (match-beginning 2)) + (if (not (verilog-parenthesis-depth)) ;; ignore parenthsized exprs + (if (eq (char-after) ?=) + (indent-to (1+ ind)) ; line up the = of the <= with surrounding = + (indent-to ind) + ))) + ((verilog-continued-line-1 start) + (goto-char e) + (indent-line-to ind)) + (t ; Must be comment or white space + (goto-char e) + (verilog-forward-ws&directives) + (forward-line -1)) + ) + (forward-line 1)) + (unless quiet (message "")) + ))))) (defun verilog-just-one-space (myre) "Remove extra spaces around regular expression MYRE." @@ -5157,8 +5459,8 @@ (indent-line-to val) ;; Use previous declaration (in this module) as template. - (if (or (memq 'all verilog-auto-lineup) - (memq 'declaration verilog-auto-lineup)) + (if (or (eq 'all verilog-auto-lineup) + (eq 'declarations verilog-auto-lineup)) (if (verilog-re-search-backward (or (and verilog-indent-declaration-macros verilog-declaration-re-1-macro) @@ -5697,12 +5999,16 @@ (verilog-buffer-to-use (current-buffer)) (label (if (not (string= default "")) ;; Do completion with default - (completing-read (concat "Label: (default " default ") ") + (completing-read (concat "Goto-Label: (default " + default ") ") 'verilog-comp-defun nil nil "") ;; There is no default value. Complete without it - (completing-read "Label: " + (completing-read "Goto-Label: " 'verilog-comp-defun nil nil ""))) pt) + ;; Make sure library paths are correct, in case need to resolve module + (verilog-auto-reeval-locals) + (verilog-getopt-flags) ;; If there was no response on prompt, use default value (if (string= label "") (setq label default)) @@ -5846,7 +6152,8 @@ ;; Added by Subbu Meiyappan for Header (defun verilog-header () - "Insert a standard Verilog file header." + "Insert a standard Verilog file header. +See also `verilog-sk-header' for an alternative format." (interactive) (let ((start (point))) (insert "\ @@ -5911,26 +6218,14 @@ (defun verilog-insert-date () "Insert date from the system." (interactive) - (let ((timpos)) - (setq timpos (point)) - (if verilog-date-scientific-format - (shell-command "date \"+@%Y/%m/%d\"" t) - (shell-command "date \"+@%d.%m.%Y\"" t)) - (search-forward "@") - (delete-region timpos (point)) - (end-of-line)) - (delete-char 1)) + (if verilog-date-scientific-format + (insert (format-time-string "%Y/%m/%d")) + (insert (format-time-string "%d.%m.%Y")))) (defun verilog-insert-year () "Insert year from the system." (interactive) - (let ((timpos)) - (setq timpos (point)) - (shell-command "date \"+@%Y\"" t) - (search-forward "@") - (delete-region timpos (point)) - (end-of-line)) - (delete-char 1)) + (insert (format-time-string "%Y"))) ;; @@ -5961,6 +6256,8 @@ (setq str (concat str (car args))) (setq args (cdr args))) str))) +(defsubst verilog-sig-modport (sig) + (nth 8 sig)) (defsubst verilog-sig-width (sig) (verilog-make-width-expression (verilog-sig-bits sig))) @@ -6026,6 +6323,7 @@ sig highbit lowbit ; Temp information about current signal sv-name sv-highbit sv-lowbit ; Details about signal we are forming sv-comment sv-memory sv-enum sv-signed sv-type sv-multidim sv-busstring + sv-modport bus) ;; Shove signals so duplicated signals will be adjacent (setq in-list (sort in-list `verilog-signals-sort-compare)) @@ -6042,6 +6340,7 @@ sv-signed (verilog-sig-signed sig) sv-type (verilog-sig-type sig) sv-multidim (verilog-sig-multidim sig) + sv-modport (verilog-sig-modport sig) combo "" buswarn "")) ;; Extract bus details @@ -6080,7 +6379,8 @@ sv-enum (or sv-enum (verilog-sig-enum sig)) sv-signed (or sv-signed (verilog-sig-signed sig)) sv-type (or sv-type (verilog-sig-type sig)) - sv-multidim (or sv-multidim (verilog-sig-multidim sig)))) + sv-multidim (or sv-multidim (verilog-sig-multidim sig)) + sv-modport (or sv-modport (verilog-sig-modport sig)))) ;; Doesn't match next signal, add to queue, zero in prep for next ;; Note sig may also be nil for the last signal in the list (t @@ -6092,7 +6392,7 @@ (concat "[" (int-to-string sv-highbit) ":" (int-to-string sv-lowbit) "]"))) (concat sv-comment combo buswarn) - sv-memory sv-enum sv-signed sv-type sv-multidim) + sv-memory sv-enum sv-signed sv-type sv-multidim sv-modport) out-list) sv-name nil)))) ;; @@ -6162,7 +6462,8 @@ (skip-chars-backward "a-zA-Z0-9`_$") (looking-at "[a-zA-Z0-9`_\$]+") ;; Important: don't use match string, this must work with Emacs 19 font-lock on - (buffer-substring-no-properties (match-beginning 0) (match-end 0)))) + (verilog-symbol-detick + (buffer-substring-no-properties (match-beginning 0) (match-end 0)) t))) (defun verilog-read-inst-param-value () "Return list of parameters and values when point is inside instantiation." @@ -6214,9 +6515,11 @@ "Compute signal declaration information for the current module at point. Return a array of [outputs inouts inputs wire reg assign const]." (let ((end-mod-point (or (verilog-get-end-of-defun t) (point-max))) - (functask 0) (paren 0) (sig-paren 0) - sigs-in sigs-out sigs-inout sigs-wire sigs-reg sigs-assign sigs-const sigs-gparam - vec expect-signal keywd newsig rvalue enum io signed typedefed multidim) + (functask 0) (paren 0) (sig-paren 0) (v2kargs-ok t) + sigs-in sigs-out sigs-inout sigs-wire sigs-reg sigs-assign sigs-const + sigs-gparam sigs-intf + vec expect-signal keywd newsig rvalue enum io signed typedefed multidim + modport) (save-excursion (verilog-beg-of-defun) (setq sigs-const (verilog-read-auto-constants (point) end-mod-point)) @@ -6235,14 +6538,15 @@ (error "%s: Unmatched /* */, at char %d" (verilog-point-text) (point)))) ((looking-at "(\\*") (forward-char 2) - (or (looking-at "\\s-*)") ; It's a "always @ (*)" + (or (looking-at "\\s-*)") ; It's an "always @ (*)" (search-forward "*)") (error "%s: Unmatched (* *), at char %d" (verilog-point-text) (point)))) ((eq ?\" (following-char)) (or (re-search-forward "[^\\]\"" nil t) ;; don't forward-char first, since we look for a non backslash first (error "%s: Unmatched quotes, at char %d" (verilog-point-text) (point)))) ((eq ?\; (following-char)) - (setq vec nil io nil expect-signal nil newsig nil paren 0 rvalue nil) + (setq vec nil io nil expect-signal nil newsig nil paren 0 rvalue nil + v2kargs-ok nil) (forward-char 1)) ((eq ?= (following-char)) (setq rvalue t newsig nil) @@ -6283,57 +6587,61 @@ (setq keywd (concat keywd " "))) ;; Escaped ID needs space at end (cond ((equal keywd "input") (setq vec nil enum nil rvalue nil newsig nil signed nil typedefed nil multidim nil sig-paren paren - expect-signal 'sigs-in io t)) + expect-signal 'sigs-in io t modport nil)) ((equal keywd "output") (setq vec nil enum nil rvalue nil newsig nil signed nil typedefed nil multidim nil sig-paren paren - expect-signal 'sigs-out io t)) + expect-signal 'sigs-out io t modport nil)) ((equal keywd "inout") (setq vec nil enum nil rvalue nil newsig nil signed nil typedefed nil multidim nil sig-paren paren - expect-signal 'sigs-inout io t)) - ((or (equal keywd "wire") - (equal keywd "tri") - (equal keywd "tri0") - (equal keywd "tri1")) + expect-signal 'sigs-inout io t modport nil)) + ((equal keywd "parameter") + (setq vec nil enum nil rvalue nil signed nil typedefed nil multidim nil sig-paren paren + expect-signal 'sigs-gparam io t modport nil)) + ((member keywd '("wire" "tri" "tri0" "tri1" "triand" "trior" "wand" "wor")) (unless io (setq vec nil enum nil rvalue nil signed nil typedefed nil multidim nil sig-paren paren - expect-signal 'sigs-wire))) - ((member keywd (list "reg" "trireg" - "byte" "shortint" "int" "longint" "integer" "time" - "bit" "logic")) + expect-signal 'sigs-wire modport nil))) + ((member keywd '("reg" "trireg" + "byte" "shortint" "int" "longint" "integer" "time" + "bit" "logic")) (unless io (setq vec nil enum nil rvalue nil signed nil typedefed nil multidim nil sig-paren paren - expect-signal 'sigs-reg))) + expect-signal 'sigs-reg modport nil))) ((equal keywd "assign") (setq vec nil enum nil rvalue nil signed nil typedefed nil multidim nil sig-paren paren - expect-signal 'sigs-assign)) - ((or (equal keywd "supply0") - (equal keywd "supply1") - (equal keywd "supply") - (equal keywd "localparam") - (equal keywd "genvar")) + expect-signal 'sigs-assign modport nil)) + ((member keywd '("supply0" "supply1" "supply" + "localparam" "genvar")) (unless io (setq vec nil enum nil rvalue nil signed nil typedefed nil multidim nil sig-paren paren - expect-signal 'sigs-const))) - ((or (equal keywd "parameter")) - (unless io (setq vec nil enum nil rvalue nil signed nil typedefed nil multidim nil sig-paren paren - expect-signal 'sigs-gparam))) + expect-signal 'sigs-const modport nil))) ((equal keywd "signed") (setq signed "signed")) - ((or (equal keywd "function") - (equal keywd "task")) + ((member keywd '("class" "clocking" "covergroup" "function" + "property" "randsequence" "sequence" "task")) (setq functask (1+ functask))) - ((or (equal keywd "endfunction") - (equal keywd "endtask")) + ((member keywd '("endclass" "endclocking" "endgroup" "endfunction" + "endproperty" "endsequence" "endtask")) (setq functask (1- functask))) - ((or (equal keywd "`ifdef") - (equal keywd "`ifndef")) + ;; Ifdef? Ignore name of define + ((member keywd '("`ifdef" "`ifndef")) (setq rvalue t)) + ;; Type? ((verilog-typedef-name-p keywd) (setq typedefed keywd)) + ;; Interface with optional modport in v2k arglist? + ;; Skip over parsing modport, and take the interface name as the type + ((and v2kargs-ok + (eq paren 1) + (looking-at "\\s-*\\(\\.\\(\\s-*[a-zA-Z0-9`_$]+\\)\\|\\)\\s-*[a-zA-Z0-9`_$]+")) + (when (match-end 2) (goto-char (match-end 2))) + (setq vec nil enum nil rvalue nil newsig nil signed nil typedefed keywd multidim nil sig-paren paren + expect-signal 'sigs-intf io t modport (match-string 2))) + ;; New signal, maybe? ((and expect-signal (eq functask 0) (not rvalue) (eq paren sig-paren) (not (member keywd verilog-keywords))) ;; Add new signal to expect-signal's variable - (setq newsig (list keywd vec nil nil enum signed typedefed multidim)) + (setq newsig (list keywd vec nil nil enum signed typedefed multidim modport)) (set expect-signal (cons newsig (symbol-value expect-signal)))))) (t @@ -6347,7 +6655,8 @@ (nreverse sigs-reg) (nreverse sigs-assign) (nreverse sigs-const) - (nreverse sigs-gparam))))) + (nreverse sigs-gparam) + (nreverse sigs-intf))))) (eval-when-compile ;; Prevent compile warnings; these are let's, not globals @@ -6355,7 +6664,8 @@ ;; - we want a error when we are debugging this code if they are refed. (defvar sigs-in) (defvar sigs-inout) - (defvar sigs-out)) + (defvar sigs-out) + (defvar sigs-intf)) (defsubst verilog-modi-get-decls (modi) @@ -6384,12 +6694,16 @@ (aref decls 6)) (defsubst verilog-decls-get-gparams (decls) (aref decls 7)) +(defsubst verilog-decls-get-interfaces (decls) + (aref decls 8)) (defsubst verilog-subdecls-get-outputs (subdecls) (aref subdecls 0)) (defsubst verilog-subdecls-get-inouts (subdecls) (aref subdecls 1)) (defsubst verilog-subdecls-get-inputs (subdecls) (aref subdecls 2)) +(defsubst verilog-subdecls-get-interfaces (subdecls) + (aref subdecls 3)) (defun verilog-read-sub-decls-sig (submoddecls comment port sig vec multidim) @@ -6398,7 +6712,7 @@ (when sig (setq port (verilog-symbol-detick-denumber port)) (setq sig (verilog-symbol-detick-denumber sig)) - (if sig (setq sig (verilog-string-replace-matches "^[---+~!|&]+" "" nil nil sig))) + (if sig (setq sig (verilog-string-replace-matches "^\\s-*[---+~!|&]+\\s-*" "" nil nil sig))) (if vec (setq vec (verilog-symbol-detick-denumber vec))) (if multidim (setq multidim (mapcar `verilog-symbol-detick-denumber multidim))) (unless (or (not sig) @@ -6421,13 +6735,57 @@ (verilog-sig-type portdata) multidim) sigs-in))) + ((setq portdata (assoc port (verilog-decls-get-interfaces submoddecls))) + (setq sigs-intf (cons (list sig vec (concat "To/From " comment) nil nil + (verilog-sig-signed portdata) + (verilog-sig-type portdata) + multidim) + sigs-intf))) ;; (t -- warning pin isn't defined.) ; Leave for lint tool ))))) +(defun verilog-read-sub-decls-expr (submoddecls comment port expr) + "For `verilog-read-sub-decls-line', parse a subexpression and add signals." + ;;(message "vrsde: '%s'" expr) + ;; Replace special /*[....]*/ comments inserted by verilog-auto-inst-port + (setq expr (verilog-string-replace-matches "/\\*\\(\\[[^*]+\\]\\)\\*/" "\\1" nil nil expr)) + ;; Remove front operators + (setq expr (verilog-string-replace-matches "^\\s-*[---+~!|&]+\\s-*" "" nil nil expr)) + ;; + (cond + ;; {..., a, b} requires us to recurse on a,b + ((string-match "^\\s-*{\\([^{}]*\\)}\\s-*$" expr) + (let ((mlst (split-string (match-string 1 expr) ",")) + mstr) + (while (setq mstr (pop mlst)) + (verilog-read-sub-decls-expr submoddecls comment port mstr)))) + (t + (let (sig vec multidim) + (cond ;; Find \signal. Final space is part of escaped signal name + ((string-match "^\\s-*\\(\\\\[^ \t\n\f]+\\s-\\)" expr) + ;;(message "vrsde-s: '%s'" (match-string 1 expr)) + (setq sig (match-string 1 expr) + expr (substring expr (match-end 0)))) + ;; Find signal + ((string-match "^\\s-*\\([^[({).\\]+\\)" expr) + ;;(message "vrsde-s: '%s'" (match-string 1 expr)) + (setq sig (verilog-string-remove-spaces (match-string 1 expr)) + expr (substring expr (match-end 0))))) + ;; Find [vector] or [multi][multi][multi][vector] + (while (string-match "^\\s-*\\(\\[[^]]+\\]\\)" expr) + ;;(message "vrsde-v: '%s'" (match-string 1 expr)) + (when vec (setq multidim (cons vec multidim))) + (setq vec (match-string 1 expr) + expr (substring expr (match-end 0)))) + ;; If found signal, and nothing unrecognized, add the signal + ;;(message "vrsde-rem: '%s'" expr) + (when (and sig (string-match "^\\s-*$" expr)) + (verilog-read-sub-decls-sig submoddecls comment port sig vec multidim)))))) + (defun verilog-read-sub-decls-line (submoddecls comment) "For `verilog-read-sub-decls', read lines of port defs until none match anymore. Return the list of signals found, using submodi to look up each port." - (let (done port sig vec multidim) + (let (done port) (save-excursion (forward-line 1) (while (not done) @@ -6443,52 +6801,29 @@ (goto-char (match-end 0))) (t (setq port nil done t))) ;; Unknown, ignore rest of line - ;; Get signal name + ;; Get signal name. Point is at the first-non-space after ( + ;; We intentionally ignore (non-escaped) signals with .s in them + ;; this prevents AUTOWIRE etc from noticing hierarchical sigs. (when port - (setq multidim nil) - (cond ((looking-at "\\(\\\\[^ \t\n\f]*\\)\\s-*)") - (setq sig (concat (match-string 1) " ") ;; escaped id's need trailing space - vec nil)) - ; We intentionally ignore (non-escaped) signals with .s in them - ; this prevents AUTOWIRE etc from noticing hierarchical sigs. - ((looking-at "\\([^[({).]*\\)\\s-*)") - (setq sig (verilog-string-remove-spaces (match-string 1)) - vec nil)) - ((looking-at "\\([^[({).]*\\)\\s-*\\(\\[[^]]+\\]\\)\\s-*)") - (setq sig (verilog-string-remove-spaces (match-string 1)) - vec (match-string 2))) - ((looking-at "\\([^[({).]*\\)\\s-*/\\*\\(\\[[^*]+\\]\\)\\*/\\s-*)") - (setq sig (verilog-string-remove-spaces (match-string 1)) - vec nil) - (let ((parse (match-string 2))) - (while (string-match "^\\(\\[[^]]+\\]\\)\\(.*\\)$" parse) - (when vec (setq multidim (cons vec multidim))) - (setq vec (match-string 1 parse)) - (setq parse (match-string 2 parse))))) - ((looking-at "{\\(.*\\)}.*\\s-*)") - (let ((mlst (split-string (match-string 1) ",")) - mstr) - (while (setq mstr (pop mlst)) - ;;(unless noninteractive (message "sig: %s " mstr)) - (cond - ((string-match "\\(['`a-zA-Z0-9_$]+\\)\\s-*$" mstr) - (setq sig (verilog-string-remove-spaces (match-string 1 mstr)) - vec nil) - ;;(unless noninteractive (message "concat sig1: %s %s" mstr (match-string 1 mstr))) - ) - ((string-match "\\([^[({).]+\\)\\s-*\\(\\[[^]]+\\]\\)\\s-*" mstr) - (setq sig (verilog-string-remove-spaces (match-string 1 mstr)) - vec (match-string 2 mstr)) - ;;(unless noninteractive (message "concat sig2: '%s' '%s' '%s'" mstr (match-string 1 mstr) (match-string 2 mstr))) - ) - (t - (setq sig nil))) - ;; Process signals - (verilog-read-sub-decls-sig submoddecls comment port sig vec multidim)))) - (t - (setq sig nil))) - ;; Process signals - (verilog-read-sub-decls-sig submoddecls comment port sig vec multidim)) + (cond ((looking-at "\\([^[({).\\]*\\)\\s-*)") + (verilog-read-sub-decls-sig + submoddecls comment port + (verilog-string-remove-spaces (match-string 1)) ; sig + nil nil)) ; vec multidim + ;; + ((looking-at "\\([^[({).\\]*\\)\\s-*\\(\\[[^]]+\\]\\)\\s-*)") + (verilog-read-sub-decls-sig + submoddecls comment port + (verilog-string-remove-spaces (match-string 1)) ; sig + (match-string 2) nil)) ; vec multidim + ;; Fastpath was above looking-at's. + ;; For something more complicated invoke a parser + ((looking-at "[^)]+") + (verilog-read-sub-decls-expr + submoddecls comment port + (buffer-substring + (point) (1- (progn (backward-char 1) ; start at ( + (forward-sexp 1) (point)))))))) ; expr ;; (forward-line 1))))) @@ -6514,7 +6849,7 @@ (let ((end-mod-point (verilog-get-end-of-defun t)) st-point end-inst-point ;; below 3 modified by verilog-read-sub-decls-line - sigs-out sigs-inout sigs-in) + sigs-out sigs-inout sigs-in sigs-intf) (verilog-beg-of-defun) (while (verilog-re-search-forward "\\(/\\*AUTOINST\\*/\\|\\.\\*\\)" end-mod-point t) (save-excursion @@ -6532,6 +6867,9 @@ (verilog-backward-open-paren) (setq end-inst-point (save-excursion (forward-sexp 1) (point)) st-point (point)) + (while (re-search-forward "\\s *(?\\s *// Interfaces" end-inst-point t) + (verilog-read-sub-decls-line submoddecls comment)) ;; Modifies sigs-out + (goto-char st-point) (while (re-search-forward "\\s *(?\\s *// Outputs" end-inst-point t) (verilog-read-sub-decls-line submoddecls comment)) ;; Modifies sigs-out (goto-char st-point) @@ -6545,7 +6883,8 @@ ;;(setq rr (vector sigs-out sigs-inout sigs-in)) (vector (verilog-signals-combine-bus (nreverse sigs-out)) (verilog-signals-combine-bus (nreverse sigs-inout)) - (verilog-signals-combine-bus (nreverse sigs-in)))))) + (verilog-signals-combine-bus (nreverse sigs-in)) + (verilog-signals-combine-bus (nreverse sigs-intf)))))) (defun verilog-read-inst-pins () "Return an array of [ pins ] for the current instantiation at point. @@ -6677,13 +7016,16 @@ (setq ignore-next nil rvalue nil)) ((equal "?" exit-keywd) ;; x?y:z rvalue ) ;; NOP + ((equal "]" exit-keywd) ;; [x:y] rvalue + ) ;; NOP (got-sig ;; label: statement (setq ignore-next nil rvalue semi-rvalue got-sig nil)) ((not rvalue) ;; begin label (setq ignore-next t rvalue nil))) (forward-char 1)) ((equal keywd "=") - (if (eq (char-before) ?< ) + (if (and (eq (char-before) ?< ) + (not rvalue)) (setq uses-delayed 1)) (setq ignore-next nil rvalue t) (forward-char 1)) @@ -6881,6 +7223,7 @@ (if enumname (let ((enumvar (intern (concat "venum-" enumname)))) ;;(message "Define %s=%s" defname defvalue) (sleep-for 1) + (unless (boundp enumvar) (set enumvar nil)) (make-variable-buffer-local enumvar) (add-to-list enumvar defname))))) @@ -6952,16 +7295,12 @@ ;; Hack: Read parameters (goto-char (point-min)) (while (re-search-forward - "^\\s-*\\(parameter\\|localparam\\)\\(\\(\\s-*\\[[^]]*\\]\\|\\)\\s-+\\([a-zA-Z0-9_$]+\\)\\s-*=\\s-*\\([^;,]*\\),?\\|\\)\\s-*" nil t) - (let ((var (match-string-no-properties 4)) - (val (match-string-no-properties 5)) - enumname) + "^\\s-*\\(parameter\\|localparam\\)\\(\\s-*\\[[^]]*\\]\\)?\\s-+" nil t) + (let (enumname) ;; The primary way of getting defines is verilog-read-decls ;; However, that isn't called yet for included files, so we'll add another scheme (if (looking-at "[^\n]*synopsys\\s +enum\\s +\\([a-zA-Z0-9_]+\\)") (setq enumname (match-string-no-properties 1))) - (if var - (verilog-set-define var val origbuf enumname)) (forward-comment 999) (while (looking-at "\\s-*,?\\s-*\\([a-zA-Z0-9_$]+\\)\\s-*=\\s-*\\([^;,]*\\),?\\s-*") (verilog-set-define (match-string-no-properties 1) (match-string-no-properties 2) origbuf enumname) @@ -7081,23 +7420,26 @@ ((or (string-match "^\\+incdir\\+\\(.*\\)" arg) ;; +incdir+dir (string-match "^-I\\(.*\\)" arg)) ;; -Idir (verilog-add-list-unique `verilog-library-directories - (match-string 1 arg))) + (match-string 1 (substitute-in-file-name arg)))) ;; Ignore ((equal "+librescan" arg)) ((string-match "^-U\\(.*\\)" arg)) ;; -Udefine ;; Second parameters ((equal next-param "-f") (setq next-param nil) - (verilog-getopt-file arg)) + (verilog-getopt-file (substitute-in-file-name arg))) ((equal next-param "-v") (setq next-param nil) - (verilog-add-list-unique `verilog-library-files arg)) + (verilog-add-list-unique `verilog-library-files + (substitute-in-file-name arg))) ((equal next-param "-y") (setq next-param nil) - (verilog-add-list-unique `verilog-library-directories arg)) + (verilog-add-list-unique `verilog-library-directories + (substitute-in-file-name arg))) ;; Filename ((string-match "^[^-+]" arg) - (verilog-add-list-unique `verilog-library-files arg)) + (verilog-add-list-unique `verilog-library-files + (substitute-in-file-name arg))) ;; Default - ignore; no warning )))) ;;(verilog-getopt (list "+libext+.a+.b" "+incdir+foodir" "+define+a+aval" "-f" "otherf" "-v" "library" "-y" "dir")) @@ -7279,7 +7621,7 @@ (let ((ok t) symbol val) (while (and ok (string-match "`\\([a-zA-Z0-9_]+\\)" text)) (setq symbol (match-string 1 text)) - (message symbol) + ;;(message symbol) (cond ((and (boundp (intern (concat "vh-" symbol))) ;; Emacs has a bug where boundp on a buffer-local @@ -7571,7 +7913,7 @@ (defun verilog-signals-matching-regexp (in-list regexp) "Return all signals in IN-LIST matching the given REGEXP, if non-nil." - (if (not regexp) + (if (or (not regexp) (equal regexp "")) in-list (let (out-list) (while in-list @@ -7582,7 +7924,7 @@ (defun verilog-signals-not-matching-regexp (in-list regexp) "Return all signals in IN-LIST not matching the given REGEXP, if non-nil." - (if (not regexp) + (if (or (not regexp) (equal regexp "")) in-list (let (out-list) (while in-list @@ -7591,6 +7933,24 @@ (setq in-list (cdr in-list))) (nreverse out-list)))) +(defun verilog-signals-matching-dir-re (in-list decl-type regexp) + "Return all signals in IN-LIST matching the given directional REGEXP, +if non-nil." + (if (or (not regexp) (equal regexp "")) + in-list + (let (out-list to-match) + (while in-list + ;; Note verilog-insert-one-definition matches on this order + (setq to-match (concat + decl-type + " " (verilog-sig-signed (car in-list)) + " " (verilog-sig-multidim (car in-list)) + (verilog-sig-bits (car in-list)))) + (if (string-match regexp to-match) + (setq out-list (cons (car in-list) out-list))) + (setq in-list (cdr in-list))) + (nreverse out-list)))) + ;; Combined (defun verilog-decls-get-signals (decls) (append @@ -7639,7 +7999,10 @@ "Print out a definition for SIG of the given TYPE, with appropriate INDENT-PT indentation." (indent-to indent-pt) + ;; Note verilog-signals-matching-dir-re matches on this order (insert type) + (when (verilog-sig-modport sig) + (insert "." (verilog-sig-modport sig))) (when (verilog-sig-signed sig) (insert " " (verilog-sig-signed sig))) (when (verilog-sig-multidim sig) @@ -7664,7 +8027,7 @@ ;; Want "type x" or "output type x", not "wire type x" (cond ((verilog-sig-type sig) (concat - (if (not (equal direction "wire")) + (if (not (member direction '("wire" "interface"))) (concat direction " ")) (verilog-sig-type sig))) (t direction)) @@ -7694,16 +8057,19 @@ ;;(let ((indent-pt 10)) (verilog-insert-indent "hello\n" "addon" "there\n")) (defun verilog-repair-open-comma () - "If backwards-from-point is other than a open parenthesis insert comma." + "Insert comma if previous argument is other than a open parenthesis or endif." + ;; We can't just search backward for ) as it might be inside another expression. + ;; Also want "`ifdef X input foo `endif" to just leave things to the human to deal with (save-excursion (verilog-backward-syntactic-ws) - (when (save-excursion - (backward-char 1) - (and (not (looking-at "[(,]")) - (progn - (verilog-re-search-backward "[(`]" nil t) - (looking-at "(")))) - (insert ",")))) + (when (and (not (save-excursion ;; Not beginning (, or existing , + (backward-char 1) + (looking-at "[(,]"))) + (not (save-excursion ;; Not `endif, or user define + (backward-char 1) + (skip-chars-backward "[a-zA-Z0-9_`]") + (looking-at "`")))) + (insert ",")))) (defun verilog-repair-close-comma () "If point is at a comma followed by a close parenthesis, fix it. @@ -7740,11 +8106,16 @@ (setq range-exp (match-string 1 range-exp))) (cond ((not range-exp) "1") + ;; [#:#] We can compute a numeric result ((string-match "^\\s *\\([0-9]+\\)\\s *:\\s *\\([0-9]+\\)\\s *$" range-exp) (int-to-string (1+ (abs (- (string-to-number (match-string 1 range-exp)) (string-to-number (match-string 2 range-exp))))))) + ;; [PARAM-1:0] can just return PARAM + ((string-match "^\\s *\\([a-zA-Z_][a-zA-Z0-9_]*\\)\\s *-\\s *1\\s *:\\s *0\\s *$" range-exp) + (match-string 1 range-exp)) + ;; [arbitrary] need math ((string-match "^\\(.*\\)\\s *:\\s *\\(.*\\)\\s *$" range-exp) (concat "(1+(" (match-string 1 range-exp) ")" (if (equal "0" (match-string 2 range-exp)) @@ -7763,12 +8134,12 @@ (while (string-match "(\\<\\([0-9A-Z-az_]+\\)\\>)" out) (setq out (replace-match "\\1" nil nil out))) (while (string-match "\\<\\([0-9]+\\)\\>\\s *\\+\\s *\\<\\([0-9]+\\)\\>" out) - (setq out (replace-match + (setq out (replace-match (int-to-string (+ (string-to-number (match-string 1 out)) (string-to-number (match-string 2 out)))) nil nil out))) (while (string-match "\\<\\([0-9]+\\)\\>\\s *\\-\\s *\\<\\([0-9]+\\)\\>" out) - (setq out (replace-match + (setq out (replace-match (int-to-string (- (string-to-number (match-string 1 out)) (string-to-number (match-string 2 out)))) nil nil out)))) @@ -7915,10 +8286,11 @@ (verilog-regexp-words `("AUTOASCIIENUM" "AUTOCONCATCOMMENT" "AUTODEFINEVALUE" "AUTOINOUT" "AUTOINOUTCOMP" "AUTOINOUTMODULE" - "AUTOINPUT" "AUTOOUTPUT" "AUTOOUTPUTEVERY" + "AUTOINPUT" "AUTOINSERTLISP" "AUTOOUTPUT" "AUTOOUTPUTEVERY" "AUTOREG" "AUTOREGINPUT" "AUTORESET" "AUTOTIEOFF" "AUTOUNUSED" "AUTOWIRE"))) - "\\(\\|([^)]*)\\|(\"[^\"]*\")\\)" ; Optional parens or quoted parameter + ;; Optional parens or quoted parameter or .* for (((...))) + "\\(\\|([^)]*)\\|(\"[^\"]*\")\\|.*?\\)" "\\*/") 'verilog-delete-autos-lined) ;; Remove those that are in parenthesis @@ -8106,7 +8478,8 @@ (let ((curlocal (verilog-auto-read-locals))) (when (or force (not (equal verilog-auto-last-file-locals curlocal))) (setq verilog-auto-last-file-locals curlocal) - ;; Note this may cause this function to be recursively invoked. + ;; Note this may cause this function to be recursively invoked, + ;; because hack-local-variables may call (verilog-mode) ;; The above when statement will prevent it from recursing forever. (hack-local-variables) t))) @@ -8119,6 +8492,8 @@ "Print a list of ports for a AUTOINST. Takes SIGS list, adds MESSAGE to front and inserts each at INDENT-PT." (when sigs + (when verilog-auto-arg-sort + (setq sigs (sort (copy-alist sigs) `verilog-signals-sort-compare))) (insert "\n") (indent-to indent-pt) (insert message) @@ -8167,6 +8542,10 @@ output o; endmodule +The argument declarations may be printed in declaration order to best suit +order based instantiations, or alphabetically, based on the +`verilog-auto-arg-sort' variable. + Any ports declared between the ( and /*AUTOARG*/ are presumed to be predeclared and are not redeclared by AUTOARG. AUTOARG will make a conservative guess on adding a comma for the first signal, if you have @@ -8204,6 +8583,7 @@ (defvar vl-cell-type nil "See `verilog-auto-inst'.") ; Prevent compile warning (defvar vl-cell-name nil "See `verilog-auto-inst'.") ; Prevent compile warning +(defvar vl-modport nil "See `verilog-auto-inst'.") ; Prevent compile warning (defvar vl-name nil "See `verilog-auto-inst'.") ; Prevent compile warning (defvar vl-width nil "See `verilog-auto-inst'.") ; Prevent compile warning (defvar vl-dir nil "See `verilog-auto-inst'.") ; Prevent compile warning @@ -8221,6 +8601,7 @@ ;; vl-* are documented for user use (vl-name (verilog-sig-name port-st)) (vl-width (verilog-sig-width port-st)) + (vl-modport (verilog-sig-modport port-st)) (vl-bits (if (or verilog-auto-inst-vector (not (assoc port vector-skip-list)) (not (equal (verilog-sig-bits port-st) @@ -8241,10 +8622,12 @@ check-values (cdr check-values))) (setq vl-bits (verilog-simplify-range-expression vl-bits))) ; Not in the loop for speed ;; Default net value if not found - (setq tpl-net (if (verilog-sig-multidim port-st) - (concat port "/*" (verilog-sig-multidim-string port-st) - vl-bits "*/") - (concat port vl-bits))) + (setq tpl-net (concat port + (if vl-modport (concat "." vl-modport) "") + (if (verilog-sig-multidim port-st) + (concat "/*" (verilog-sig-multidim-string port-st) + vl-bits "*/") + (concat vl-bits)))) ;; Find template (cond (tpl-ass ; Template of exact port name (setq tpl-net (nth 1 tpl-ass))) @@ -8566,7 +8949,8 @@ vl-bits Bus bits portion of the input/output port ('[2:0]'). vl-width Width of the input/output port ('3' for [2:0]). May be a (...) expression if bits isn't a constant. - vl-dir Direction of the pin input/output/inout. + vl-dir Direction of the pin input/output/inout/interface. + vl-modport The modport, if an interface with a modport. vl-cell-type Module name/type of the cell ('InstModule'). vl-cell-name Instance name of the cell ('instName'). @@ -8579,6 +8963,8 @@ will evaluate any Lisp expression inside the parenthesis between the beginning of the buffer and the point of the AUTOINST. This allows functions to be defined or variables to be changed between instantiations. + (See also `verilog-auto-insert-lisp' if you want the output from your + lisp function to be inserted.) Note that when using lisp expressions errors may occur when @ is not a number; you may need to use the standard Emacs Lisp functions @@ -8629,13 +9015,25 @@ tpl-list (aref tpl-info 1))) ;; Find submodule's signals and dump (let ((sig-list (verilog-signals-not-in + (verilog-decls-get-interfaces submoddecls) + skip-pins)) + (vl-dir "interface")) + (when sig-list + (when (not did-first) (verilog-auto-inst-first) (setq did-first t)) + (indent-to indent-pt) + ;; Note these are searched for in verilog-read-sub-decls. + (insert "// Interfaces\n") + (mapc (lambda (port) + (verilog-auto-inst-port port indent-pt + tpl-list tpl-num for-star par-values)) + sig-list))) + (let ((sig-list (verilog-signals-not-in (verilog-decls-get-outputs submoddecls) skip-pins)) (vl-dir "output")) (when sig-list (when (not did-first) (verilog-auto-inst-first) (setq did-first t)) (indent-to indent-pt) - ;; Note these are searched for in verilog-read-sub-decls. (insert "// Outputs\n") (mapc (lambda (port) (verilog-auto-inst-port port indent-pt @@ -8957,7 +9355,7 @@ (setq pnt (point)) (verilog-pretty-declarations quiet) (goto-char pnt) - (verilog-pretty-expr "//")))))) + (verilog-pretty-expr t "//")))))) (defun verilog-auto-output (&optional with-params) "Expand AUTOOUTPUT statements, as part of \\[verilog-auto]. @@ -9293,11 +9691,25 @@ signals matching the regular expression will be included. For example the same expansion will result from only extracting signals starting with i: - /*AUTOINOUTMODULE(\"ExampMain\",\"^i\")*/" + /*AUTOINOUTMODULE(\"ExampMain\",\"^i\")*/ + +You may also provide an optional second regulat expression, in +which case only signals which have that pin direction and data +type will be included. This matches against everything before +the signal name in the declaration, for example against +\"input\" (single bit), \"output logic\" (direction and type) or +\"output [1:0]\" (direction and implicit type). You also +probably want to skip spaces in your regexp. + +For example, the below will result in matching the output \"o\" +against the previous example's module: + + /*AUTOINOUTMODULE(\"ExampMain\",\"\",\"^output.*\")*/" (save-excursion - (let* ((params (verilog-read-auto-params 1 2)) + (let* ((params (verilog-read-auto-params 1 3)) (submod (nth 0 params)) (regexp (nth 1 params)) + (direction-re (nth 2 params)) submodi) ;; Lookup position, etc of co-module ;; Note this may raise an error @@ -9319,15 +9731,23 @@ (append (verilog-decls-get-outputs moddecls)))) (sig-list-io (verilog-signals-not-in (verilog-decls-get-inouts submoddecls) - (append (verilog-decls-get-inouts moddecls))))) + (append (verilog-decls-get-inouts moddecls)))) + (sig-list-if (verilog-signals-not-in + (verilog-decls-get-interfaces submoddecls) + (append (verilog-decls-get-interfaces moddecls))))) (forward-line 1) - (when regexp - (setq sig-list-i (verilog-signals-matching-regexp - sig-list-i regexp) - sig-list-o (verilog-signals-matching-regexp - sig-list-o regexp) - sig-list-io (verilog-signals-matching-regexp - sig-list-io regexp))) + (setq sig-list-i (verilog-signals-matching-dir-re + (verilog-signals-matching-regexp sig-list-i regexp) + "input" direction-re) + sig-list-o (verilog-signals-matching-dir-re + (verilog-signals-matching-regexp sig-list-o regexp) + "output" direction-re) + sig-list-io (verilog-signals-matching-dir-re + (verilog-signals-matching-regexp sig-list-io regexp) + "inout" direction-re) + sig-list-if (verilog-signals-matching-dir-re + (verilog-signals-matching-regexp sig-list-if regexp) + "interface" direction-re)) (when v2k (verilog-repair-open-comma)) (when (or sig-list-i sig-list-o sig-list-io) (verilog-insert-indent "// Beginning of automatic in/out/inouts (from specific module)\n") @@ -9335,6 +9755,7 @@ (verilog-insert-definition sig-list-o "output" indent-pt v2k t) (verilog-insert-definition sig-list-io "inout" indent-pt v2k t) (verilog-insert-definition sig-list-i "input" indent-pt v2k t) + (verilog-insert-definition sig-list-if "interface" indent-pt v2k t) (verilog-modi-cache-add-inputs modi sig-list-i) (verilog-modi-cache-add-outputs modi sig-list-o) (verilog-modi-cache-add-inouts modi sig-list-io) @@ -9392,6 +9813,69 @@ /*AUTOINOUTCOMP(\"ExampMain\",\"^i\")*/" (verilog-auto-inout-module t)) +(defun verilog-auto-insert-lisp () + "Expand AUTOINSERTLISP statements, as part of \\[verilog-auto]. +The Lisp code provided is called, and the Lisp code calls +`insert` to insert text into the current file beginning on the +line after the AUTOINSERTLISP. + +See also AUTO_LISP, which takes a Lisp expression and evaluates +it during `verilog-auto-inst' but does not insert any text. + +An example: + + module ExampInsertLisp; + /*AUTOINSERTLISP(my-verilog-insert-hello \"world\")*/ + endmodule + + // For this example we declare the function in the + // module's file itself. Often you'd define it instead + // in a site-start.el or .emacs file. + /* + Local Variables: + eval: + (defun my-verilog-insert-hello (who) + (insert (concat \"initial $write(\\\"hello \" who \"\\\");\\n\"))) + End: + */ + +Typing \\[verilog-auto] will call my-verilog-insert-hello and +expand the above into: + + // Beginning of automatic insert lisp + initial $write(\"hello world\"); + // End of automatics + +You can also call an external program and insert the returned +text: + + /*AUTOINSERTLISP(insert (shell-command-to-string \"echo //hello\"))*/ + // Beginning of automatic insert lisp + //hello + // End of automatics" + (save-excursion + ;; Point is at end of /*AUTO...*/ + (let* ((indent-pt (current-indentation)) + (cmd-end-pt (save-excursion (search-backward ")") + (forward-char) + (point))) ;; Closing paren + (cmd-beg-pt (save-excursion (goto-char cmd-end-pt) + (backward-sexp 1) + (point))) ;; Beginning paren + (cmd (buffer-substring-no-properties cmd-beg-pt cmd-end-pt))) + (forward-line 1) + (let ((pre-eval-pt (point))) + ;;Debug: (insert cmd) + ;; Don't use eval-region as Xemacs has a bug where it goto-char's begin-pt + (eval (read cmd)) + ;; If inserted something add the begin/end blocks + (when (not (equal pre-eval-pt (point))) + (when (not (bolp)) (insert "\n")) ;; If user forgot final newline, add it + (save-excursion + (goto-char pre-eval-pt) + (verilog-insert-indent "// Beginning of automatic insert lisp\n")) + (verilog-insert-indent "// End of automatics\n")))))) + (defun verilog-auto-sense-sigs (moddecls presense-sigs) "Return list of signals for current AUTOSENSE block." (let* ((sigss (verilog-read-always-signals)) @@ -9510,7 +9994,8 @@ Limitations: AUTORESET will not clear memories. - AUTORESET uses <= if there are any <= in the block, else it uses =. + AUTORESET uses <= if there are any <= assigmnents in the block, + else it uses =. /*AUTORESET*/ presumes that any signals mentioned between the previous begin/case/if statement and the AUTORESET comment are being reset manually @@ -9770,6 +10255,10 @@ Finally, a AUTOASCIIENUM command is used. The first parameter is the name of the signal to be decoded. + If and only if the first parameter width is 2^(number of states + in enum) and does NOT match the width of the enum, the signal + is assumed to be a one hot decode. Otherwise, it's a normal + encoded state vector. The second parameter is the name to store the ASCII code into. For the signal foo, I suggest the name _foo__ascii, where the leading _ indicates @@ -9787,12 +10276,10 @@ SM_SEND = 3'b001, SM_WAIT1 = 3'b010; //== State variables - reg [2:0] /* synopsys enum state_info */ - state_r; /* synopsys state_vector state_r */ - reg [2:0] /* synopsys enum state_info */ - state_e1; - - //== ASCII state decoding + reg [2:0] /* synopsys enum state_info */ + state_r; /* synopsys state_vector state_r */ + reg [2:0] /* synopsys enum state_info */ + state_e1; /*AUTOASCIIENUM(\"state_r\", \"state_ascii_r\", \"SM_\")*/ @@ -9835,10 +10322,19 @@ (undecode-enum (or (verilog-sig-enum undecode-sig) (error "%s: Signal %s does not have a enum tag" (verilog-point-text) undecode-name))) ;; - (enum-sigs (or (verilog-signals-matching-enum sig-list-consts undecode-enum) - (error "%s: No state definitions for %s" (verilog-point-text) undecode-enum))) + (enum-sigs (verilog-signals-not-in + (or (verilog-signals-matching-enum sig-list-consts undecode-enum) + (error "%s: No state definitions for %s" (verilog-point-text) undecode-enum)) + nil)) ;; - (enum-chars 0) + (one-hot (and ;; width(enum) != width(sig) + (or (not (verilog-sig-bits (car enum-sigs))) + (not (equal (verilog-sig-width (car enum-sigs)) + (verilog-sig-width undecode-sig)))) + ;; count(enums) == width(sig) + (equal (number-to-string (length enum-sigs)) + (verilog-sig-width undecode-sig)))) + (enum-chars 0) (ascii-chars 0)) ;; ;; Find number of ascii chars needed @@ -9864,14 +10360,23 @@ (setq indent-pt (+ indent-pt verilog-case-indent)) ;; (let ((tmp-sigs enum-sigs) - (chrfmt (format "%%-%ds %s = \"%%-%ds\";\n" (1+ (max 8 enum-chars)) + (chrfmt (format "%%-%ds %s = \"%%-%ds\";\n" + (+ (if one-hot 9 1) (max 8 enum-chars)) ascii-name ascii-chars)) (errname (substring "%Error" 0 (min 6 ascii-chars)))) (while tmp-sigs (verilog-insert-indent - (format chrfmt (concat (verilog-sig-name (car tmp-sigs)) ":") - (verilog-enum-ascii (verilog-sig-name (car tmp-sigs)) - elim-regexp))) + (concat + (format chrfmt + (concat (if one-hot "(") + (if one-hot (verilog-sig-width undecode-sig)) + ;; We use a shift instead of var[index] + ;; so that a non-one hot value will show as error. + (if one-hot "'b1<<") + (verilog-sig-name (car tmp-sigs)) + (if one-hot ")") ":") + (verilog-enum-ascii (verilog-sig-name (car tmp-sigs)) + elim-regexp)))) (setq tmp-sigs (cdr tmp-sigs))) (verilog-insert-indent (format chrfmt "default:" errname))) ;; @@ -9946,6 +10451,7 @@ `verilog-auto-inout-module' for AUTOINOUTMODULE copying i/o from elsewhere `verilog-auto-inout' for AUTOINOUT making hierarchy inouts `verilog-auto-input' for AUTOINPUT making hierarchy inputs + `verilog-auto-insert-lisp' for AUTOINSERTLISP insert code from lisp function `verilog-auto-inst' for AUTOINST instantiation pins `verilog-auto-star' for AUTOINST .* SystemVerilog pins `verilog-auto-inst-param' for AUTOINSTPARAM instantiation params @@ -9962,8 +10468,9 @@ `verilog-read-defines' for reading `define values `verilog-read-includes' for reading `includes -If you have bugs with these autos, try contacting the AUTOAUTHOR -Wilson Snyder (wsnyder@wsnyder.org), and/or see http://www.veripool.org." +If you have bugs with these autos, please file an issue at +http://www.veripool.org/verilog-mode or contact the AUTOAUTHOR +Wilson Snyder (wsnyder@wsnyder.org)." (interactive) (unless noninteractive (message "Updating AUTOs...")) (if (fboundp 'dinotrace-unannotate-all) @@ -10006,6 +10513,11 @@ (verilog-inject-sense) (verilog-inject-arg)) ;; + ;; Do user inserts first, so their code can insert AUTOs + ;; We may provide a AUTOINSERTLISPLAST if another cleanup pass is needed + (verilog-auto-re-search-do "/\\*AUTOINSERTLISP(.*?)\\*/" + 'verilog-auto-insert-lisp) + ;; Expand instances before need the signals the instances input/output (verilog-auto-re-search-do "/\\*AUTOINSTPARAM\\*/" 'verilog-auto-inst-param) (verilog-auto-re-search-do "/\\*AUTOINST\\*/" 'verilog-auto-inst) (verilog-auto-re-search-do "\\.\\*" 'verilog-auto-star) @@ -10158,7 +10670,8 @@ (if (> (point) verilog-sk-p) "] " " "))) (defun verilog-sk-header () - "Insert a descriptive header at the top of the file." + "Insert a descriptive header at the top of the file. +See also `verilog-header' for an alternative format." (interactive "*") (save-excursion (goto-char (point-min)) @@ -10172,8 +10685,8 @@ "\n// Description : " str "\n// Author : " (user-full-name) "\n// Created On : " (current-time-string) - "\n// Last Modified By: ." - "\n// Last Modified On: ." + "\n// Last Modified By: " (user-full-name) + "\n// Last Modified On: " (current-time-string) "\n// Update Count : 0" "\n// Status : Unknown, Use with caution!" "\n") @@ -10269,7 +10782,7 @@ and the case items." "[selector expression]: " > "case (" str ") " \n - > ("case selector: " str ": begin" \n > _ \n > (- verilog-indent-level-behavioral) "end" \n ) + > ("case selector: " str ": begin" \n > _ \n > (- verilog-indent-level-behavioral) "end" \n > ) resume: > (- verilog-case-indent) "endcase" (progn (electric-verilog-terminate-line) nil)) (define-skeleton verilog-sk-casex @@ -10277,7 +10790,7 @@ and the case items." "[selector expression]: " > "casex (" str ") " \n - > ("case selector: " str ": begin" \n > _ \n > (- verilog-indent-level-behavioral) "end" \n ) + > ("case selector: " str ": begin" \n > _ \n > (- verilog-indent-level-behavioral) "end" \n > ) resume: > (- verilog-case-indent) "endcase" (progn (electric-verilog-terminate-line) nil)) (define-skeleton verilog-sk-casez @@ -10285,7 +10798,7 @@ and the case items." "[selector expression]: " > "casez (" str ") " \n - > ("case selector: " str ": begin" \n > _ \n > (- verilog-indent-level-behavioral) "end" \n ) + > ("case selector: " str ": begin" \n > _ \n > (- verilog-indent-level-behavioral) "end" \n > ) resume: > (- verilog-case-indent) "endcase" (progn (electric-verilog-terminate-line) nil)) (define-skeleton verilog-sk-if