* [PATCH] LoongArch: Enable -fsched-pressure by default at -O1 and higher.
@ 2023-09-08 2:00 Guo Jie
2023-09-08 8:06 ` Xi Ruoyao
0 siblings, 1 reply; 5+ messages in thread
From: Guo Jie @ 2023-09-08 2:00 UTC (permalink / raw)
To: gcc-patches; +Cc: xuchenghua, chenglulu, i, xry111, Guo Jie
gcc/ChangeLog:
* common/config/loongarch/loongarch-common.cc:
(default_options loongarch_option_optimization_table):
Default to -fsched-pressure.
---
gcc/common/config/loongarch/loongarch-common.cc | 1 +
1 file changed, 1 insertion(+)
diff --git a/gcc/common/config/loongarch/loongarch-common.cc b/gcc/common/config/loongarch/loongarch-common.cc
index c5ed37d27a6..b6901910b70 100644
--- a/gcc/common/config/loongarch/loongarch-common.cc
+++ b/gcc/common/config/loongarch/loongarch-common.cc
@@ -36,6 +36,7 @@ static const struct default_options loongarch_option_optimization_table[] =
{ OPT_LEVELS_ALL, OPT_fasynchronous_unwind_tables, NULL, 1 },
{ OPT_LEVELS_1_PLUS, OPT_fsection_anchors, NULL, 1 },
{ OPT_LEVELS_2_PLUS, OPT_free, NULL, 1 },
+ { OPT_LEVELS_1_PLUS, OPT_fsched_pressure, NULL, 1 },
{ OPT_LEVELS_NONE, 0, NULL, 0 }
};
--
2.20.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] LoongArch: Enable -fsched-pressure by default at -O1 and higher.
2023-09-08 2:00 [PATCH] LoongArch: Enable -fsched-pressure by default at -O1 and higher Guo Jie
@ 2023-09-08 8:06 ` Xi Ruoyao
2023-09-08 8:29 ` Guo Jie
0 siblings, 1 reply; 5+ messages in thread
From: Xi Ruoyao @ 2023-09-08 8:06 UTC (permalink / raw)
To: Guo Jie, gcc-patches; +Cc: xuchenghua, chenglulu, i
On Fri, 2023-09-08 at 10:00 +0800, Guo Jie wrote:
> gcc/ChangeLog:
>
> * common/config/loongarch/loongarch-common.cc:
"common/" should be removed. You can use "git gcc-verify" to figure out
this kind of error before sending a patch in the future.
> (default_options loongarch_option_optimization_table):
> Default to -fsched-pressure.
"Default to -fsched-pressure at -O1 or above."
Otherwise OK.
> ---
> gcc/common/config/loongarch/loongarch-common.cc | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/gcc/common/config/loongarch/loongarch-common.cc
> b/gcc/common/config/loongarch/loongarch-common.cc
> index c5ed37d27a6..b6901910b70 100644
> --- a/gcc/common/config/loongarch/loongarch-common.cc
> +++ b/gcc/common/config/loongarch/loongarch-common.cc
> @@ -36,6 +36,7 @@ static const struct default_options
> loongarch_option_optimization_table[] =
> { OPT_LEVELS_ALL, OPT_fasynchronous_unwind_tables, NULL, 1 },
> { OPT_LEVELS_1_PLUS, OPT_fsection_anchors, NULL, 1 },
> { OPT_LEVELS_2_PLUS, OPT_free, NULL, 1 },
> + { OPT_LEVELS_1_PLUS, OPT_fsched_pressure, NULL, 1 },
> { OPT_LEVELS_NONE, 0, NULL, 0 }
> };
>
--
Xi Ruoyao <xry111@xry111.site>
School of Aerospace Science and Technology, Xidian University
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] LoongArch: Enable -fsched-pressure by default at -O1 and higher.
2023-09-08 8:06 ` Xi Ruoyao
@ 2023-09-08 8:29 ` Guo Jie
2023-09-08 8:32 ` Xi Ruoyao
0 siblings, 1 reply; 5+ messages in thread
From: Guo Jie @ 2023-09-08 8:29 UTC (permalink / raw)
To: Xi Ruoyao, gcc-patches; +Cc: xuchenghua, chenglulu, i
Hi,
What I wanna change is "gcc/common/config/loongarch/loongarch-common.cc",
and the patch is automatically generated by "git gcc-commit-mklog".
Is it necessary to to remove "common/" ?
Thanks for the review.
在 2023/9/8 下午4:06, Xi Ruoyao 写道:
> On Fri, 2023-09-08 at 10:00 +0800, Guo Jie wrote:
>> gcc/ChangeLog:
>>
>> * common/config/loongarch/loongarch-common.cc:
> "common/" should be removed. You can use "git gcc-verify" to figure out
> this kind of error before sending a patch in the future.
>
>> (default_options loongarch_option_optimization_table):
>> Default to -fsched-pressure.
> "Default to -fsched-pressure at -O1 or above."
>
> Otherwise OK.
>
>> ---
>> gcc/common/config/loongarch/loongarch-common.cc | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/gcc/common/config/loongarch/loongarch-common.cc
>> b/gcc/common/config/loongarch/loongarch-common.cc
>> index c5ed37d27a6..b6901910b70 100644
>> --- a/gcc/common/config/loongarch/loongarch-common.cc
>> +++ b/gcc/common/config/loongarch/loongarch-common.cc
>> @@ -36,6 +36,7 @@ static const struct default_options
>> loongarch_option_optimization_table[] =
>> { OPT_LEVELS_ALL, OPT_fasynchronous_unwind_tables, NULL, 1 },
>> { OPT_LEVELS_1_PLUS, OPT_fsection_anchors, NULL, 1 },
>> { OPT_LEVELS_2_PLUS, OPT_free, NULL, 1 },
>> + { OPT_LEVELS_1_PLUS, OPT_fsched_pressure, NULL, 1 },
>> { OPT_LEVELS_NONE, 0, NULL, 0 }
>> };
>>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] LoongArch: Enable -fsched-pressure by default at -O1 and higher.
2023-09-08 8:29 ` Guo Jie
@ 2023-09-08 8:32 ` Xi Ruoyao
2023-09-08 9:54 ` chenglulu
0 siblings, 1 reply; 5+ messages in thread
From: Xi Ruoyao @ 2023-09-08 8:32 UTC (permalink / raw)
To: Guo Jie, gcc-patches; +Cc: xuchenghua, chenglulu, i
On Fri, 2023-09-08 at 16:29 +0800, Guo Jie wrote:
> Hi,
>
> What I wanna change is "gcc/common/config/loongarch/loongarch-
> common.cc",
>
> and the patch is automatically generated by "git gcc-commit-mklog".
>
> Is it necessary to to remove "common/" ?
My bad. I didn't realized the file has been moved to common.
Don't change it :(.
> Thanks for the review.
>
>
> 在 2023/9/8 下午4:06, Xi Ruoyao 写道:
> > On Fri, 2023-09-08 at 10:00 +0800, Guo Jie wrote:
> > > gcc/ChangeLog:
> > >
> > > * common/config/loongarch/loongarch-common.cc:
> > "common/" should be removed. You can use "git gcc-verify" to figure
> > out
> > this kind of error before sending a patch in the future.
> >
> > > (default_options loongarch_option_optimization_table):
> > > Default to -fsched-pressure.
> > "Default to -fsched-pressure at -O1 or above."
> >
> > Otherwise OK.
> >
> > > ---
> > > gcc/common/config/loongarch/loongarch-common.cc | 1 +
> > > 1 file changed, 1 insertion(+)
> > >
> > > diff --git a/gcc/common/config/loongarch/loongarch-common.cc
> > > b/gcc/common/config/loongarch/loongarch-common.cc
> > > index c5ed37d27a6..b6901910b70 100644
> > > --- a/gcc/common/config/loongarch/loongarch-common.cc
> > > +++ b/gcc/common/config/loongarch/loongarch-common.cc
> > > @@ -36,6 +36,7 @@ static const struct default_options
> > > loongarch_option_optimization_table[] =
> > > { OPT_LEVELS_ALL, OPT_fasynchronous_unwind_tables, NULL, 1 },
> > > { OPT_LEVELS_1_PLUS, OPT_fsection_anchors, NULL, 1 },
> > > { OPT_LEVELS_2_PLUS, OPT_free, NULL, 1 },
> > > + { OPT_LEVELS_1_PLUS, OPT_fsched_pressure, NULL, 1 },
> > > { OPT_LEVELS_NONE, 0, NULL, 0 }
> > > };
> > >
>
--
Xi Ruoyao <xry111@xry111.site>
School of Aerospace Science and Technology, Xidian University
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re:[pushed] [PATCH] LoongArch: Enable -fsched-pressure by default at -O1 and higher.
2023-09-08 8:32 ` Xi Ruoyao
@ 2023-09-08 9:54 ` chenglulu
0 siblings, 0 replies; 5+ messages in thread
From: chenglulu @ 2023-09-08 9:54 UTC (permalink / raw)
To: Xi Ruoyao, Guo Jie, gcc-patches; +Cc: xuchenghua, i
Pushed to r14-3805.
在 2023/9/8 下午4:32, Xi Ruoyao 写道:
> On Fri, 2023-09-08 at 16:29 +0800, Guo Jie wrote:
>> Hi,
>>
>> What I wanna change is "gcc/common/config/loongarch/loongarch-
>> common.cc",
>>
>> and the patch is automatically generated by "git gcc-commit-mklog".
>>
>> Is it necessary to to remove "common/" ?
> My bad. I didn't realized the file has been moved to common.
>
> Don't change it :(.
>
>> Thanks for the review.
>>
>>
>> 在 2023/9/8 下午4:06, Xi Ruoyao 写道:
>>> On Fri, 2023-09-08 at 10:00 +0800, Guo Jie wrote:
>>>> gcc/ChangeLog:
>>>>
>>>> * common/config/loongarch/loongarch-common.cc:
>>> "common/" should be removed. You can use "git gcc-verify" to figure
>>> out
>>> this kind of error before sending a patch in the future.
>>>
>>>> (default_options loongarch_option_optimization_table):
>>>> Default to -fsched-pressure.
>>> "Default to -fsched-pressure at -O1 or above."
>>>
>>> Otherwise OK.
>>>
>>>> ---
>>>> gcc/common/config/loongarch/loongarch-common.cc | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/gcc/common/config/loongarch/loongarch-common.cc
>>>> b/gcc/common/config/loongarch/loongarch-common.cc
>>>> index c5ed37d27a6..b6901910b70 100644
>>>> --- a/gcc/common/config/loongarch/loongarch-common.cc
>>>> +++ b/gcc/common/config/loongarch/loongarch-common.cc
>>>> @@ -36,6 +36,7 @@ static const struct default_options
>>>> loongarch_option_optimization_table[] =
>>>> { OPT_LEVELS_ALL, OPT_fasynchronous_unwind_tables, NULL, 1 },
>>>> { OPT_LEVELS_1_PLUS, OPT_fsection_anchors, NULL, 1 },
>>>> { OPT_LEVELS_2_PLUS, OPT_free, NULL, 1 },
>>>> + { OPT_LEVELS_1_PLUS, OPT_fsched_pressure, NULL, 1 },
>>>> { OPT_LEVELS_NONE, 0, NULL, 0 }
>>>> };
>>>>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-09-08 9:55 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-08 2:00 [PATCH] LoongArch: Enable -fsched-pressure by default at -O1 and higher Guo Jie
2023-09-08 8:06 ` Xi Ruoyao
2023-09-08 8:29 ` Guo Jie
2023-09-08 8:32 ` Xi Ruoyao
2023-09-08 9:54 ` chenglulu
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).