[PATCH] D106850: [InstCombine] Simplify llvm.vscale when vscale_range attribute exists

[David Sherwood via Phabricator via llvm-commits]

david-arm added a comment.

Hi @junparser, thanks for looking into this performance improvement! However, I personally think it makes more sense to just always replace a call to vscale directly with a constant if you know the value of vscale at runtime - possibly done in InstSimplify or something like that? That way you don't need to add lots of different instcombine changes to support all the different folds that can happen. Constant folds would just kick in automatically.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106850