[PATCH] D82277: [InstCombine] fold fmul/fdiv with fabs operands

Michael Berg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 22 10:12:56 PDT 2020

Previous message: [PATCH] D82315: [PowerPC][PCRelative] Thread Local Storage Support for General Dynamic
Next message: [PATCH] D82277: [InstCombine] fold fmul/fdiv with fabs operands
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

mcberg2017 accepted this revision.
mcberg2017 added a comment.
This revision is now accepted and ready to land.

LGTM.  My read on this change is that you have set it up to never make more code than we have, and transformation is safe.


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

https://reviews.llvm.org/D82277

Previous message: [PATCH] D82315: [PowerPC][PCRelative] Thread Local Storage Support for General Dynamic
Next message: [PATCH] D82277: [InstCombine] fold fmul/fdiv with fabs operands
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list