[polly] r295713 - [DeLICM] Map values hoisted by LICM back to the array.

[Tobias Grosser via llvm-commits]

On Tue, Feb 21, 2017, at 03:21 PM, Michael Kruse wrote:
> 2017-02-21 13:50 GMT+01:00 Tobias Grosser <tobias.grosser at inf.ethz.ch>:
> > Very cool. Thanks for putting this in!
> >
> > Btw, does it make sense to already enable this pass by default. As this
> > is a conservative minimal implementation it seems perfect to already get
> > test coverage from the start (also from our per-commit tests).
> 
> I don't think it makes sense to enable it by default atm. That would
> mean that everybody who tries Polly would get a compile time increase,
> potentially more crashes/miscompiles without gaining anything in
> practice. It's experimental.
> 
> For getting coverage, we could set up a buildbot.

Then maybe we should enable this flag on the
-polly-position=before-vectorizer. Can you commit this flag change to
zorg? Then it will be run from the next restart of the zorg buildbots.

Best,
Tobias