[PATCH] D94290: [clang][AArch64][SVE] Avoid going through memory for coerced VLST return values

Cullen Rhodes via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 8 03:43:02 PST 2021


c-rhodes accepted this revision.
c-rhodes added a comment.
This revision is now accepted and ready to land.

I've left one minor nit but looks otherwise looks fine to me



================
Comment at: clang/lib/CodeGen/CGCall.cpp:1273
+    if (auto *FixedSrc =
+            dyn_cast<llvm::FixedVectorType>(Src.getElementType())) {
+      if (ScalableDst->getElementType() == FixedSrc->getElementType()) {
----------------
nit: `s/Src.getElementType()/SrcTy`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94290



More information about the cfe-commits mailing list