Re: [myhdl-list] intbv with max != 2**n. Error or annoyance?
Brought to you by:
jandecaluwe
From: Per K. <bas...@gm...> - 2013-01-29 14:06:55
|
Hm, that would enforce a coding style where every combinatorial cloud must be written monolithically. You could never break down a combinatorial problem into parts. I think this is a bad coding style. It makes for unreadable code, and it limits the amount of complexity that can be tackled successfully. /Per On Tue, Jan 29, 2013 at 2:43 PM, Christopher Felton <chr...@gm...>wrote: > On 1/29/2013 6:21 AM, Per Karlsson wrote: > > OK, the range check is essentially an assertion. Now, you wouldn't put an > > assertion in a combinatorical block, would you? > > > > Good point. In general this is true because we have > these "transitional" values, so yes. > > > So the least you can expect is a warning message that you got one when > you > > used an intbv with ranges != 2**n as a combinatorical net. > > > > At some point you would want the bound check, presumably > once everything settles out. Does this simply suggest > the assignment should be in clocked process? Is it fair > to assume the value will be clocked into a register eventually > and the combinatorial assignment shouldn't be used? In other > words, does the bound assertion help enforce goodness? > > > Don't worry, otherwise I like intbv as much as the next guy! > > /Per > > > > ps. It behaves the same in 0.7 and 0.8-dev. > > > > Thanks for the additional information, > Chris > > > > > > > > ------------------------------------------------------------------------------ > Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS, > MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current > with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft > MVPs and experts. ON SALE this month only -- learn more at: > http://p.sf.net/sfu/learnnow-d2d > _______________________________________________ > myhdl-list mailing list > myh...@li... > https://lists.sourceforge.net/lists/listinfo/myhdl-list > |