On Mon, 2025-11-17 at 20:04 -0500, Joel Fernandes wrote:
> I don't care about the if/else as much as I care about the opportunity to
> just specify FalconLoadTarget once instead twice. I think the match here is
> cleaner for this
> snippet, but I am Ok with the if/else as well.
>
> Something like:
> imem_sec_load_target: FalconLoadTarget {
> src_start: if chipset > Chipset::GA100 {
> app0.offset
> } else {
> load_hdr.os_code_size
> },
>
> That would be one more line of code, but pretty much the same.
Interesting. I would have thought that duplicating the if-statement is the
higher offense.
