[PATCH] D85548: Allow fneg + strict_fadd -> strict_fsub in DAGCombiner
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 18 08:55:11 PDT 2020
spatel added a comment.
Why wait until codegen to do this? IIUC, it's impossible for any backend to spontaneously create strict FP ops, so this pattern must be present in the original IR.
Ie, wouldn't we be better off reducing these in IR (InstCombinerImpl::visitCallInst())?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85548/new/
https://reviews.llvm.org/D85548
More information about the llvm-commits
mailing list