* [PATCH] Testsuite: Disable micromips for MSA tests
@ 2023-02-21 2:39 Xin Liu
2023-03-11 15:42 ` Jeff Law
0 siblings, 1 reply; 4+ messages in thread
From: Xin Liu @ 2023-02-21 2:39 UTC (permalink / raw)
To: gcc-patches; +Cc: Matthew Fortune
[-- Attachment #1: Type: text/plain, Size: 793 bytes --]
From: Matthew Fortune <matthew.fortune@imgtec.com>
---
gcc/testsuite/gcc.target/mips/mips.exp | 1 +
1 file changed, 1 insertion(+)
diff --git a/gcc/testsuite/gcc.target/mips/mips.exp b/gcc/testsuite/gcc.target/mips/mips.exp
index 81e19f39853..bf32fe0c93f 100644
--- a/gcc/testsuite/gcc.target/mips/mips.exp
+++ b/gcc/testsuite/gcc.target/mips/mips.exp
@@ -1463,6 +1463,7 @@ proc mips-dg-options { args } {
mips_option_dependency options "-msoft-float" "-mno-paired-single"
mips_option_dependency options "-mno-paired-single" "-mno-mips3d"
+ mips_option_dependency options "-mmsa" "-mno-micromips"
mips_option_dependency options "-mmsa" "-mno-mips16"
# If the test requires an unsupported option, change run tests
# to link tests.
--
2.30.2
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] Testsuite: Disable micromips for MSA tests
2023-02-21 2:39 [PATCH] Testsuite: Disable micromips for MSA tests Xin Liu
@ 2023-03-11 15:42 ` Jeff Law
2023-03-14 5:46 ` Xin Liu
0 siblings, 1 reply; 4+ messages in thread
From: Jeff Law @ 2023-03-11 15:42 UTC (permalink / raw)
To: Xin Liu, gcc-patches; +Cc: Matthew Fortune
On 2/20/23 19:39, Xin Liu wrote:
> From: Matthew Fortune <matthew.fortune@imgtec.com>
>
> ---
> gcc/testsuite/gcc.target/mips/mips.exp | 1 +
> 1 file changed, 1 insertion(+)
Unfortunately, you haven't given anyone any background that would allow
them to evaluate/review this patch.
I'm guessing that MSA does not support micromips, is that correct?
And for the future, please include a ChangeLog entry with patches. A
ChangeLog entry describes what changed, not why something changes. A
reasonable ChangeLog for this patch might be:
gcc/testsuite:
* gcc.target/mips/mips.exp (mips-dg-options): Disable micromips
for MSA tests.
Jeff
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] Testsuite: Disable micromips for MSA tests
2023-03-11 15:42 ` Jeff Law
@ 2023-03-14 5:46 ` Xin Liu
2023-03-19 17:31 ` Jeff Law
0 siblings, 1 reply; 4+ messages in thread
From: Xin Liu @ 2023-03-14 5:46 UTC (permalink / raw)
To: Jeff Law, gcc-patches; +Cc: Matthew Fortune
[-- Attachment #1: Type: text/plain, Size: 1078 bytes --]
Thanks for your feedback. You're right that MicroMIPS doesn't support
MSA, so disabling micromips for MSA tests is a reasonable change.
I'll make sure to include a ChangeLog entry with a clear description of
future patches. Thanks for the suggestions, and I'll strive to improve
my work based on your feedback.
On 2023/3/11 23:42, Jeff Law wrote:
>
>
> On 2/20/23 19:39, Xin Liu wrote:
>> From: Matthew Fortune <matthew.fortune@imgtec.com>
>>
>> ---
>> gcc/testsuite/gcc.target/mips/mips.exp | 1 +
>> 1 file changed, 1 insertion(+)
> Unfortunately, you haven't given anyone any background that would
> allow them to evaluate/review this patch.
>
> I'm guessing that MSA does not support micromips, is that correct?
>
> And for the future, please include a ChangeLog entry with patches. A
> ChangeLog entry describes what changed, not why something changes. A
> reasonable ChangeLog for this patch might be:
>
> gcc/testsuite:
> * gcc.target/mips/mips.exp (mips-dg-options): Disable micromips
> for MSA tests.
>
> Jeff
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] Testsuite: Disable micromips for MSA tests
2023-03-14 5:46 ` Xin Liu
@ 2023-03-19 17:31 ` Jeff Law
0 siblings, 0 replies; 4+ messages in thread
From: Jeff Law @ 2023-03-19 17:31 UTC (permalink / raw)
To: Xin Liu, gcc-patches; +Cc: Matthew Fortune
On 3/13/23 23:46, Xin Liu wrote:
> Thanks for your feedback. You're right that MicroMIPS doesn't support
> MSA, so disabling micromips for MSA tests is a reasonable change.
> I'll make sure to include a ChangeLog entry with a clear description of
> future patches. Thanks for the suggestions, and I'll strive to improve
> my work based on your feedback.
THanks. I've pushed your patch to the trunk.
jeff
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-03-19 17:31 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-21 2:39 [PATCH] Testsuite: Disable micromips for MSA tests Xin Liu
2023-03-11 15:42 ` Jeff Law
2023-03-14 5:46 ` Xin Liu
2023-03-19 17:31 ` Jeff Law
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).