[all-commits] [llvm/llvm-project] 79d185: [VPlan] Move Load/Store checks out of tryToWiden (...

Florian Hahn via All-commits all-commits at lists.llvm.org
Wed Apr 15 02:23:55 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 79d185c79271455c6b05c86ba422f40e70c7e970
      https://github.com/llvm/llvm-project/commit/79d185c79271455c6b05c86ba422f40e70c7e970
  Author: Florian Hahn <flo at fhahn.com>
  Date:   2020-04-15 (Wed, 15 Apr 2020)

  Changed paths:
    M llvm/lib/Transforms/Vectorize/LoopVectorize.cpp

  Log Message:
  -----------
  [VPlan] Move Load/Store checks out of tryToWiden (NFC).

Handling LoadInst and StoreInst in tryToWiden seems a bit
counter-intuitive, as there is only an assertion for them and in no
case VPWidenRefipes are created for them.

I think it makes sense to move the assertion to handleReplication, where
the non-widened loads and store are handled.

Reviewers: gilr, rengolin, Ayal, hsaito

Reviewed By: Ayal

Differential Revision: https://reviews.llvm.org/D77972




More information about the All-commits mailing list