Menu

#297 Multiple output gates do not work

devel
closed-fixed
5
2008-09-17
2007-09-04
Cary R.
No

Icarus currently only supports a single output. Gates that accept multiple outputs leave the unconnected outputs floating (pulldown/pullup) or in the case of the buf/not gates the first floating output is used as the input! I will send in a patch to check for this and other primitive port inconsistencies. For now the checks limit you to a single output to prevent unexpected results. The attached file can be used to demonstrate the problem, though once the patch is applied these will be caught as elaboration errors.

Discussion

  • Cary R.

    Cary R. - 2007-09-04

    Verilog source file

     
  • Cary R.

    Cary R. - 2007-09-04

    Logged In: YES
    user_id=1651735
    Originator: YES

    This is a gating item for solving pr1750878.

     
  • Stephen Williams

    Logged In: YES
    user_id=97566
    Originator: NO

    I think I might take this on over my July vacation. Looks not too hard.

     
  • Stephen Williams

    Logged In: YES
    user_id=97566
    Originator: NO

    test

     
  • Cary R.

    Cary R. - 2008-06-25

    Logged In: YES
    user_id=1651735
    Originator: YES

    As you probably already realize you solved pr1750878 with a different technique, so this is now just the simple gates (buf, not, pullup, pulldown).

     
  • Cary R.

    Cary R. - 2008-08-22
    • labels: 776822 --> Missing functionality
     
  • Stephen Williams

    I'm working on getting support for multiple-output gates working properly, so I'm claiming this item for now.

     
  • Stephen Williams

    • assigned_to: nobody --> stevewilliams
     
  • Stephen Williams

    • status: open --> open-fixed
     
  • Stephen Williams

    I've pushed into git master a patch that makes the example work properly. It appears in the source that there are cases that are not yet covered (and should trigger assertions) but I've not been able to make them trip at this late hour in a Sunday evening. It's possible that the special case of a single input makes those cases impossible, but I need to at least document that, so this bug report stays open (yet fixed) until I work out the extra details.

     
  • Cary R.

    Cary R. - 2008-09-15

    Here's a more test suite ready example.
    File Added: pr1787423.v

     
  • Cary R.

    Cary R. - 2008-09-15

    Example file

     
  • Stephen Williams

    • status: open-fixed --> closed-fixed
     
  • Stephen Williams

    Now completely fixed in git master. (See pr1787423c.v in the ivtest suite.)

     

Log in to post a comment.