[llvm] [ConstraintElim] Check the other signedness when hasSameSign (PR #128168)
Marina Taylor via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 21 09:47:39 PST 2025
citymarina wrote:
> Looks reasonable, but I think the test coverage is lacking: how about a dominating samesign condition as well?
Does the added test from https://github.com/llvm/llvm-project/issues/120089#issuecomment-2635581659 cover the scenario that you were looking for? If not, could you sketch out for me what you had in mind?
https://github.com/llvm/llvm-project/pull/128168
More information about the llvm-commits
mailing list