[PATCH] D155790: PreISelIntrinsicLowering: don't expand memcpys in minsize functions, even with no-builtins.

Tue Aug 1 15:04:32 PDT 2023

aemerson added a comment.

In D155790#4551886 <https://reviews.llvm.org/D155790#4551886>, @arsenm wrote:

> In D155790#4551830 <https://reviews.llvm.org/D155790#4551830>, @aemerson wrote:
>
>>> I think the fix that avoids the current problem, and solves my desires is to move from checking TargetLibraryInfo.hasLibFunc(memcpy) to checking if TargetLoweringInfo::getLibcallName(memcpy) is valid (plus maybe some additional libcall-is-valid-for-addrspace checks?). Sorting out the general libcall mess for freestanding is a problem for another day
>>
>> Are you working on this? If not we'd like to get this fixed and have the previous behavior restored.
>>
>>> 
>
> No, I thought you were since you posted the patch

Ok got it.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155790