Skip to content

Conversation

@arsenm
Copy link
Contributor

@arsenm arsenm commented Oct 24, 2025

This has been dead since 97bfb93

Copy link
Contributor Author

arsenm commented Oct 24, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@arsenm arsenm added the llvm:SelectionDAG SelectionDAGISel as well label Oct 24, 2025 — with Graphite App
@arsenm arsenm marked this pull request as ready for review October 24, 2025 12:27
@arsenm arsenm enabled auto-merge (squash) October 24, 2025 12:27
@llvmbot
Copy link
Member

llvmbot commented Oct 24, 2025

@llvm/pr-subscribers-llvm-selectiondag

Author: Matt Arsenault (arsenm)

Changes

This has been dead since 97bfb93


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

1 Files Affected:

  • (modified) llvm/include/llvm/CodeGen/TargetLowering.h (-4)
diff --git a/llvm/include/llvm/CodeGen/TargetLowering.h b/llvm/include/llvm/CodeGen/TargetLowering.h
index a70c9c02d3a83..b99d4acb60215 100644
--- a/llvm/include/llvm/CodeGen/TargetLowering.h
+++ b/llvm/include/llvm/CodeGen/TargetLowering.h
@@ -3813,10 +3813,6 @@ class LLVM_ABI TargetLoweringBase {
   /// The list of libcalls that the target will use.
   RTLIB::RuntimeLibcallsInfo Libcalls;
 
-  /// The ISD::CondCode that should be used to test the result of each of the
-  /// comparison libcall against zero.
-  ISD::CondCode CmpLibcallCCs[RTLIB::UNKNOWN_LIBCALL];
-
   /// The bits of IndexedModeActions used to store the legalisation actions
   /// We store the data as   | ML | MS |  L |  S | each taking 4 bits.
   enum IndexedModeActionsBits {

@arsenm arsenm merged commit 76b6399 into main Oct 24, 2025
14 checks passed
@arsenm arsenm deleted the users/arsenm/dag/remove-unused-targetlowering-field branch October 24, 2025 13:12
dvbuka pushed a commit to dvbuka/llvm-project that referenced this pull request Oct 27, 2025
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