public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [Committed] Handle COPYSIGN in dwarf2out.cc'd mem_loc_descriptor
@ 2023-07-06  9:00 Roger Sayle
  0 siblings, 0 replies; only message in thread
From: Roger Sayle @ 2023-07-06  9:00 UTC (permalink / raw)
  To: gcc-patches; +Cc: 'Hans-Peter Nilsson'

[-- Attachment #1: Type: text/plain, Size: 733 bytes --]


Many thanks to Hans-Peter Nilsson for reminding me that new RTX codes
need to be added to dwarf2out.cc's mem_loc_descriptor, and for doing
this for BITREVERSE.  This patch does the same for the recently added
COPYSIGN.  I'd been testing these on a target that doesn't use DWARF
(nvptx-none) and so didn't exhibit the issue, and my additional testing
on x86_64-pc-linux-gnu to double check that changes were safe, doesn't
(yet) trigger the problematic assert in dwarf2out.cc's mem_loc_descriptor.

Committed to mainline as obvious, after bootstrapping and regression
testing on x86_64-pc-linux-gnu.


2023-07-06  Roger Sayle  <roger@nextmovesoftware.com>

gcc/ChangeLog
        * dwarf2out.cc (mem_loc_descriptor): Handle COPYSIGN.


[-- Attachment #2: patchhp.txt --]
[-- Type: text/plain, Size: 341 bytes --]

diff --git a/gcc/dwarf2out.cc b/gcc/dwarf2out.cc
index e973644..238d0a9 100644
--- a/gcc/dwarf2out.cc
+++ b/gcc/dwarf2out.cc
@@ -16941,6 +16941,7 @@ mem_loc_descriptor (rtx rtl, machine_mode mode,
     case SMUL_HIGHPART:
     case UMUL_HIGHPART:
     case BITREVERSE:
+    case COPYSIGN:
       break;
 
     case CONST_STRING:

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2023-07-06  9:00 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-06  9:00 [Committed] Handle COPYSIGN in dwarf2out.cc'd mem_loc_descriptor Roger Sayle

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).