Skip to content

Conversation

@fhossein-quic
Copy link
Contributor

Correct the definition of HWBF16 to reference VecPBF16 rather than VecBF16, aligning it with the existing pattern-leaf conventions for pointer vector types.

@llvmbot
Copy link
Member

llvmbot commented Dec 3, 2025

@llvm/pr-subscribers-backend-hexagon

Author: Fateme Hosseini (fhossein-quic)

Changes

Correct the definition of HWBF16 to reference VecPBF16 rather than VecBF16, aligning it with the existing pattern-leaf conventions for pointer vector types.


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

1 Files Affected:

  • (modified) llvm/lib/Target/Hexagon/HexagonPatternsHVX.td (+1-1)
diff --git a/llvm/lib/Target/Hexagon/HexagonPatternsHVX.td b/llvm/lib/Target/Hexagon/HexagonPatternsHVX.td
index 4cb29e7f00317..8783aa8ce829f 100644
--- a/llvm/lib/Target/Hexagon/HexagonPatternsHVX.td
+++ b/llvm/lib/Target/Hexagon/HexagonPatternsHVX.td
@@ -22,7 +22,7 @@ def HWI16:  PatLeaf<(VecPI16 HvxWR:$R)>;
 def HWI32:  PatLeaf<(VecPI32 HvxWR:$R)>;
 def HWF16:  PatLeaf<(VecPF16 HvxWR:$R)>;
 def HWF32:  PatLeaf<(VecPF32 HvxWR:$R)>;
-def HWBF16: PatLeaf<(VecBF16 HvxWR:$R)>;
+def HWBF16: PatLeaf<(VecPBF16 HvxWR:$R)>;
 
 def SDTVecUnaryOp:
   SDTypeProfile<1, 1, [SDTCisVec<0>, SDTCisVec<1>]>;

Adds custom load/store and masked/vector-building lowering for
v128bf16, updates HVX patterns to use the proper VecPBF16 leaf.

Patch By: Fateme Hosseini

Co-authored-by: Muntasir Mallick <[email protected]>
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.

5 participants