[PATCH] D78967: [SVE] Remove invalid usage of VectorType::getNumElements in Function

Christopher Tetreault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 27 15:39:43 PDT 2020


ctetreau marked an inline comment as done.
ctetreau added inline comments.


================
Comment at: llvm/lib/IR/Function.cpp:1080
+      return Ty;
+    // FIXME: Explicit conversion from fixed to scalable vector. Can/should this
+    // happen?
----------------
This behavior was introduced in D65930. It seems to me that this would have just returned the decoded type if it were expected that the decoded type would be scalable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78967





More information about the llvm-commits mailing list