search

YosysHQ / nextpnr

nextpnr portable FPGA place and route tool
ISC License
1.24k stars 237 forks source link

Add trivially copyable invariant for ID types #1270

Closed gatecat closed 6 months ago

gatecat commented 6 months ago

This has always been a design intent of nextpnr, and is now strictly required by #1260.

Ravenslofty commented 6 months ago

...we should also have the invariant that sizeof({Bel,Pip,Wire}Id) <= 8) or else #1260 would only partially capture some of those types.

gatecat commented 6 months ago

True - but for now I would suggest putting that in the Rust bindings rather than here.