Re: [Iverilog-devel] Two people working on pr1879226

[Cary R. <cy...@ya...>]

--- Larry Doolittle <ldo...@re...> wrote:

> Hey, competition is good!  ;-)

Yes it is, What you did is almost exactly the same as what I came up with.
Though I would remove the disabled "if" since it serves no purpose other
than to confuse someone later. I would also change the comment to be "If
the entire value is shifted away connect the output to a constant V0 or
for 
arithmetic right shift ">>>"  connect all bits to the MSB."
  
> > It may be time to consider formalizing a method for claiming bug
> > reports and feature requests, as progress on these bugs and features
> > is really heating up. It seems to me that the "Assigned To:"
> > field in the trackers is perfect for this.
> 
> It is, but that shouldn't keep others from jumping in if they have
> something helpful to add.

I agree "Assign To:" is what we should be using and is what I used in this
case.

> In this case, the construct that Uwe found is very close to what
> I use routinely, so I really really wanted this bug fixed before
> it decided to bite me, too.

What is interesting is that if the conditional was correct >= vs > we
would have known exactly where the bug or in this case missing
functionality was and no incorrect code would have been generated.

Cary


      ____________________________________________________________________________________
Looking for last minute shopping deals?  
Find them fast with Yahoo! Search.  http://tools.search.yahoo.com/newsearch/category.php?category=shopping