Committed to trunk.

Thanks,
Di Zhao

> -----Original Message-----
> From: Jeff Law <jeffreya...@gmail.com>
> Sent: Monday, September 30, 2024 6:28 AM
> To: Di Zhao OS <diz...@os.amperecomputing.com>
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH] [tree-optimization/110279] fix testcase pr110279-1.c
> 
> 
> 
> On 5/23/24 3:55 AM, Di Zhao OS wrote:
> >> -----Original Message-----
> >> From: Jeff Law <jeffreya...@gmail.com>
> >> Sent: Wednesday, May 22, 2024 11:14 PM
> >> To: Di Zhao OS <diz...@os.amperecomputing.com>; gcc-patches@gcc.gnu.org
> >> Subject: Re: [PATCH] [tree-optimization/110279] fix testcase pr110279-1.c
> >>
> >>
> >>
> >> On 5/22/24 5:46 AM, Di Zhao OS wrote:
> >>> The test case is for targets that support FMA. Previously
> >>> the "target" selector is missed in dg-final command.
> >>>
> >>> Tested on x86_64-pc-linux-gnu.
> >>>
> >>> Thanks
> >>> Di Zhao
> >>>
> >>> gcc/testsuite/ChangeLog:
> >>>
> >>>           * gcc.dg/pr110279-1.c: add target selector.
> >> Rather than list targets explicitly in the test, wouldn't it be better
> >> to have a common routine that could be used in other cases where we have
> >> a test that requires FMA?
> >>
> >> So something similar to check_effective_target_scalar_all_fma?
> >>
> >>
> >> Jeff
> >
> > Here is an updated version of the patch. Sorry I'm not very familiar
> > with the testsuite commands.
> >
> > gcc/testsuite/ChangeLog:
> >
> >          * gcc.dg/pr110279-1.c: add target selector.
> This is OK.   Thanks for your patience.  SOrry it's taken so long.  Just
> never seemed to get near the top of my todo list.
> 
> Ideally we'll see maintainers adjust the scalar_all_fma selector and if
> we see other targets that have some fma, but not the full set, then
> we'll see a new target selector for the subsets of fma.
> 
> jeff

Reply via email to