[llvm] [Scalarizer][DirectX] support structs return types (PR #111569)

Justin Bogner via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 15 09:06:51 PDT 2024


================
@@ -667,14 +685,25 @@ bool ScalarizerVisitor::splitBinary(Instruction &I, const Splitter &Split) {
 bool ScalarizerVisitor::isTriviallyScalarizable(Intrinsic::ID ID) {
   if (isTriviallyVectorizable(ID))
     return true;
+  // TODO: investigate vectorizable frexp
+  switch (ID) {
+  case Intrinsic::frexp:
----------------
bogner wrote:

It feels pretty questionable to handle `frexp` completely non-generically here. If the follow up work to make `frexp` vectorizable is non-trivial can we at least link to a github issue tracking the follow up work here?

https://github.com/llvm/llvm-project/pull/111569


More information about the llvm-commits mailing list