[ScopInfo] Do not add array name into memory reference ids
authorTobias Grosser <tobias@grosser.es>
Wed, 3 May 2017 07:57:35 +0000 (07:57 +0000)
committerTobias Grosser <tobias@grosser.es>
Wed, 3 May 2017 07:57:35 +0000 (07:57 +0000)
commit8133128c172fb42224ec62596d3c9d673b552ce9
tree658be0f408837364dc9f925ec8247c104cf94f92
parentba9aefc0ec758217e12381181c6f71035ee88fde
[ScopInfo] Do not add array name into memory reference ids

Before this change a memory reference identifier had the form:

  <STMT>_<ACCESSTYPE><ID>_<MEMREF>, e.g., Stmt_bb9_Write0_MemRef_tmp11

After this change, we use the format:

  <STMT>_<ACCESSTYPE><ID>, e.g., Stmt_bb9_Write0

The name of the array that is accessed through a memory reference is not
necessary to uniquely identify a memory reference, but was only added to
provide additional information for debugging. We drop this information now
for the following two reasons:

  1) This shortens the names and consequently improves readability
  2) This removes a second location where we decide on the name of a scop array,
     leaving us only with the location where the actual scop array is created.

Having after 2) only a single location to name scop arrays will allow us to
change the naming convention of scop arrays more easily, which we will do
in a future commit to reduce compilation time.

llvm-svn: 302004
polly/lib/Analysis/ScopInfo.cpp
polly/test/DependenceInfo/computeout.ll
polly/test/DependenceInfo/different_schedule_dimensions.ll
polly/test/DependenceInfo/do_pluto_matmult.ll
polly/test/DependenceInfo/fine_grain_dep_0.ll
polly/test/DependenceInfo/sequential_loops.ll