Catherine[m] changed the topic of #amaranth-lang to: Amaranth hardware definition language · weekly meetings on Mondays at 1700 UTC · code https://github.com/amaranth-lang · logs https://libera.irclog.whitequark.org/amaranth-lang · Matrix #amaranth-lang:matrix.org
Degi_ has joined #amaranth-lang
Degi has quit [Ping timeout: 246 seconds]
Degi_ is now known as Degi
Sarayan has quit [Ping timeout: 240 seconds]
Sarayan has joined #amaranth-lang
oter has quit [Ping timeout: 260 seconds]
oter has joined #amaranth-lang
Lord_Nightmare has quit [Quit: ZNC - http://znc.in]
Lord_Nightmare has joined #amaranth-lang
balrog has quit [Quit: Bye]
balrog has joined #amaranth-lang
_luke_w_[m] has quit [Quit: Bridge terminating on SIGTERM]
_catircservices has quit [Quit: Bridge terminating on SIGTERM]
whitequark[cis] has quit [Quit: Bridge terminating on SIGTERM]
_catircservices has joined #amaranth-lang
_catircservices has quit [Quit: Bridge terminating on SIGTERM]
_catircservices has joined #amaranth-lang
_catircservices has quit [Client Quit]
_catircservices has joined #amaranth-lang
_catircservices has quit [Client Quit]
_catircservices has joined #amaranth-lang
whitequark[cis] has joined #amaranth-lang
<whitequark[cis]> it's copied from migen, IIRC
jfng[m] has joined #amaranth-lang
<jfng[m]> the alternative would be to use ceil(log2(n)), and explicitly handle 0 and negative values, if needed
<jfng[m]> what was the rationale for having our own `log2_int()` helper? it seems to be a custom log2 that is also:... (full message at <https://catircservices.org/_matrix/media/v3/download/catircservices.org/UzDjVyrOqrAFylWNBrelJPKL>)
<jfng[m]> it is only used in `lib.fifo` ... which ends up check for n=0 in the caller
<jfng[m]> > <@jfng:matrix.org> it is only used in `lib.fifo` ... which ends up check for n=0 in the caller
<jfng[m]> and probably in a bunch of user code, also
<whitequark[cis]> I think it is now much less needed now that Signal(range(...)) is a thing
<whitequark[cis]> want to write an RFC for deprecating it?
<jfng[m]> yep
<josuah> > it's copied from migen, IIRC
<josuah> I did see it in migen code lately, so I second that!
jjsuperpower has joined #amaranth-lang
jjsuperpower has quit [Ping timeout: 260 seconds]
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283545700
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283548869
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283547249
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283551737
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283550783
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283551831
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283551925
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283594432
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283627181
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283633346
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283635536
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283639722
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283649854
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283652714
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283652881
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283653311
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283656979
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283659271
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283659358
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283659465
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283659560
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283660776
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283668915
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283692700
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283698255
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283701065
feldim2425 has quit [Quit: ZNC 1.8.2+deb2build5 - https://znc.in]
feldim2425 has joined #amaranth-lang
pbsds has quit [Ping timeout: 260 seconds]
pbsds has joined #amaranth-lang
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283740426
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283744049
notgull has joined #amaranth-lang
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283791799
<_whitenotifier-8> [amaranth] bl0x reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283792438
jjsuperpower has joined #amaranth-lang
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283813362
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283813603
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283813912
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283814242
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283816450
<_whitenotifier-8> [amaranth] whitequark reviewed pull request #734 commit - https://github.com/amaranth-lang/amaranth/pull/734#discussion_r1283816703