Skip to content

TargetSelectionDAG: Add missing atomic fp opd classes #102269

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 7, 2024
Merged

Conversation

jsji
Copy link
Member

@jsji jsji commented Aug 7, 2024

The defs are missing after 5c9352e .

@llvmbot llvmbot added the llvm:SelectionDAG SelectionDAGISel as well label Aug 7, 2024
@jsji jsji requested review from arsenm and lukel97 August 7, 2024 03:47
@jsji jsji self-assigned this Aug 7, 2024
@llvmbot
Copy link
Member

llvmbot commented Aug 7, 2024

@llvm/pr-subscribers-llvm-selectiondag

Author: Jinsong Ji (jsji)

Changes

The defs are missing after 5c9352e .


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

1 Files Affected:

  • (modified) llvm/include/llvm/Target/TargetSelectionDAG.td (+2)
diff --git a/llvm/include/llvm/Target/TargetSelectionDAG.td b/llvm/include/llvm/Target/TargetSelectionDAG.td
index 46044aab79a832..fd47ddda6ca3a5 100644
--- a/llvm/include/llvm/Target/TargetSelectionDAG.td
+++ b/llvm/include/llvm/Target/TargetSelectionDAG.td
@@ -1748,6 +1748,8 @@ multiclass binary_atomic_op_fp<SDNode atomic_op> {
       let IsAtomic = true;
       let MemoryVT = vt;
     }
+
+    defm NAME#_#vt : binary_atomic_op_ord;
   }
 }
 

@jsji jsji merged commit ca7368d into llvm:main Aug 7, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
llvm:SelectionDAG SelectionDAGISel as well
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants