Skip to content

Conversation

@dakersnar
Copy link
Contributor

Fix bug in this change: #119622 (comment)

@llvmbot
Copy link
Member

llvmbot commented Dec 17, 2024

@llvm/pr-subscribers-backend-nvptx

Author: Drew Kersnar (dakersnar)

Changes

Fix bug in this change: #119622 (comment)


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

1 Files Affected:

  • (modified) llvm/lib/Target/NVPTX/NVPTXISelLowering.cpp (+1-1)
diff --git a/llvm/lib/Target/NVPTX/NVPTXISelLowering.cpp b/llvm/lib/Target/NVPTX/NVPTXISelLowering.cpp
index 3e01def18e6e70..a033a8247fac51 100644
--- a/llvm/lib/Target/NVPTX/NVPTXISelLowering.cpp
+++ b/llvm/lib/Target/NVPTX/NVPTXISelLowering.cpp
@@ -221,7 +221,7 @@ getVectorLoweringShape(EVT VectorVT) {
   }
 
   llvm_unreachable("All cases in switch should return.");
-};
+}
 
 /// ComputePTXValueVTs - For the given Type \p Ty, returns the set of primitive
 /// EVTs that compose it.  Unlike ComputeValueVTs, this will break apart vectors

@AlexMaclean AlexMaclean merged commit 9d11aa1 into llvm:main Dec 18, 2024
7 of 9 checks passed
steven-studio pushed a commit to steven-studio/llvm-project that referenced this pull request Sep 11, 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