Menu

#1242 Fix for pr3534333

closed-accepted
nobody
5
2012-07-01
2012-06-16
No

The parser has previously been changed to support null statements in sequential and parallel blocks (a feature introduced in SystemVerilog), but this was not supported in elaboration, leading to a compiler crash. This patch fixes this by discarding the null statements during parsing.

An alternative fix would be to discard the null statements during elaboration, but this would require more widespread changes, and for performance reasons it seems better to discard the null statements as early as possible in the compilation process.

Also attached is a regression test to add to the test suite. This test doesn;t need any special compiler flags or a gold file.

This bug is only present in devel.

Discussion

  • Martin Whitaker

    Martin Whitaker - 2012-06-16

    Proposed patch for compiler

     
  • Martin Whitaker

    Martin Whitaker - 2012-06-16

    Regression test for test suite

     
  • Stephen Williams

    Also added regression test to ivtest master.

     
  • Stephen Williams

    Applied to git master.

     
  • Stephen Williams

    • status: open --> closed-accepted
     

Log in to post a comment.