* sim prune_warnings
@ 2023-08-14 8:13 Alan Modra
2023-08-31 13:25 ` Tom Tromey
2023-10-15 9:38 ` Mike Frysinger
0 siblings, 2 replies; 5+ messages in thread
From: Alan Modra @ 2023-08-14 8:13 UTC (permalink / raw)
To: gdb-patches
Remove some of the warnings generated by newer versions of ld.
Without this patch "make check" in sim/ doesn't do much for some
targets.
OK?
* testsuite/lib/sim-defs.exp (prune_warnings_extra): New.
Arrange to run it from prune_warnings.
diff --git a/sim/testsuite/lib/sim-defs.exp b/sim/testsuite/lib/sim-defs.exp
index 5528d64684b..54860f2ea51 100644
--- a/sim/testsuite/lib/sim-defs.exp
+++ b/sim/testsuite/lib/sim-defs.exp
@@ -651,3 +651,22 @@ proc slurp_options { file } {
close $f
return $opt_array
}
+
+proc prune_warnings_extra { text } {
+
+ regsub -all "(^|\n)(\[^\n\]*: warning:\[^\n\]*missing \\.note\\.GNU-stack section\[^\n\]*\n?)+" $text "\\1" text
+ regsub -all "(^|\n)(\[^\n\]*: NOTE: This behaviour is deprecated\[^\n\]*\n?)+" $text "\\1" text
+
+ regsub -all "(^|\n)(\[^\n\]*: warning:\[^\n\]*has a LOAD segment with RWX permissions\[^\n\]*\n?)+" $text "\\1" text
+
+ return $text
+}
+
+if { [info procs saved-prune_warnings] == [list] } {
+ rename prune_warnings saved-prune_warnings
+ proc prune_warnings { text } {
+ set text [saved-prune_warnings $text]
+ set text [prune_warnings_extra $text]
+ return $text
+ }
+}
--
Alan Modra
Australia Development Lab, IBM
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: sim prune_warnings
2023-08-14 8:13 sim prune_warnings Alan Modra
@ 2023-08-31 13:25 ` Tom Tromey
2023-08-31 22:12 ` Alan Modra
2023-10-15 9:38 ` Mike Frysinger
1 sibling, 1 reply; 5+ messages in thread
From: Tom Tromey @ 2023-08-31 13:25 UTC (permalink / raw)
To: Alan Modra via Gdb-patches; +Cc: Alan Modra
>>>>> "Alan" == Alan Modra via Gdb-patches <gdb-patches@sourceware.org> writes:
Alan> Remove some of the warnings generated by newer versions of ld.
Alan> Without this patch "make check" in sim/ doesn't do much for some
Alan> targets.
Alan> OK?
Alan> * testsuite/lib/sim-defs.exp (prune_warnings_extra): New.
Alan> Arrange to run it from prune_warnings.
I didn't see a response to this, but IMO it seems fine.
Please go ahead.
thanks,
Tom
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: sim prune_warnings
2023-08-31 13:25 ` Tom Tromey
@ 2023-08-31 22:12 ` Alan Modra
2023-09-01 3:07 ` Tom Tromey
0 siblings, 1 reply; 5+ messages in thread
From: Alan Modra @ 2023-08-31 22:12 UTC (permalink / raw)
To: Tom Tromey; +Cc: Alan Modra via Gdb-patches
On Thu, Aug 31, 2023 at 07:25:50AM -0600, Tom Tromey wrote:
> >>>>> "Alan" == Alan Modra via Gdb-patches <gdb-patches@sourceware.org> writes:
>
> Alan> Remove some of the warnings generated by newer versions of ld.
> Alan> Without this patch "make check" in sim/ doesn't do much for some
> Alan> targets.
>
> Alan> OK?
>
> Alan> * testsuite/lib/sim-defs.exp (prune_warnings_extra): New.
> Alan> Arrange to run it from prune_warnings.
>
> I didn't see a response to this, but IMO it seems fine.
> Please go ahead.
I committed the patch on Aug 19, when pushing the sim regen series.
It seemed obvious enough to me. Without it I would have needed to
edit commit 9d4f36166d626 log to report a lot more fails.
--
Alan Modra
Australia Development Lab, IBM
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: sim prune_warnings
2023-08-31 22:12 ` Alan Modra
@ 2023-09-01 3:07 ` Tom Tromey
0 siblings, 0 replies; 5+ messages in thread
From: Tom Tromey @ 2023-09-01 3:07 UTC (permalink / raw)
To: Alan Modra; +Cc: Tom Tromey, Alan Modra via Gdb-patches
Alan> I committed the patch on Aug 19, when pushing the sim regen series.
Alan> It seemed obvious enough to me. Without it I would have needed to
Alan> edit commit 9d4f36166d626 log to report a lot more fails.
Ok, thanks. I sometimes reply to patches when I haven't seen any
traffic; I didn't think to check the git log.
Tom
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: sim prune_warnings
2023-08-14 8:13 sim prune_warnings Alan Modra
2023-08-31 13:25 ` Tom Tromey
@ 2023-10-15 9:38 ` Mike Frysinger
1 sibling, 0 replies; 5+ messages in thread
From: Mike Frysinger @ 2023-10-15 9:38 UTC (permalink / raw)
To: Alan Modra; +Cc: gdb-patches
[-- Attachment #1: Type: text/plain, Size: 597 bytes --]
On 14 Aug 2023 17:43, Alan Modra via Gdb-patches wrote:
> + regsub -all "(^|\n)(\[^\n\]*: warning:\[^\n\]*missing \\.note\\.GNU-stack section\[^\n\]*\n?)+" $text "\\1" text
> + regsub -all "(^|\n)(\[^\n\]*: warning:\[^\n\]*has a LOAD segment with RWX permissions\[^\n\]*\n?)+" $text "\\1" text
imo these are bugs that should be fixed rather than ignored.
i filed some for people to fixup. haven't kept track of the
overall status though.
> + regsub -all "(^|\n)(\[^\n\]*: NOTE: This behaviour is deprecated\[^\n\]*\n?)+" $text "\\1" text
not sure that i've seen this one before
-mike
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-10-15 9:38 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-08-14 8:13 sim prune_warnings Alan Modra
2023-08-31 13:25 ` Tom Tromey
2023-08-31 22:12 ` Alan Modra
2023-09-01 3:07 ` Tom Tromey
2023-10-15 9:38 ` Mike Frysinger
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).