s-barannikov wrote:

> I don't think this is the right direction for memset. I would prefer actually 
> overloading it and making the memset/memset.pattern distinction redundant. 
> See also [#97583 
> (comment)](https://github.com/llvm/llvm-project/pull/97583#issuecomment-2288302946)
>  and following comments.

Thanks for the link. I'm in favor of the RFC and in fact I considered this 
approach, too. It will take long time for the llvm.memset to be replaced with 
llvm.memset.pattern though, and until then we need to get llvm.memset working 
with non-8-bit bytes somehow. The suggested approach requires minimum changes, 
is backward compatible, and follows the semantics specified by the LangRef:

> The ‘llvm.memset.*’ intrinsics fill “len” bytes of memory starting at the 
> destination location.

That said, I can keep this patch downstream if you think it does something 
wrong.


https://github.com/llvm/llvm-project/pull/106537
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to