HomePhabricator

[OpenMPIRBuilder] Various changes required for tileLoops.

Authored by Meinersbur on Dec 11 2020, 8:55 AM.

Description

[OpenMPIRBuilder] Various changes required for tileLoops.

Extract some changes not directly related to tileLoops out of D92974:

  • Refactor createLoopSkeleton out of createCanonicalLoop.
  • Introduce ComputeIP parameter to the createCanonicalLoop overload inserts instructions to compute the trip count. Specifying the location is necessary to make these instructions appear before the outermost loop of a loop nest that is tiled.
  • Introduce Name parameter to createCanonicalLoop. This can help better understanding the origin of values of basic blocks with many loops. The default value is "loop" instead of "for" which could be confused with the "for directive" (aka worksharing-loop) and does not apply to Fortran.
  • Remove CanonicalLoopInfo::eraseFromParent which is currently unused and untested and was added in anticipation to be used by tileLoops. eraseFromParent has shown to be insufficient when more than a single loop is involved and is replaced by removeUnusedBlocksFromParent in D92974.

Reviewed By: SouraVX

Differential Revision: https://reviews.llvm.org/D93088