Forum Discussion

MoZdk's avatar
MoZdk
Icon for Occasional Contributor rankOccasional Contributor
7 years ago

Should Quartus 18.1 support FF creation by this: Q <= D when rising_edge(CLK); The D and CLK are input ports on a module, and Q is a output port. The Quartus 18.1 breaks with internal error due to this construction.

5 Replies

  • sstrell's avatar
    sstrell
    Icon for Super Contributor rankSuper Contributor

    That's not standard synthesizable Verilog or VHDL, so of course it will not work.

  • Tricky's avatar
    Tricky
    Icon for Occasional Contributor rankOccasional Contributor

    @sstrell

    While it is not officially supported (I cant find it in any coding guidelines) it has been supported by many tools for many years.

    In VHDL, one liners like this actually infer a process sensitive to all signals on the RHS. So

    q <= d when rising_edge(clk);

    is functionally identical to

    process(clk, d)  -- D not needed, but it is needed for full equivolence to one liner.
    begin
      if rising_edge(clk) then
        q <= d;
      end if;
    end process;
  • MoZdk's avatar
    MoZdk
    Icon for Occasional Contributor rankOccasional Contributor

    Thanks for the answer.

    It may not be supported coding style for inference of flip-flops, and it is used very little, if at all in practice, so may be OK not to support in Intel coding guidelines.

    However, the coding style makes Quartus 18.1 break with internal error, which indicates a bug in the tool.

    ? Any suggestion for how I can report this bug ?

  • PLehm's avatar
    PLehm
    Icon for New Contributor rankNew Contributor

    This code line is standard synthesizable code.

    It was specified in IEEE Std. 1076.6-2004 Clause 6.1.3.5

    6.1.3.5 Edge-sensitive storage using concurrent signal assignment statements

    A concurrent conditional signal assignment statement may be used to model an edge-sensitive storage element

    provided that the assignment can be mapped to a process that adheres to the rules in 6.1.3.1.

    Example:

    COND_SIG_ASSGN: Q <= '0' when RESET = '1' else
                         '1' when SET = '1' else
                          A when ASYNC_LOAD = '1' else
                          D when CLOCK'EVENT and CLOCK = '1';

    More over it was supported by previous Quartus versions as this tool was called "Altera Quartus II".

    Even if it's an unsupported syntax, a tool should never crash.

    In addition, this syntax is supported by all competitors (I'm listing only synthesizer tools):

    • Xilinx ISE
    • Xilinx Vivado
    • Lattice LSE
    • Synopsys Simplify PRO

    Kind regards

    Patrick Lehmann

    Vice-Chair of the IEEE P1076 Working Group