[PATCH] D86578: [TargetLowering] Combine known bits for icmp in SimplifySetCC (PR41182)

Matt D'Arcangelo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 26 16:33:09 PDT 2020


darcangelo.matthew added a comment.

In D86578#2239392 <https://reviews.llvm.org/D86578#2239392>, @nikic wrote:

> What is the motivation for this patch? The added tests should already get folded before reaching the backend (by InstCombine). We should only add folds to DAGCombine if the folded pattern is created by legalization. What is the original context for this change?

This was an unassigned enhancement - https://bugs.llvm.org/show_bug.cgi?id=41182 - that I was interested in and wanted to have a go at. Unfortunately that's all I have for context on my end


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86578/new/

https://reviews.llvm.org/D86578



More information about the llvm-commits mailing list