Dinar Temirbulatov <dtemirbula...@gmail.com> writes: > I found typo in the patch instead of checking *set_after != 0 it was > set_after != 0, here is corrected version of patch. I retested the > patch without typo on mipsel-unknown-linux-gnu with no new > regressions.
But my point was that *set_after should always be nonnull if *delayed_reg is. So we shouldn't need that check anyway. The patch looks OK otherwise apart from formatting (the formatting should be the same as in the version I suggested before: else if (*delayed_reg != 0 && reg_referenced_p (*delayed_reg, pattern) && !(recog_memoized (insn) == CODE_FOR_load_highdf && recog_memoized (*set_after) == CODE_FOR_load_lowdf)) ) but it needs a copyright assignment. Richard