Skip to content

Conversation

@kazutakahirata
Copy link
Contributor

ThisPartition is already of type int.

Identified with readability-redundant-casting.

ThisPartition is already of type int.

Identified with readability-redundant-casting.
@llvmbot
Copy link
Member

llvmbot commented Nov 16, 2025

@llvm/pr-subscribers-llvm-transforms

Author: Kazu Hirata (kazutakahirata)

Changes

ThisPartition is already of type int.

Identified with readability-redundant-casting.


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

1 Files Affected:

  • (modified) llvm/lib/Transforms/Scalar/LoopDistribute.cpp (+1-1)
diff --git a/llvm/lib/Transforms/Scalar/LoopDistribute.cpp b/llvm/lib/Transforms/Scalar/LoopDistribute.cpp
index 0c8b9043fcbbb..ab292e833852f 100644
--- a/llvm/lib/Transforms/Scalar/LoopDistribute.cpp
+++ b/llvm/lib/Transforms/Scalar/LoopDistribute.cpp
@@ -520,7 +520,7 @@ class InstPartitionContainer {
         // -1 means belonging to multiple partitions.
         else if (Partition == -1)
           break;
-        else if (Partition != (int)ThisPartition)
+        else if (Partition != ThisPartition)
           Partition = -1;
       }
       assert(Partition != -2 && "Pointer not belonging to any partition");

@kazutakahirata kazutakahirata merged commit 977d6cf into llvm:main Nov 16, 2025
12 checks passed
@kazutakahirata kazutakahirata deleted the cleanup_20251116_clang_tidy_readability-redundant-casting branch November 16, 2025 22:06
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