[clang][x86][bytecode] remove trailing returns type from interp__builtin_elementwise_int_unaryop callbacks

[donneypr]

Regarding the discussion in #162346, this PR is to remove the trailing type from the 'interp__builtin_elementwise_int_unaryop' callbacks.

[llvmbot]

@llvm/pr-subscribers-clang

Author: don (donneypr)

Changes

This was missed in my pr #162346 for #160288.

---

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

1 Files Affected:

• (modified) clang/lib/AST/ByteCode/InterpBuiltin.cpp (+1-1)

diff --git a/clang/lib/AST/ByteCode/InterpBuiltin.cpp b/clang/lib/AST/ByteCode/InterpBuiltin.cpp
index 922d67940e22f..517c51a5f7de1 100644
--- a/clang/lib/AST/ByteCode/InterpBuiltin.cpp
+++ b/clang/lib/AST/ByteCode/InterpBuiltin.cpp
@@ -3173,7 +3173,7 @@ bool InterpretBuiltin(InterpState &S, CodePtr OpPC, const CallExpr *Call,
   case Builtin::BI__builtin_ffsl:
   case Builtin::BI__builtin_ffsll:
     return interp__builtin_elementwise_int_unaryop(
-        S, OpPC, Call, [](const APSInt &Val) {
+        S, OpPC, Call, [](const APSInt &Val) -> APInt {
           return APInt(Val.getBitWidth(),
                        Val.isZero() ? 0u : Val.countTrailingZeros() + 1u);
         });

[donneypr]

@RKSimon @tbaederr Following up on our discussion in #162346, I've created PR to add the trailing return type for the function that was missed.

[donneypr]

I've added it for the other callbacks in InterpBuiltin.cpp as well now

[RKSimon]

I thought the plan was to remove all of these things?

[donneypr]

Sorry for the misunderstanding, removed them in InterpBuiltin.cpp

[RKSimon]

LGTM - cheers