[clang][bytecode] VisitBuiltinCallExpr - remove urem() call in rotate builtins, called rotl/rotr directly

[rdez13]

Fixes #162046

[llvmbot]

@llvm/pr-subscribers-clang

Author: None (rdez13)

Changes

Fixes #162046

---

Full diff: https://github.com/llvm/llvm-project/pull/162113.diff

1 Files Affected:

• (modified) clang/lib/AST/ExprConstant.cpp (+2-2)

diff --git a/clang/lib/AST/ExprConstant.cpp b/clang/lib/AST/ExprConstant.cpp
index 7bf28d988f405..a9fedc1df56ce 100644
--- a/clang/lib/AST/ExprConstant.cpp
+++ b/clang/lib/AST/ExprConstant.cpp
@@ -14265,7 +14265,7 @@ bool IntExprEvaluator::VisitBuiltinCallExpr(const CallExpr *E,
         !EvaluateInteger(E->getArg(1), Amt, Info))
       return false;
 
-    return Success(Val.rotl(Amt.urem(Val.getBitWidth())), E);
+    return Success(Val.rotl(Amt), E);
   }
 
   case Builtin::BI__builtin_rotateright8:
@@ -14282,7 +14282,7 @@ bool IntExprEvaluator::VisitBuiltinCallExpr(const CallExpr *E,
         !EvaluateInteger(E->getArg(1), Amt, Info))
       return false;
 
-    return Success(Val.rotr(Amt.urem(Val.getBitWidth())), E);
+    return Success(Val.rotr(Amt), E);
   }
 
   case Builtin::BI__builtin_elementwise_add_sat: {

[rdez13]

Hey @RKSimon, can I please get a review. Thanks

[RKSimon]

LGTM

[rdez13]

Thanks for the review and merge!