[clang] [clang][x86][bytecode] Replace interp__builtin_rotate with static bool interp__builtin_elementwise_int_binop callback #160289 (PR #161924)

[Timm Baeder via cfe-commits]

================
@@ -3162,7 +3143,10 @@ bool InterpretBuiltin(InterpState &S, CodePtr OpPC, const CallExpr *Call,
   case Builtin::BI_rotl:
   case Builtin::BI_lrotl:
   case Builtin::BI_rotl64:
-    return interp__builtin_rotate(S, OpPC, Frame, Call, /*Right=*/false);
+    return interp__builtin_elementwise_int_binop(
+        S, OpPC, Call, [](const APSInt &A, const APSInt &B) -> APInt {
+          return A.rotl(B.urem(A.getBitWidth()));
----------------
tbaederr wrote:

@RKSimon We could open a follow-up issue to remove the `rem()` calls from the current interpreter.

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