https://github.com/DavidTruby commented:

Sorry it took me so long to review this!

> The SVE support generates unsigned comparison predicates for comparisons of 
> signed types. This looks very unintuitive to me, so I changed it to use 
> signed comparisons. I think the SVE case may just be a bug?

Looks like I made a mistake here when I originally wrote this for SVE, thanks 
for pointing it out! It looks like someone has already fixed this in clang 
trunk since, thankfully.

> Originally written by Claude and heavily evolved by me

Could you add an `Assisted-by:` tag to the PR description as per the [AI Tool 
Use Policy](https://llvm.org/docs/AIToolPolicy.html)? Thanks!

https://github.com/llvm/llvm-project/pull/192722
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to