[PATCH] D45236: [Polly][IslAst] Fix minimal dependence distance.

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 3 17:53:42 PDT 2018


Meinersbur added inline comments.


================
Comment at: test/Isl/Ast/dependence_distance_minimal.ll:1
+; RUN: opt %loadPolly -polly-ast -polly-ast-detect-parallel -analyze -instnamer < %s | FileCheck %s
+;
----------------
Sorry, this is not what I meant.

The instructions and BasicBlocks in this .ll file have no names, making LLVM assigning numeric "names" on the fly. Such ids are instable, and without result in different statement names for statements (`Stmt1` and `Stmt1_b`) depending on `-polly-use-llvm-names` and iteration order.
Please regenerate this files _with_ string names which fixes those names to those in this file. The `-instnamer` pass automatically assigns string names, so you can just run this file through `opt -instnamer` and get a new file with instruction names.


Repository:
  rL LLVM

https://reviews.llvm.org/D45236





More information about the llvm-commits mailing list