[PATCH] D107353: [LoopIdiom] let the pass deal with runtime memset size

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 13 11:25:05 PDT 2021


lebedev.ri added a comment.

In D107353#2944144 <https://reviews.llvm.org/D107353#2944144>, @eopXD wrote:

> In D107353#2944129 <https://reviews.llvm.org/D107353#2944129>, @lebedev.ri wrote:
>
>> Actually, we've talked about not needing the overflow check, but IIRC just saturating the number of bytes.
>> Can you point me at the code that does that?
>
> No code for saturation here.
> I thought restricting to address space zero prevents the need of saturation.

@efriedma does that sound right?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107353



More information about the llvm-commits mailing list