[clang] [CIR][X86]Implement handling for Select/Selectsh builtins in CIR (PR #174003)

[Priyanshu Kumar via cfe-commits]

Priyanshu3820 wrote:

> > why does copilot get requested for review automatically? Is it due to something from my side?
> 
> I wondered the same thing. I have only seen it on your PRs. Project-wide it appears to be happening [elsewhere too](https://github.com/llvm/llvm-project/pulls?q=is%3Apr+reviewed-by%3A%40Copilot+is%3Aopen+), but in the cases I spot-checked it appears to have been explicitly requested by the author of the PR. Do you have a Copilot Pro subscription? If so, that's probably the cause, and you can choose to disable it.
> 
Yes, I have the github education pack, but I had disabled it the last time it was requested for review. Don't know why it got requested again.
> I think it's fine, maybe even useful, to have Copilot as a reviewer so long as you don't accept its suggestions uncritically. It made a few suggestions in one of your previous PRs that were wrong. The suggestions here seem reasonable.

I too feel so, it is good to have it as a reviewer to some extent. Although it seems to be a bit outdated as it had suggested a change to builder.create<...> in the previous pr.


https://github.com/llvm/llvm-project/pull/174003