[llvm] [GlobalISel] Add G_SUB for computeNumSignBits (PR #158384)
David Green via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 2 12:59:10 PDT 2025
================
@@ -177,6 +177,51 @@ TEST_F(AArch64SelectionDAGTest, ComputeNumSignBits_VASHR) {
EXPECT_EQ(DAG->ComputeNumSignBits(Fr2), 5u);
}
+TEST_F(AArch64SelectionDAGTest, ComputeNumSignBits_SUB) {
----------------
davemgreen wrote:
Hi - I think it is fine to keep them here, either way is fine. You might find that in general the more focussed the patches are, the easier they are to review and get committed. But in this case it is fine to keep it here as the testing is related.
https://github.com/llvm/llvm-project/pull/158384
More information about the llvm-commits
mailing list