[PATCH] Testsuite: Disable micromips for MSA tests

[PATCH] Testsuite: Disable micromips for MSA tests

[Xin Liu]

[-- 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

Re: [PATCH] Testsuite: Disable micromips for MSA tests

[Jeff Law]

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

Re: [PATCH] Testsuite: Disable micromips for MSA tests

[Xin Liu]

[-- 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

Re: [PATCH] Testsuite: Disable micromips for MSA tests

[Jeff Law]

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
>