[PATCH] D82049: Fix crash in VectorCombine when attempting to peephole ConstantVector sequences

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 19 08:05:23 PDT 2020


nikic added a comment.

Compile-time numbers: https://llvm-compile-time-tracker.com/compare.php?from=35651fdd4537c081ca47acde58b6fd4b5f6997b3&to=5ff2db86f51083878ff9a458a39b355a03b31e73&stat=instructions

It should be noted that while InstSimplify is cheap relative to InstCombine, it's likely a good bit more expansive than the actual VectorCombine itself :)

As the original patch does not look particularly complicated, I would prefer the more targeted fix.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82049





More information about the llvm-commits mailing list