aboutsummaryrefslogtreecommitdiff
path: root/gcc/fortran
diff options
context:
space:
mode:
authorAlex Coplan <alex.coplan@arm.com>2025-03-05 15:45:09 +0000
committerAlex Coplan <alex.coplan@arm.com>2025-03-06 11:37:00 +0000
commitd6d7da92fb598c50d25332011bffe6b6515c9669 (patch)
tree8c78f08940570b44c286155b95ec60a19271d7c1 /gcc/fortran
parentde231924b73bc120bf2b7ada4eeccd884c249ee1 (diff)
downloadgcc-d6d7da92fb598c50d25332011bffe6b6515c9669.zip
gcc-d6d7da92fb598c50d25332011bffe6b6515c9669.tar.gz
gcc-d6d7da92fb598c50d25332011bffe6b6515c9669.tar.bz2
pair-fusion: Add singleton move_range asserts [PR114492]
The PR claims that pair-fusion has invalid uses of gcc_assert (such that the pass will misbehave with --disable-checking). As noted in the comments, in the case of the calls to restrict_movement, the only way we can possibly depend on the side effects is if we call it with a non-singleton move range. However, the intent is that we always have a singleton move range here, and thus we do not rely on the side effects. This patch therefore adds asserts to check for a singleton move range before calling restrict_movement, thus clarifying the intent and hopefully dispelling any concerns that having the calls wrapped in asserts is problematic here. gcc/ChangeLog: PR rtl-optimization/114492 * pair-fusion.cc (pair_fusion_bb_info::fuse_pair): Check for singleton move range before calling restrict_movement. (pair_fusion::try_promote_writeback): Likewise.
Diffstat (limited to 'gcc/fortran')
0 files changed, 0 insertions, 0 deletions