[polly] r248916 - Move remapping functionality in the ScopExpander

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 30 13:32:44 PDT 2015


On 09/30/2015 06:52 PM, Johannes Doerfert via llvm-commits wrote:
> Author: jdoerfert
> Date: Wed Sep 30 11:52:03 2015
> New Revision: 248916
>
> URL: http://llvm.org/viewvc/llvm-project?rev=248916&view=rev
> Log:
> Move remapping functionality in the ScopExpander
>
>    Because we handle more than SCEV does it is not possible to rewrite an
>    expression on the top-level using the SCEVParameterRewriter only. With
>    this patch we will do the rewriting on demand only and also
>    recursively, thus not only on the top-level.

Since this or the last commit I see a LNT failure:

http://lab.llvm.org:8011/builders/perf-x86_64-penryn-O3-polly-fast/builds/12422

Also, I think a test case would be good here.

> +#include "llvm/ADT/DenseMap.h"
> +#include "llvm/ADT/SmallPtrSet.h"
> +#include "llvm/ADT/SmallVector.h"
> +#include "llvm/Analysis/AliasAnalysis.h"

Most of these includes were not needed. I dropped all but the first.

Best,
Tobias



More information about the llvm-commits mailing list