<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 25, 2016 at 8:45 AM, Hongbin Zheng via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: ether<br>
Date: Thu Feb 25 10:45:37 2016<br>
New Revision: 261889<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=261889&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=261889&view=rev</a><br>
Log:<br>
Revert "Introduce RegionInfoAnalysis, which compute Region Tree in the new PassManager. NFC"<br>
<br>
This reverts commit 8228b4d374edeb4cc0c5fddf6e1ab876918ee126.<br></blockquote><div><br></div><div>When reverting patches, please include a reason and the original SVN revision number (if you're using git-svn, we have a utility to help with this in llvm/utils/git-svn/git-svnrevert if I recall correctly)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Modified:<br>
llvm/trunk/include/llvm/Analysis/RegionInfo.h<br>
llvm/trunk/lib/Analysis/RegionInfo.cpp<br>
llvm/trunk/lib/Passes/PassBuilder.cpp<br>
llvm/trunk/lib/Passes/PassRegistry.def<br>
llvm/trunk/test/Analysis/RegionInfo/20100809_bb_not_in_domtree.ll<br>
llvm/trunk/test/Analysis/RegionInfo/block_sort.ll<br>
llvm/trunk/test/Analysis/RegionInfo/cond_loop.ll<br>
llvm/trunk/test/Analysis/RegionInfo/condition_complicated.ll<br>
llvm/trunk/test/Analysis/RegionInfo/condition_complicated_2.ll<br>
llvm/trunk/test/Analysis/RegionInfo/condition_forward_edge.ll<br>
llvm/trunk/test/Analysis/RegionInfo/condition_same_exit.ll<br>
llvm/trunk/test/Analysis/RegionInfo/condition_simple.ll<br>
llvm/trunk/test/Analysis/RegionInfo/exit_in_condition.ll<br>
llvm/trunk/test/Analysis/RegionInfo/loop_with_condition.ll<br>
llvm/trunk/test/Analysis/RegionInfo/loops_1.ll<br>
llvm/trunk/test/Analysis/RegionInfo/loops_2.ll<br>
llvm/trunk/test/Analysis/RegionInfo/mix_1.ll<br>
llvm/trunk/test/Analysis/RegionInfo/nested_loops.ll<br>
llvm/trunk/test/Analysis/RegionInfo/next.ll<br>
llvm/trunk/test/Analysis/RegionInfo/paper.ll<br>
llvm/trunk/test/Analysis/RegionInfo/two_loops_same_header.ll<br>
llvm/trunk/test/Analysis/RegionInfo/unreachable_bb.ll<br>
<br>
Modified: llvm/trunk/include/llvm/Analysis/RegionInfo.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/RegionInfo.h?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/RegionInfo.h?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/include/llvm/Analysis/RegionInfo.h (original)<br>
+++ llvm/trunk/include/llvm/Analysis/RegionInfo.h Thu Feb 25 10:45:37 2016<br>
@@ -47,11 +47,6 @@<br>
<br>
namespace llvm {<br>
<br>
-// FIXME: Replace this brittle forward declaration with the include of the new<br>
-// PassManager.h when doing so doesn't break the PassManagerBuilder.<br>
-template <typename IRUnitT> class AnalysisManager;<br>
-class PreservedAnalyses;<br>
-<br>
// Class to be specialized for different users of RegionInfo<br>
// (i.e. BasicBlocks or MachineBasicBlocks). This is only to avoid needing to<br>
// pass around an unreasonable number of template parameters.<br>
@@ -681,22 +676,6 @@ class RegionInfoBase {<br>
RegionInfoBase(const RegionInfoBase &) = delete;<br>
const RegionInfoBase &operator=(const RegionInfoBase &) = delete;<br>
<br>
- RegionInfoBase(RegionInfoBase &&Arg)<br>
- : DT(std::move(Arg.DT)), PDT(std::move(Arg.PDT)), DF(std::move(Arg.DF)),<br>
- TopLevelRegion(std::move(Arg.TopLevelRegion)),<br>
- BBtoRegion(std::move(Arg.BBtoRegion)) {<br>
- Arg.wipe();<br>
- }<br>
- RegionInfoBase &operator=(RegionInfoBase &&RHS) {<br>
- DT = std::move(RHS.DT);<br>
- PDT = std::move(RHS.PDT);<br>
- DF = std::move(RHS.DF);<br>
- TopLevelRegion = std::move(RHS.TopLevelRegion);<br>
- BBtoRegion = std::move(RHS.BBtoRegion);<br>
- RHS.wipe();<br>
- return *this;<br>
- }<br>
-<br>
DomTreeT *DT;<br>
PostDomTreeT *PDT;<br>
DomFrontierT *DF;<br>
@@ -708,18 +687,6 @@ private:<br>
/// Map every BB to the smallest region, that contains BB.<br>
BBtoRegionMap BBtoRegion;<br>
<br>
- /// \brief Wipe this region tree's state without releasing any resources.<br>
- ///<br>
- /// This is essentially a post-move helper only. It leaves the object in an<br>
- /// assignable and destroyable state, but otherwise invalid.<br>
- void wipe() {<br>
- DT = nullptr;<br>
- PDT = nullptr;<br>
- DF = nullptr;<br>
- TopLevelRegion = nullptr;<br>
- BBtoRegion.clear();<br>
- }<br>
-<br>
// Check whether the entries of BBtoRegion for the BBs of region<br>
// SR are correct. Triggers an assertion if not. Calls itself recursively for<br>
// subregions.<br>
@@ -869,19 +836,10 @@ public:<br>
<br>
class RegionInfo : public RegionInfoBase<RegionTraits<Function>> {<br>
public:<br>
- typedef RegionInfoBase<RegionTraits<Function>> Base;<br>
-<br>
explicit RegionInfo();<br>
<br>
~RegionInfo() override;<br>
<br>
- RegionInfo(RegionInfo &&Arg)<br>
- : Base(std::move(static_cast<Base &>(Arg))) {}<br>
- RegionInfo &operator=(RegionInfo &&RHS) {<br>
- Base::operator=(std::move(static_cast<Base &>(RHS)));<br>
- return *this;<br>
- }<br>
-<br>
// updateStatistics - Update statistic about created regions.<br>
void updateStatistics(Region *R) final;<br>
<br>
@@ -926,40 +884,6 @@ public:<br>
//@}<br>
};<br>
<br>
-/// \brief Analysis pass that exposes the \c RegionInfo for a function.<br>
-class RegionInfoAnalysis {<br>
- static char PassID;<br>
-<br>
-public:<br>
- typedef RegionInfo Result;<br>
-<br>
- /// \brief Opaque, unique identifier for this analysis pass.<br>
- static void *ID() { return (void *)&PassID; }<br>
-<br>
- /// \brief Provide a name for the analysis for debugging and logging.<br>
- static StringRef name() { return "RegionInfoAnalysis"; }<br>
-<br>
- RegionInfo run(Function &F, AnalysisManager<Function> *AM);<br>
-};<br>
-<br>
-/// \brief Printer pass for the \c RegionInfo.<br>
-class RegionInfoPrinterPass {<br>
- raw_ostream &OS;<br>
-<br>
-public:<br>
- explicit RegionInfoPrinterPass(raw_ostream &OS);<br>
- PreservedAnalyses run(Function &F, AnalysisManager<Function> *AM);<br>
-<br>
- static StringRef name() { return "RegionInfoPrinterPass"; }<br>
-};<br>
-<br>
-/// \brief Verifier pass for the \c RegionInfo.<br>
-struct RegionInfoVerifierPass {<br>
- PreservedAnalyses run(Function &F, AnalysisManager<Function> *AM);<br>
-<br>
- static StringRef name() { return "RegionInfoVerifierPass"; }<br>
-};<br>
-<br>
template <><br>
template <><br>
inline BasicBlock *<br>
<br>
Modified: llvm/trunk/lib/Analysis/RegionInfo.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/RegionInfo.cpp?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/RegionInfo.cpp?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Analysis/RegionInfo.cpp (original)<br>
+++ llvm/trunk/lib/Analysis/RegionInfo.cpp Thu Feb 25 10:45:37 2016<br>
@@ -15,7 +15,6 @@<br>
#include "llvm/Analysis/LoopInfo.h"<br>
#include "llvm/Analysis/RegionInfoImpl.h"<br>
#include "llvm/Analysis/RegionIterator.h"<br>
-#include "llvm/IR/PassManager.h"<br>
#include "llvm/Support/CommandLine.h"<br>
#include "llvm/Support/Debug.h"<br>
#include "llvm/Support/ErrorHandling.h"<br>
@@ -181,36 +180,3 @@ namespace llvm {<br>
}<br>
}<br>
<br>
-//===----------------------------------------------------------------------===//<br>
-// RegionInfoAnalysis implementation<br>
-//<br>
-<br>
-char RegionInfoAnalysis::PassID;<br>
-<br>
-RegionInfo RegionInfoAnalysis::run(Function &F, AnalysisManager<Function> *AM) {<br>
- RegionInfo RI;<br>
- auto *DT = &AM->getResult<DominatorTreeAnalysis>(F);<br>
- auto *PDT = &AM->getResult<PostDominatorTreeAnalysis>(F);<br>
- auto *DF = &AM->getResult<DominanceFrontierAnalysis>(F);<br>
-<br>
- RI.recalculate(F, DT, PDT, DF);<br>
- return RI;<br>
-}<br>
-<br>
-RegionInfoPrinterPass::RegionInfoPrinterPass(raw_ostream &OS)<br>
- : OS(OS) {}<br>
-<br>
-PreservedAnalyses<br>
-RegionInfoPrinterPass::run(Function &F, FunctionAnalysisManager *AM) {<br>
- OS << "Region Tree for function: " << F.getName() << "\n";<br>
- AM->getResult<RegionInfoAnalysis>(F).print(OS);<br>
-<br>
- return PreservedAnalyses::all();<br>
-}<br>
-<br>
-PreservedAnalyses RegionInfoVerifierPass::run(Function &F,<br>
- AnalysisManager<Function> *AM) {<br>
- AM->getResult<RegionInfoAnalysis>(F).verifyAnalysis();<br>
-<br>
- return PreservedAnalyses::all();<br>
-}<br>
<br>
Modified: llvm/trunk/lib/Passes/PassBuilder.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Passes/PassBuilder.cpp?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Passes/PassBuilder.cpp?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Passes/PassBuilder.cpp (original)<br>
+++ llvm/trunk/lib/Passes/PassBuilder.cpp Thu Feb 25 10:45:37 2016<br>
@@ -26,7 +26,6 @@<br>
#include "llvm/Analysis/LazyCallGraph.h"<br>
#include "llvm/Analysis/LoopInfo.h"<br>
#include "llvm/Analysis/PostDominators.h"<br>
-#include "llvm/Analysis/RegionInfo.h"<br>
#include "llvm/Analysis/ScalarEvolution.h"<br>
#include "llvm/Analysis/ScalarEvolutionAliasAnalysis.h"<br>
#include "llvm/Analysis/ScopedNoAliasAA.h"<br>
<br>
Modified: llvm/trunk/lib/Passes/PassRegistry.def<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Passes/PassRegistry.def?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Passes/PassRegistry.def?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Passes/PassRegistry.def (original)<br>
+++ llvm/trunk/lib/Passes/PassRegistry.def Thu Feb 25 10:45:37 2016<br>
@@ -60,7 +60,6 @@ FUNCTION_ANALYSIS("domtree", DominatorTr<br>
FUNCTION_ANALYSIS("postdomtree", PostDominatorTreeAnalysis())<br>
FUNCTION_ANALYSIS("domfrontier", DominanceFrontierAnalysis())<br>
FUNCTION_ANALYSIS("loops", LoopAnalysis())<br>
-FUNCTION_ANALYSIS("regions", RegionInfoAnalysis())<br>
FUNCTION_ANALYSIS("no-op-function", NoOpFunctionAnalysis())<br>
FUNCTION_ANALYSIS("scalar-evolution", ScalarEvolutionAnalysis())<br>
FUNCTION_ANALYSIS("targetlibinfo", TargetLibraryAnalysis())<br>
@@ -95,13 +94,11 @@ FUNCTION_PASS("print<domtree>", Dominato<br>
FUNCTION_PASS("print<postdomtree>", PostDominatorTreePrinterPass(dbgs()))<br>
FUNCTION_PASS("print<domfrontier>", DominanceFrontierPrinterPass(dbgs()))<br>
FUNCTION_PASS("print<loops>", LoopPrinterPass(dbgs()))<br>
-FUNCTION_PASS("print<regions>", RegionInfoPrinterPass(dbgs()))<br>
FUNCTION_PASS("print<scalar-evolution>", ScalarEvolutionPrinterPass(dbgs()))<br>
FUNCTION_PASS("simplify-cfg", SimplifyCFGPass())<br>
FUNCTION_PASS("sroa", SROA())<br>
FUNCTION_PASS("verify", VerifierPass())<br>
FUNCTION_PASS("verify<domtree>", DominatorTreeVerifierPass())<br>
-FUNCTION_PASS("verify<regions>", RegionInfoVerifierPass())<br>
#undef FUNCTION_PASS<br>
<br>
#ifndef LOOP_ANALYSIS<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/20100809_bb_not_in_domtree.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/20100809_bb_not_in_domtree.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/20100809_bb_not_in_domtree.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/20100809_bb_not_in_domtree.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/20100809_bb_not_in_domtree.ll Thu Feb 25 10:45:37 2016<br>
@@ -1,6 +1,4 @@<br>
; RUN: opt -regions < %s<br>
-; RUN: opt < %s -passes='print<regions>'<br>
-<br>
define i32 @main() nounwind {<br>
entry:<br>
br label %for.cond<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/block_sort.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/block_sort.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/block_sort.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/block_sort.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/block_sort.ll Thu Feb 25 10:45:37 2016<br>
@@ -4,8 +4,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define void @BZ2_blockSort() nounwind {<br>
start:<br>
br label %while<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/cond_loop.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/cond_loop.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/cond_loop.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/cond_loop.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/cond_loop.ll Thu Feb 25 10:45:37 2016<br>
@@ -4,8 +4,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define void @normal_condition() nounwind {<br>
5:<br>
br label %"0"<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/condition_complicated.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/condition_complicated.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/condition_complicated.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/condition_complicated.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/condition_complicated.ll Thu Feb 25 10:45:37 2016<br>
@@ -4,8 +4,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define internal fastcc zeroext i8 @handle_compress() nounwind {<br>
end165:<br>
br i1 1, label %false239, label %true181<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/condition_complicated_2.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/condition_complicated_2.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/condition_complicated_2.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/condition_complicated_2.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/condition_complicated_2.ll Thu Feb 25 10:45:37 2016<br>
@@ -4,8 +4,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define internal fastcc void @compress() nounwind {<br>
end33:<br>
br i1 1, label %end124, label %lor.lhs.false95<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/condition_forward_edge.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/condition_forward_edge.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/condition_forward_edge.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/condition_forward_edge.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/condition_forward_edge.ll Thu Feb 25 10:45:37 2016<br>
@@ -4,8 +4,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define void @normal_condition() nounwind {<br>
0:<br>
br label %"1"<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/condition_same_exit.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/condition_same_exit.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/condition_same_exit.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/condition_same_exit.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/condition_same_exit.ll Thu Feb 25 10:45:37 2016<br>
@@ -4,8 +4,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define void @normal_condition() nounwind {<br>
0:<br>
br i1 1, label %"1", label %"4"<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/condition_simple.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/condition_simple.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/condition_simple.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/condition_simple.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/condition_simple.ll Thu Feb 25 10:45:37 2016<br>
@@ -4,8 +4,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define void @normal_condition() nounwind {<br>
0:<br>
br label %"1"<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/exit_in_condition.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/exit_in_condition.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/exit_in_condition.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/exit_in_condition.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/exit_in_condition.ll Thu Feb 25 10:45:37 2016<br>
@@ -4,8 +4,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define internal fastcc zeroext i8 @handle_compress() nounwind {<br>
entry:<br>
br label %outer<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/loop_with_condition.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/loop_with_condition.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/loop_with_condition.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/loop_with_condition.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/loop_with_condition.ll Thu Feb 25 10:45:37 2016<br>
@@ -5,8 +5,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define void @normal_condition() nounwind {<br>
0:<br>
br label %"1"<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/loops_1.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/loops_1.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/loops_1.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/loops_1.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/loops_1.ll Thu Feb 25 10:45:37 2016<br>
@@ -4,8 +4,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define internal fastcc zeroext i8 @loops_1() nounwind {<br>
entry:<br>
br i1 1, label %outer , label %a<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/loops_2.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/loops_2.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/loops_2.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/loops_2.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/loops_2.ll Thu Feb 25 10:45:37 2016<br>
@@ -4,8 +4,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define void @meread_() nounwind {<br>
entry:<br>
br label %bb23<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/mix_1.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/mix_1.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/mix_1.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/mix_1.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/mix_1.ll Thu Feb 25 10:45:37 2016<br>
@@ -5,8 +5,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define void @a_linear_impl_fig_1() nounwind {<br>
0:<br>
<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/nested_loops.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/nested_loops.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/nested_loops.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/nested_loops.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/nested_loops.ll Thu Feb 25 10:45:37 2016<br>
@@ -5,8 +5,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define internal fastcc zeroext i8 @handle_compress() nounwind {<br>
entry:<br>
br label %outer<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/next.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/next.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/next.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/next.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/next.ll Thu Feb 25 10:45:37 2016<br>
@@ -4,8 +4,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define void @MAIN__() nounwind {<br>
entry:<br>
br label %__label_002001.outer<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/paper.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/paper.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/paper.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/paper.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/paper.ll Thu Feb 25 10:45:37 2016<br>
@@ -4,8 +4,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define void @a_linear_impl_fig_1() nounwind {<br>
0:<br>
br label %"1"<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/two_loops_same_header.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/two_loops_same_header.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/two_loops_same_header.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/two_loops_same_header.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/two_loops_same_header.ll Thu Feb 25 10:45:37 2016<br>
@@ -4,8 +4,6 @@<br>
; RUN: opt -regions -print-region-style=bb -analyze < %s 2>&1 | FileCheck -check-prefix=BBIT %s<br>
; RUN: opt -regions -print-region-style=rn -analyze < %s 2>&1 | FileCheck -check-prefix=RNIT %s<br>
<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
-<br>
define internal fastcc zeroext i8 @handle_compress() nounwind {<br>
entry:<br>
br label %outer<br>
<br>
Modified: llvm/trunk/test/Analysis/RegionInfo/unreachable_bb.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/unreachable_bb.ll?rev=261889&r1=261888&r2=261889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/unreachable_bb.ll?rev=261889&r1=261888&r2=261889&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/Analysis/RegionInfo/unreachable_bb.ll (original)<br>
+++ llvm/trunk/test/Analysis/RegionInfo/unreachable_bb.ll Thu Feb 25 10:45:37 2016<br>
@@ -1,5 +1,4 @@<br>
; RUN: opt -regions -analyze < %s | FileCheck %s<br>
-; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s<br>
<br>
; We should not crash if there are some bbs that are not reachable.<br>
define void @f() {<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div></div>