Why is this an issue?

Functions should have an automatic lifetime to be used in synthesizable code.

How to fix it

Code examples

Noncompliant code example

function logic [2:0] foo( // Noncompliant: No "automatic" lifetime
  logic [2:0] a,
  logic [2:0] b
);
  return a ^ b;
endfunction : foo

Compliant solution

function automatic logic [2:0] foo(
  logic [2:0] a,
  logic [2:0] b
);
  return a ^ b;
endfunction : foo

Resources

Articles & blog posts

• lowRISC Verilog Coding Style Guide - Language Features - Functions and Tasks