[PATCH] D139163: Utils: Add utility pass to lower ifuncs

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 22 15:57:47 PST 2022


arsenm added a comment.

In D139163#4014314 <https://reviews.llvm.org/D139163#4014314>, @MaskRay wrote:

> You previously said "OpenCL forbids function pointers. If you use target_clones in clang, there's no function pointer in the source and the compiler takes care of everything. I want to patch up subtarget specific implementations of builtin functions. It's not for end users"
>
> I take it that if `target_clones` supports non-ifunc use, this patch is not needed. It seems that  `target_clones` indeed works without ifunc (inefficient now, but can be improved), so may I assume that this is not needed?

This is still a way for llvm-reduce to break this into something else it understands how to reduce. This patch does not try to run this anywhere, but adds the testing for the situations for the reduction


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

https://reviews.llvm.org/D139163



More information about the llvm-commits mailing list