Skip to content

Conversation

@kazutakahirata
Copy link
Contributor

Local is already of type unsigned.

Local is already of type unsigned.
@llvmbot
Copy link
Member

llvmbot commented Oct 29, 2025

@llvm/pr-subscribers-backend-webassembly

Author: Kazu Hirata (kazutakahirata)

Changes

Local is already of type unsigned.


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

1 Files Affected:

  • (modified) llvm/lib/Target/WebAssembly/WebAssemblyFrameLowering.cpp (+1-1)
diff --git a/llvm/lib/Target/WebAssembly/WebAssemblyFrameLowering.cpp b/llvm/lib/Target/WebAssembly/WebAssemblyFrameLowering.cpp
index 27f7e1ada1250..5a1779c2c80fb 100644
--- a/llvm/lib/Target/WebAssembly/WebAssemblyFrameLowering.cpp
+++ b/llvm/lib/Target/WebAssembly/WebAssemblyFrameLowering.cpp
@@ -81,7 +81,7 @@ WebAssemblyFrameLowering::getLocalForStackObject(MachineFunction &MF,
   // Abuse object size to record number of WebAssembly locals allocated to
   // this object.
   MFI.setObjectSize(FrameIndex, ValueVTs.size());
-  return static_cast<unsigned>(Local);
+  return Local;
 }
 
 /// We need a base pointer in the case of having items on the stack that

@kazutakahirata kazutakahirata merged commit 7d6f4d0 into llvm:main Oct 29, 2025
12 checks passed
@kazutakahirata kazutakahirata deleted the cleanup_20251028_clang_tidy_readability_redundant_casting_WebAssembly branch October 30, 2025 00:18
aokblast pushed a commit to aokblast/llvm-project that referenced this pull request Oct 30, 2025
DEBADRIBASAK pushed a commit to DEBADRIBASAK/llvm-project that referenced this pull request Nov 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants