[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