[PATCH] D58521: [DAGCombiner] allow truncation of binops after legalization if desirable

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 29 11:04:28 PDT 2021


spatel added a comment.

In D58521#2655302 <https://reviews.llvm.org/D58521#2655302>, @RKSimon wrote:

> @spatel Is this necessary any more?

The SystemZ code hasn't changed, so the code is still not ideal for that target. I'm not sure what we can do about the remaining regressions. I'll rebase, so we at least have an updated view of test diffs


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

https://reviews.llvm.org/D58521



More information about the llvm-commits mailing list