diff options
author | Adam Nemet <anemet@apple.com> | 2016-04-28 23:08:30 +0000 |
---|---|---|
committer | Adam Nemet <anemet@apple.com> | 2016-04-28 23:08:30 +0000 |
commit | adeccf7658246b5a7f75a1d8b379e371bf9a2c9a (patch) | |
tree | d065faabec516171830d7320eac28005c2daca1a /llvm/lib/Transforms/Scalar/LoopDistribute.cpp | |
parent | 7f38e1199a5b25100eab7cb7f60391e0748a3b57 (diff) | |
download | llvm-adeccf7658246b5a7f75a1d8b379e371bf9a2c9a.zip llvm-adeccf7658246b5a7f75a1d8b379e371bf9a2c9a.tar.gz llvm-adeccf7658246b5a7f75a1d8b379e371bf9a2c9a.tar.bz2 |
[LoopDist] Improve debug messages
The next patch will start using these for -Rpass-analysis so they won't
be internal-only anymore.
Move the 'Skipping; ' prefix that some of the message are using into the
'fail' function. We don't want to include this prefix in
the -Rpass-analysis report.
llvm-svn: 267951
Diffstat (limited to 'llvm/lib/Transforms/Scalar/LoopDistribute.cpp')
-rw-r--r-- | llvm/lib/Transforms/Scalar/LoopDistribute.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/llvm/lib/Transforms/Scalar/LoopDistribute.cpp b/llvm/lib/Transforms/Scalar/LoopDistribute.cpp index 7fa678a..1dcfcda 100644 --- a/llvm/lib/Transforms/Scalar/LoopDistribute.cpp +++ b/llvm/lib/Transforms/Scalar/LoopDistribute.cpp @@ -602,19 +602,19 @@ public: BasicBlock *PH = L->getLoopPreheader(); if (!PH) - return fail("Skipping; no preheader"); + return fail("no preheader"); if (!L->getExitBlock()) - return fail("Skipping; multiple exit blocks"); + return fail("multiple exit blocks"); // LAA will check that we only have a single exiting block. // Currently, we only distribute to isolate the part of the loop with // dependence cycles to enable partial vectorization. if (LAI.canVectorizeMemory()) - return fail("Skipping; memory operations are safe for vectorization"); + return fail("memory operations are safe for vectorization"); auto *Dependences = LAI.getDepChecker().getDependences(); if (!Dependences || Dependences->empty()) - return fail("Skipping; No unsafe dependences to isolate"); + return fail("no unsafe dependences to isolate"); InstPartitionContainer Partitions(L, LI, DT); @@ -694,7 +694,7 @@ public: if (Pred.getComplexity() > (IsForced.getValueOr(false) ? PragmaDistributeSCEVCheckThreshold : DistributeSCEVCheckThreshold)) - return fail("Too many SCEV run-time checks needed.\n"); + return fail("too many SCEV run-time checks needed.\n"); DEBUG(dbgs() << "\nDistributing loop: " << *L << "\n"); // We're done forming the partitions set up the reverse mapping from @@ -745,7 +745,7 @@ public: /// \brief Provide diagnostics then \return with false. bool fail(llvm::StringRef Message) { - DEBUG(dbgs() << Message << "\n"); + DEBUG(dbgs() << "Skipping; " << Message << "\n"); return false; } |