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