* Re: oss-fuzz: binutils: Fuzzing build failure [not found] ` <E183C216-573D-46D3-AA24-371315273B70@catenacyber.fr> @ 2020-09-21 9:16 ` Alan Modra 2020-09-21 11:53 ` Philippe Antoine 0 siblings, 1 reply; 2+ messages in thread From: Alan Modra @ 2020-09-21 9:16 UTC (permalink / raw) To: Philippe Antoine; +Cc: binutils On Fri, Sep 18, 2020 at 09:14:42PM +0200, Philippe Antoine wrote: > Latest clang introduced a new warning again. > And it found a bug in binutils, preventing the oss-fuzz build Thanks, I committed a variation on your patch, trimming off trailing commas too. * rx-dis.c (flag_names): Add missing comma. (register_names, flag_names, double_register_names), (double_register_high_names, double_register_low_names), (double_control_register_names, double_condition_names): Remove trailing commas. diff --git a/opcodes/rx-dis.c b/opcodes/rx-dis.c index 29ba71c121..1e5591ca98 100644 --- a/opcodes/rx-dis.c +++ b/opcodes/rx-dis.c @@ -85,7 +85,7 @@ static char const * register_names[] = "psw", "pc", "usp", "fpsw", NULL, NULL, NULL, NULL, "bpsw", "bpc", "isp", "fintv", "intb", "extb", NULL, NULL, "a0", "a1", NULL, NULL, NULL, NULL, NULL, NULL, - NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, + NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL }; static char const * condition_names[] = @@ -99,36 +99,36 @@ static const char * flag_names[] = { "c", "z", "s", "o", "", "", "", "", "", "", "", "", "", "", "", "", - "i", "u", "", "", "", "", "", "" - "", "", "", "", "", "", "", "", + "i", "u", "", "", "", "", "", "", + "", "", "", "", "", "", "", "" }; static const char * double_register_names[] = { "dr0", "dr1", "dr2", "dr3", "dr4", "dr5", "dr6", "dr7", - "dr8", "dr9", "dr10", "dr11", "dr12", "dr13", "dr14", "dr15", + "dr8", "dr9", "dr10", "dr11", "dr12", "dr13", "dr14", "dr15" }; static const char * double_register_high_names[] = { "drh0", "drh1", "drh2", "drh3", "drh4", "drh5", "drh6", "drh7", - "drh8", "drh9", "drh10", "drh11", "drh12", "drh13", "drh14", "drh15", + "drh8", "drh9", "drh10", "drh11", "drh12", "drh13", "drh14", "drh15" }; static const char * double_register_low_names[] = { "drl0", "drl1", "drl2", "drl3", "drl4", "drl5", "drl6", "drl7", - "drl8", "drl9", "drl10", "drl11", "drl12", "drl13", "drl14", "drl15", + "drl8", "drl9", "drl10", "drl11", "drl12", "drl13", "drl14", "drl15" }; static const char * double_control_register_names[] = { - "dpsw", "dcmr", "decnt", "depc", + "dpsw", "dcmr", "decnt", "depc" }; static const char * double_condition_names[] = { - "", "un", "eq", "", "lt", "", "le", + "", "un", "eq", "", "lt", "", "le" }; static inline const char * -- Alan Modra Australia Development Lab, IBM ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: oss-fuzz: binutils: Fuzzing build failure 2020-09-21 9:16 ` oss-fuzz: binutils: Fuzzing build failure Alan Modra @ 2020-09-21 11:53 ` Philippe Antoine 0 siblings, 0 replies; 2+ messages in thread From: Philippe Antoine @ 2020-09-21 11:53 UTC (permalink / raw) To: Alan Modra; +Cc: binutils [-- Attachment #1: Type: text/plain, Size: 2699 bytes --] Thanks Alan > Le 21 sept. 2020 à 11:16, Alan Modra <amodra@gmail.com> a écrit : > > On Fri, Sep 18, 2020 at 09:14:42PM +0200, Philippe Antoine wrote: >> Latest clang introduced a new warning again. >> And it found a bug in binutils, preventing the oss-fuzz build > > Thanks, I committed a variation on your patch, trimming off trailing > commas too. > > * rx-dis.c (flag_names): Add missing comma. > (register_names, flag_names, double_register_names), > (double_register_high_names, double_register_low_names), > (double_control_register_names, double_condition_names): Remove > trailing commas. > > diff --git a/opcodes/rx-dis.c b/opcodes/rx-dis.c > index 29ba71c121..1e5591ca98 100644 > --- a/opcodes/rx-dis.c > +++ b/opcodes/rx-dis.c > @@ -85,7 +85,7 @@ static char const * register_names[] = > "psw", "pc", "usp", "fpsw", NULL, NULL, NULL, NULL, > "bpsw", "bpc", "isp", "fintv", "intb", "extb", NULL, NULL, > "a0", "a1", NULL, NULL, NULL, NULL, NULL, NULL, > - NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, > + NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL > }; > > static char const * condition_names[] = > @@ -99,36 +99,36 @@ static const char * flag_names[] = > { > "c", "z", "s", "o", "", "", "", "", > "", "", "", "", "", "", "", "", > - "i", "u", "", "", "", "", "", "" > - "", "", "", "", "", "", "", "", > + "i", "u", "", "", "", "", "", "", > + "", "", "", "", "", "", "", "" > }; > > static const char * double_register_names[] = > { > "dr0", "dr1", "dr2", "dr3", "dr4", "dr5", "dr6", "dr7", > - "dr8", "dr9", "dr10", "dr11", "dr12", "dr13", "dr14", "dr15", > + "dr8", "dr9", "dr10", "dr11", "dr12", "dr13", "dr14", "dr15" > }; > > static const char * double_register_high_names[] = > { > "drh0", "drh1", "drh2", "drh3", "drh4", "drh5", "drh6", "drh7", > - "drh8", "drh9", "drh10", "drh11", "drh12", "drh13", "drh14", "drh15", > + "drh8", "drh9", "drh10", "drh11", "drh12", "drh13", "drh14", "drh15" > }; > > static const char * double_register_low_names[] = > { > "drl0", "drl1", "drl2", "drl3", "drl4", "drl5", "drl6", "drl7", > - "drl8", "drl9", "drl10", "drl11", "drl12", "drl13", "drl14", "drl15", > + "drl8", "drl9", "drl10", "drl11", "drl12", "drl13", "drl14", "drl15" > }; > > static const char * double_control_register_names[] = > { > - "dpsw", "dcmr", "decnt", "depc", > + "dpsw", "dcmr", "decnt", "depc" > }; > > static const char * double_condition_names[] = > { > - "", "un", "eq", "", "lt", "", "le", > + "", "un", "eq", "", "lt", "", "le" > }; > > static inline const char * > > > -- > Alan Modra > Australia Development Lab, IBM [-- Attachment #2: smime.p7s --] [-- Type: application/pkcs7-signature, Size: 2740 bytes --] ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-09-21 11:53 UTC | newest] Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <0=1d79688ba06d41b624f807a991e6c9ee=59d48433e0764783d31f65a1dd0c2b85=oss-fuzz@monorail-prod.appspotmail.com> [not found] ` <0000000000001f161f0598158d92@google.com> [not found] ` <F580BE4E-75A4-493D-A5C1-DEE6E68071F3@catenacyber.fr> [not found] ` <411708c7-7c97-9a48-c072-1f1c1109c3c7@redhat.com> [not found] ` <0FBB84B3-27CB-4884-8AB5-0216EA447A28@catenacyber.fr> [not found] ` <ce754fc9-edcc-7ca0-9d76-ae16fece10f3@redhat.com> [not found] ` <E183C216-573D-46D3-AA24-371315273B70@catenacyber.fr> 2020-09-21 9:16 ` oss-fuzz: binutils: Fuzzing build failure Alan Modra 2020-09-21 11:53 ` Philippe Antoine
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).