verified-network-toolchain / petr4

Petr4: Formal Semantics for P4
Apache License 2.0
74 stars 20 forks source link

size field in ValBaseStack #303

Closed QinshiWang closed 2 years ago

QinshiWang commented 2 years ago

Do we need the size field in ValBaseStack? Should we expect size to come from the length of headers?

hackedy commented 2 years ago

Hi, I see you removed the size field. I think we can just use the length of headers if we're careful to initialize it properly and never change it.