opt_dff: fix sequence point copy paste bug
authorAustin Seipp <aseipp@pobox.com>
Tue, 4 Jan 2022 16:49:54 +0000 (10:49 -0600)
committerMarcelina Kościelnicka <mwk@0x04.net>
Tue, 4 Jan 2022 17:18:08 +0000 (18:18 +0100)
commitb022fe61a7e0cbe93124a5db243083e65f51fd85
tree24f42af92cace7939a5ed19db66a9425cae96f9b
parent493b5e03e7ce9496980c9235bb2979bd259995a6
opt_dff: fix sequence point copy paste bug

Newer GCCs emit the following warning for opt_dff:

    passes/opt/opt_dff.cc:560:17: warning: operation on ‘ff.Yosys::FfData::has_clk’ may be undefined [-Wsequence-point]
      560 |      ff.has_clk = ff.has_ce = ff.has_clk = false;
          |      ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Which is correct: the order of whether the read or write of has_clk
occurs first is undefined since there is no sequence point between them.

This is almost certainly just a typo/copy paste error and objectively
wrong, so just fix it.

Signed-off-by: Austin Seipp <aseipp@pobox.com>
passes/opt/opt_dff.cc