[PATCH] D122836: [SimplifyLibCalls] Fold more memchr calls

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 4 02:51:55 PDT 2022


xbolva00 added a comment.

In D122836#3425739 <https://reviews.llvm.org/D122836#3425739>, @nikic wrote:

> I've taken the liberty of splitting this up into the following four commits while landing the change:
> https://github.com/llvm/llvm-project/commit/0f088757449d63182fb424dce7856213739c563e
> https://github.com/llvm/llvm-project/commit/d18991debfde6be335a14af3c7d2dcdecbcd758d
> https://github.com/llvm/llvm-project/commit/5197d2791f908815134ad48d7b966de2d8c47eeb
> https://github.com/llvm/llvm-project/commit/5ccfd5f6d43069a5bc152b335bc704c6d2584ddc
>
> The length truncation can be addressed separately, as it's a pre-existing issue (and it looks like there are a few more substr calls in the same file that should be handled at the same time).

Great, nice small patches!


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

https://reviews.llvm.org/D122836



More information about the llvm-commits mailing list