public inbox for gcc-cvs@sourceware.org help / color / mirror / Atom feed
From: Michael Meissner <meissner@gcc.gnu.org> To: gcc-cvs@gcc.gnu.org Subject: [gcc(refs/users/meissner/heads/work076)] Revert patch. Date: Thu, 20 Jan 2022 19:35:14 +0000 (GMT) [thread overview] Message-ID: <20220120193514.A22093858D37@sourceware.org> (raw) https://gcc.gnu.org/g:6fa29a1bc39077ad22b14df2f6a6d5b83e7eacf3 commit 6fa29a1bc39077ad22b14df2f6a6d5b83e7eacf3 Author: Michael Meissner <meissner@linux.ibm.com> Date: Thu Jan 20 14:34:35 2022 -0500 Revert patch. gcc/ 2022-01-20 Michael Meissner <meissner@linux.ibm.com> Revert patch. PR target/104136 * config/rs6000/rs6000.cc (rs6000_final_prescan_insn): Do not prepend a 'p' for xxspltiw and xxspltidp. * config/rs6000/rs6000.md (maybe_prefixed): Indicate that vector permute instructions might be prefixed. Diff: --- gcc/ChangeLog.meissner | 10 +--------- gcc/config/rs6000/rs6000.cc | 7 ++----- gcc/config/rs6000/rs6000.md | 2 +- 3 files changed, 4 insertions(+), 15 deletions(-) diff --git a/gcc/ChangeLog.meissner b/gcc/ChangeLog.meissner index 9c40e20ff7f..9fe603ce991 100644 --- a/gcc/ChangeLog.meissner +++ b/gcc/ChangeLog.meissner @@ -5,15 +5,7 @@ work076 patch #3 compiler used to build the current compiler defaults to IEEE 128-bit long double, make that the default for this build. -work076 patch #1 -2022-01-20 Michael Meissner <meissner@linux.ibm.com> - - PR target/104136 - * config/rs6000/rs6000.cc (rs6000_final_prescan_insn): Do not - prepend a 'p' for xxspltiw and xxspltidp. - * config/rs6000/rs6000.md (maybe_prefixed): Indicate that vector - permute instructions might be prefixed. - +work076 patch #1 reverted. 2022-01-19 Michael Meissner <meissner@linux.ibm.com> diff --git a/gcc/config/rs6000/rs6000.cc b/gcc/config/rs6000/rs6000.cc index 0a9d62b3dfd..b34962da27d 100644 --- a/gcc/config/rs6000/rs6000.cc +++ b/gcc/config/rs6000/rs6000.cc @@ -26661,9 +26661,7 @@ static bool prepend_p_to_next_insn; /* Define FINAL_PRESCAN_INSN if some processing needs to be done before outputting the assembler code. On the PowerPC, we remember if the current - insn is a prefixed insn where we need to emit a 'p' before the insn. We do - not do this for permute type operations (such as XXSPLTIW or XXSPLTIDP), - since those operations do not have a non-prefixed format. + insn is a prefixed insn where we need to emit a 'p' before the insn. In addition, if the insn is part of a PC-relative reference to an external label optimization, this is recorded also. */ @@ -26672,8 +26670,7 @@ rs6000_final_prescan_insn (rtx_insn *insn, rtx [], int) { prepend_p_to_next_insn = (get_attr_maybe_prefixed (insn) == MAYBE_PREFIXED_YES - && get_attr_prefixed (insn) == PREFIXED_YES - && get_attr_type (insn) != TYPE_VECPERM); + && get_attr_prefixed (insn) == PREFIXED_YES); return; } diff --git a/gcc/config/rs6000/rs6000.md b/gcc/config/rs6000/rs6000.md index 890edfc5318..59531b6d07e 100644 --- a/gcc/config/rs6000/rs6000.md +++ b/gcc/config/rs6000/rs6000.md @@ -287,7 +287,7 @@ ;; Whether this insn has a prefixed form and a non-prefixed form. (define_attr "maybe_prefixed" "no,yes" (if_then_else (eq_attr "type" "load,fpload,vecload,store,fpstore,vecstore, - integer,add,vecperm") + integer,add") (const_string "yes") (const_string "no")))
reply other threads:[~2022-01-20 19:35 UTC|newest] Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20220120193514.A22093858D37@sourceware.org \ --to=meissner@gcc.gnu.org \ --cc=gcc-cvs@gcc.gnu.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).