This patch implements __builtin_reduce_xor as specified in D111529.
nit: the element type will be restricted to int, so maybe say something like only supports vectors of integers>
vector of integers type sounds a bit off. Not sure if that's much better, but how about dropping the type from the end?
Hi, thanks for take a look at this.
Yes, you're right. The reason I split them is that I'm a beginner, so I want to send a "experimental patch" first, this will make review work a little bit easier if I mess something up. I think it's reasonable to add all similar builtins if currently nothing wrong about this one.
LGTM aside from a nit. Please wait a day or two before landing in case other reviewers have concerns they haven't had the chance to raise yet.
If we're going to be renaming this, we should probably drop the Sema prefix from the name. (I'd note that we have a lot of functions prefixed with Sema that should be renamed, in case anyone really wants an NFC task that should be pretty trivial.)