[PATCH] D90015: [InstSimpify][ValueTracking] Use new FP matchers

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 23 02:03:44 PDT 2020


foad added a comment.

I split this out from D89038 <https://reviews.llvm.org/D89038> because of @lebedev.ri's comment:

> I would strongly recommend to split this into two separate commits - adding the matchers, and using them.
> Because while the matchers themselves LG, i'm not at all confident with users.
> When doing latter, please ensure that each separate change has appropriate undef test coverage, and ideally check that alive2 is happy about those transforms.

I haven't looked at adding the extra test coverage yet.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90015



More information about the llvm-commits mailing list