On Tue, Jul 25, 2023 at 12:45 PM Jakub Jelinek via Gcc-patches <gcc-patches@gcc.gnu.org> wrote: > > On Tue, Jul 25, 2023 at 03:42:21PM -0400, David Edelsohn via Gcc-patches > wrote: > > Hi, Drew > > > > Thanks for addressing this missed optimization. > > > > The testcase includes an incorrect assumption: signed char, which > > causes the testcase to fail on PowerPC. > > > > Should the testcase be updated to specify signed char in the function > > signatures or should -fsigned-char be added to the command line > > options? > > I think we should use signed char instead of char in the testcase.
I also think it should be `signed char` instead as I mentioned in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110803 . Thanks, Andrew > > Jakub >
