From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1363) id 956F23836419; Fri, 14 Jan 2022 15:06:09 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 956F23836419 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="utf-8" From: Uros Bizjak To: gcc-cvs@gcc.gnu.org Subject: [gcc r12-6587] i386: Mark some of strict_low_part insn constraints earlyclobbered X-Act-Checkin: gcc X-Git-Author: Uros Bizjak X-Git-Refname: refs/heads/master X-Git-Oldrev: 74abb0beb420830e52dfc6b3ee74e77dae8e31a3 X-Git-Newrev: ffb7d4b2b76746e4189979f9d27d80be2195308a Message-Id: <20220114150609.956F23836419@sourceware.org> Date: Fri, 14 Jan 2022 15:06:09 +0000 (GMT) X-BeenThere: gcc-cvs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 14 Jan 2022 15:06:09 -0000 https://gcc.gnu.org/g:ffb7d4b2b76746e4189979f9d27d80be2195308a commit r12-6587-gffb7d4b2b76746e4189979f9d27d80be2195308a Author: Uros Bizjak Date: Fri Jan 14 16:05:17 2022 +0100 i386: Mark some of strict_low_part insn constraints earlyclobbered While there is practically impossible that input registers are matched with in-out register, better mark the output operand of the split alternative as earlyclobbered - we do output early to the output operand when the insn is split. 2022-01-14 Uroš Bizjak gcc/ChangeLog: * config/i386/i386.md (*add_1_slp"): Mark alternative 1 output operand earlyclobbered. (*sub_1_slp): Ditto. (*and_1_slp): Ditto. (*_1_slp): Ditto. (*neg_1_slp): Ditto. (*one_cmpl_1_slp): Ditto. (*ashl3_1_slp): Ditto. (*3_1_slp): Ditto. (*3_1_slp): Ditto. Diff: --- gcc/config/i386/i386.md | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/gcc/config/i386/i386.md b/gcc/config/i386/i386.md index 9477ca9b41d..7b1694314ea 100644 --- a/gcc/config/i386/i386.md +++ b/gcc/config/i386/i386.md @@ -5902,7 +5902,7 @@ ;; Alternative 1 is needed to work around LRA limitation, see PR82524. (define_insn_and_split "*add_1_slp" - [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,")) + [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,&")) (plus:SWI12 (match_operand:SWI12 1 "nonimmediate_operand" "%0,!") (match_operand:SWI12 2 "general_operand" "mn,mn"))) (clobber (reg:CC FLAGS_REG))] @@ -6856,7 +6856,7 @@ ;; Alternative 1 is needed to work around LRA limitation, see PR82524. (define_insn_and_split "*sub_1_slp" - [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,")) + [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,&")) (minus:SWI12 (match_operand:SWI12 1 "register_operand" "0,!") (match_operand:SWI12 2 "general_operand" "mn,mn"))) (clobber (reg:CC FLAGS_REG))] @@ -9905,7 +9905,7 @@ ;; Alternative 1 is needed to work around LRA limitation, see PR82524. (define_insn_and_split "*and_1_slp" - [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,")) + [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,&")) (and:SWI12 (match_operand:SWI12 1 "nonimmediate_operand" "%0,!") (match_operand:SWI12 2 "general_operand" "mn,mn"))) (clobber (reg:CC FLAGS_REG))] @@ -10542,7 +10542,7 @@ ;; Alternative 1 is needed to work around LRA limitation, see PR82524. (define_insn_and_split "*_1_slp" - [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,")) + [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,&")) (any_or:SWI12 (match_operand:SWI12 1 "nonimmediate_operand" "%0,!") (match_operand:SWI12 2 "general_operand" "mn,mn"))) (clobber (reg:CC FLAGS_REG))] @@ -10896,7 +10896,7 @@ ;; Alternative 1 is needed to work around LRA limitation, see PR82524. (define_insn_and_split "*neg_1_slp" - [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,")) + [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,&")) (neg:SWI12 (match_operand:SWI12 1 "register_operand" "0,!"))) (clobber (reg:CC FLAGS_REG))] "!TARGET_PARTIAL_REG_STALL || optimize_function_for_size_p (cfun)" @@ -11489,7 +11489,7 @@ ;; Alternative 1 is needed to work around LRA limitation, see PR82524. (define_insn_and_split "*one_cmpl_1_slp" - [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,")) + [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,&")) (not:SWI12 (match_operand:SWI12 1 "register_operand" "0,!")))] "!TARGET_PARTIAL_REG_STALL || optimize_function_for_size_p (cfun)" "@ @@ -12185,7 +12185,7 @@ ;; Alternative 1 is needed to work around LRA limitation, see PR82524. (define_insn_and_split "*ashl3_1_slp" - [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,")) + [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,&")) (ashift:SWI12 (match_operand:SWI12 1 "register_operand" "0,!") (match_operand:QI 2 "nonmemory_operand" "cI,cI"))) (clobber (reg:CC FLAGS_REG))] @@ -13062,7 +13062,7 @@ ;; Alternative 1 is needed to work around LRA limitation, see PR82524. (define_insn_and_split "*3_1_slp" - [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,")) + [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,&")) (any_shiftrt:SWI12 (match_operand:SWI12 1 "register_operand" "0,!") (match_operand:QI 2 "nonmemory_operand" "cI,cI"))) (clobber (reg:CC FLAGS_REG))] @@ -13606,7 +13606,7 @@ ;; Alternative 1 is needed to work around LRA limitation, see PR82524. (define_insn_and_split "*3_1_slp" - [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,")) + [(set (strict_low_part (match_operand:SWI12 0 "register_operand" "+,&")) (any_rotate:SWI12 (match_operand:SWI12 1 "register_operand" "0,!") (match_operand:QI 2 "nonmemory_operand" "cI,cI"))) (clobber (reg:CC FLAGS_REG))]