On 7/11/24 1:20 AM, Jerry Zhang Jian wrote:
The original method tried to overwrite the march option when the target
doesn't support v exctension, which caused unexpected compile and
runtime test failures

This patch change the way to handle targets that don't support v
extension by simply skip the test cases that requires v extension

The test cases under g[cc|++].dg/vect/vect.exp will be skipped on rv64gc after 
this patch

gcc/testsuite/ChangeLog:

        * lib/target-supports.exp: skip vector tests if target not supporting v 
extension
But isn't the whole point of this code to inject the V flag so that we can run those tests, even on rv64gc?

Wouldn't it be better to understand why injection didn't work for you? Others certainly run testsuites on rv64gc without needing to skip the vect.exp tests.

jeff

Reply via email to