[llvm] [LVI] Use m_AddLike instead of m_Add when matching simple condition (PR #86058)

via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 21 11:39:38 PDT 2024


goldsteinn wrote:

> Okay, it looks like there is some impact, so I guess it's fine.
> 
> However, I don't think your test coverage is right. The test in icmp.ll is not affected by the change. You need a disjoint or in the dominating condition, not at the use-site.

Sorry, was pretty lax on the tests as figured it was straight forward, will update

https://github.com/llvm/llvm-project/pull/86058


More information about the llvm-commits mailing list