<div dir="ltr">Hi all,<div><br></div><div>Sorry for the inconvenience, These 3 patches break the Hexagon target.</div><div><br></div><div>Thanks</div><div>Hongbin</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 26, 2016 at 12:55 AM, Michael Kruse <span dir="ltr"><<a href="mailto:llvm-commits@meinersbur.de" target="_blank">llvm-commits@meinersbur.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">What's the reason for the revert?<br>
<br>
The main repository is SVN, so it would be more useful to specify the<br>
SVN revision instead of a git sha1.<br>
<br>
Michael<br>
<br>
<br>
2016-02-25 17:45 GMT+01:00 Hongbin Zheng via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>>:<br>
<div class="HOEnZb"><div class="h5">> 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>
><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>
</div></div></blockquote></div><br></div>