[PATCH] D54899: [LoopStrengthReduce] ComplexityLimit as an option

[Gil Rapaport via Phabricator via llvm-commits]

gilr added a subscriber: qcolombet.
gilr added a comment.

In D54899#1311068 <https://reviews.llvm.org/D54899#1311068>, @samparker wrote:

> Thanks for taking a look and for the value renamer tip! The test case has now been renamed and reduced.


Sure. Credit for the tip goes to @qcolombet who enlightened me about this :)



================
Comment at: test/Transforms/LoopStrengthReduce/lsr-comp-time.ll:2
 ; RUN: opt -loop-reduce -S < %s | FileCheck %s
+; RUN: opt -loop-reduce -lsr-complexity-limit=2147483647 -S < %s | FileCheck %s
+
----------------
Won't this effectively limit LSR to search spaces it can process w/o pruning?


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

https://reviews.llvm.org/D54899