On 10/3/24 5:40 PM, Maciej W. Rozycki wrote:
On Thu, 3 Oct 2024, Jeff Law wrote:
We can remove a couple of XFAILs in the rv32 space as it's behaving much more
like rv64 at this point.
I'm glad to see them gone. I have a couple of concerns with your change
though.
Given:
* gcc.target/riscv/cset-sext.c: Similarly. No longer allow
"not" in asm output.
and:
+/* { dg-final { scan-assembler-not "\\sneg\\s" } } */
Clearly I typo'd in one or both of those two contexts :-) I'll figure
out which and fix.
More importantly may I ask you to review the second paragraph of commit
6c3365e715fa ("RISC-V: Also handle sign extension in branch costing") to
see if any of the other issues referred there have also been now sorted
and mention that in the change description, possibly with a commit hash
reference to Andrew P's recent improvements? And in particular can the
branch costs requested be lowered for gcc.target/riscv/cset-sext.c now?
Sure, that's easy enough to check and take appropriate action.
Jeff
