[PATCH] D123819: [InstCombine] Fold strlen and strnlen recursively.

Martin Sebor via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 27 09:55:50 PDT 2022


msebor added a comment.

In D123819#3542702 <https://reviews.llvm.org/D123819#3542702>, @xbolva00 wrote:

> Cases like  strlen (x ? s1 : s2) should be handled in more general fashion

Agreed.  Such a general rewrite seems outside the scope of the simple transformations done by the folder (when either `s1` or `s2` is not constant).  Does that mean we don't want the proposed change and should instead wait for the more general transform?


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

https://reviews.llvm.org/D123819



More information about the llvm-commits mailing list