[PATCH] D152600: [InstCombine] Add target option to bail scalarization

Sat Jun 10 00:23:56 PDT 2023

nikic requested changes to this revision.
nikic added a comment.
This revision now requires changes to proceed.

InstCombine is a target-independent canonicalization pass. The use of TTI hooks is forbidden as a matter of policy.

If there is no subset of scalarization that is universally profitable, then the transform should be moved into VectorCombine, which is cost-model driven.

================
Comment at: llvm/include/llvm/Transforms/InstCombine/InstCombiner.h:49
-  /// It must **NOT** be used for any other purpose, as InstCombine is a
-  /// target-independent canonicalization transform.
-  TargetTransformInfo &TTI;
----------------
Read the comment.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152600