[PATCH] D80388: InstSimplify: Add constant folding/simplify for ldexp intrinsic

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 21 14:05:56 PDT 2020


spatel added subscribers: bogner, mehdi_amini, compnerd, echristo.
spatel added a comment.

There was general agreement that even the blob in instcombine is a mistake:
http://lists.llvm.org/pipermail/llvm-dev/2016-July/102317.html
...but the proposed cleanup has not happened yet. cc @bogner @compnerd @mehdi_amini @echristo

So I think it would be better to keep all of this in instcombine for now. At least that way the mess is not spread across multiple passes any more than now.


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

https://reviews.llvm.org/D80388





More information about the llvm-commits mailing list