[llvm] r230068 - [RewriteStatepointsForGC] More style cleanup [NFC]
David Blaikie
dblaikie at gmail.com
Fri Feb 20 14:24:43 PST 2015
On Fri, Feb 20, 2015 at 2:05 PM, Philip Reames <listmail at philipreames.com>
wrote:
> Author: reames
> Date: Fri Feb 20 16:05:18 2015
> New Revision: 230068
>
> URL: http://llvm.org/viewvc/llvm-project?rev=230068&view=rev
> Log:
> [RewriteStatepointsForGC] More style cleanup [NFC]
>
> Use llvm_unreachable where appropriate, use SmallVector where easy to do
> so, introduce typedefs for planned type migrations.
>
>
> Modified:
> llvm/trunk/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp
>
> Modified: llvm/trunk/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp?rev=230068&r1=230067&r2=230068&view=diff
>
> ==============================================================================
> --- llvm/trunk/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp (original)
> +++ llvm/trunk/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp Fri Feb
> 20 16:05:18 2015
> @@ -96,10 +96,11 @@ namespace {
> // base relation will remain. Internally, we add a mixture of the two
> // types, then update all the second type to the first type
> typedef std::map<Value *, Value *> DefiningValueMapTy;
> +typedef std::set<llvm::Value *> StatepointLiveSetTy;
>
> struct PartiallyConstructedSafepointRecord {
> /// The set of values known to be live accross this safepoint
> - std::set<llvm::Value *> liveset;
> + StatepointLiveSetTy liveset;
>
> /// Mapping from live pointers to a base-defining-value
> DenseMap<llvm::Value *, llvm::Value *> PointerToBase;
> @@ -283,7 +284,7 @@ analyzeParsePointLiveness(DominatorTree
> // Note: This output is used by several of the test cases
> // The order of elemtns in a set is not stable, put them in a vec and
> sort
> // by name
> - std::vector<Value *> temp;
> + SmallVector<Value *, 64> temp;
> temp.insert(temp.end(), liveset.begin(), liveset.end());
> std::sort(temp.begin(), temp.end(), order_by_name);
> errs() << "Live Variables:\n";
> @@ -583,13 +584,14 @@ private:
> Value *base; // non null only if status == base
> };
>
> +typedef std::map<Value *, PhiState> ConflictStateMapTy;
> // Values of type PhiState form a lattice, and this is a helper
> // class that implementes the meet operation. The meat of the meet
> // operation is implemented in MeetPhiStates::pureMeet
> class MeetPhiStates {
> public:
> // phiStates is a mapping from PHINodes and SelectInst's to PhiStates.
> - explicit MeetPhiStates(const std::map<Value *, PhiState> &phiStates)
> + explicit MeetPhiStates(const ConflictStateMapTy &phiStates)
> : phiStates(phiStates) {}
>
> // Destructively meet the current result with the base V. V can
> @@ -607,7 +609,7 @@ public:
> PhiState getResult() const { return currentResult; }
>
> private:
> - const std::map<Value *, PhiState> &phiStates;
> + const ConflictStateMapTy &phiStates;
> PhiState currentResult;
>
> /// Return a phi state for a base defining value. We'll generate a new
> @@ -687,7 +689,7 @@ static Value *findBasePointer(Value *I,
> // analougous to pessimistic data flow and would likely lead to an
> // overall worse solution.
>
> - std::map<Value *, PhiState> states;
> + ConflictStateMapTy states;
> states[def] = PhiState();
> // Recursively fill in all phis & selects reachable from the initial one
> // for which we don't already know a definite base value for
> @@ -820,9 +822,8 @@ static Value *findBasePointer(Value *I,
> v->getParent()->getParent()->getParent()->getContext(),
> MDConst);
> basesel->setMetadata("is_base_value", md);
> states[v] = PhiState(PhiState::Conflict, basesel);
> - } else {
> - assert(false);
> - }
> + } else
> + llvm_unreachable("unknown conflict type");
>
Rather than llvm_unreachable in an else - take the 'if' and move it into an
assert instead?
> }
> }
>
> @@ -922,9 +923,8 @@ static Value *findBasePointer(Value *I,
> }
> basesel->setOperand(i, base);
> }
> - } else {
> - assert(false && "unexpected type");
> - }
> + } else
> + llvm_unreachable("unexpected conflict type");
> }
> }
>
> @@ -1356,7 +1356,7 @@ static void stablize_order(SmallVectorIm
> SmallVectorImpl<Value *> &livevec) {
> assert(basevec.size() == livevec.size());
>
> - std::vector<name_ordering> temp;
> + SmallVector<name_ordering, 64> temp;
> for (size_t i = 0; i < basevec.size(); i++) {
> name_ordering v;
> v.base = basevec[i];
> @@ -1654,9 +1654,8 @@ static void insertUseHolderAfter(CallSit
> Func, Values, "", invoke->getUnwindDest()->getFirstInsertionPt());
> holders.push_back(normal_holder);
> holders.push_back(unwind_holder);
> - } else {
> - assert(false && "Unsupported");
> - }
> + } else
> + llvm_unreachable("unsupported call type");
> }
>
> static void findLiveReferences(
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150220/664d6d9f/attachment.html>
More information about the llvm-commits
mailing list