[PATCH] D143016: [InstCombine] Promote expression tree with @llvm.vscale when zero-extending result.

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added a comment.

Thanks for the quick feedback!



================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineCasts.cpp:260
+            Intrinsic::getDeclaration(I->getModule(), Intrinsic::vscale, {Ty});
+        Res = CallInst::Create(Fn->getFunctionType(), Fn);
+        break;
----------------
goldstein.w.n wrote:
> Since `Res` is conditionally set here (i.e we can have `Call` w.o it being `vscale` think this may segfault b.c `Res` is set `nullptr` above and unconditionally derefenced below.
Good spot!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143016