1215746b4SAdam Nemet //===- LoopVersioning.cpp - Utility to version a loop ---------------------===// 2215746b4SAdam Nemet // 32946cd70SChandler Carruth // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. 42946cd70SChandler Carruth // See https://llvm.org/LICENSE.txt for license information. 52946cd70SChandler Carruth // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception 6215746b4SAdam Nemet // 7215746b4SAdam Nemet //===----------------------------------------------------------------------===// 8215746b4SAdam Nemet // 9215746b4SAdam Nemet // This file defines a utility class to perform loop versioning. The versioned 10215746b4SAdam Nemet // loop speculates that otherwise may-aliasing memory accesses don't overlap and 11215746b4SAdam Nemet // emits checks to prove this. 12215746b4SAdam Nemet // 13215746b4SAdam Nemet //===----------------------------------------------------------------------===// 14215746b4SAdam Nemet 1594c83370SDavid Blaikie #include "llvm/Transforms/Utils/LoopVersioning.h" 16215746b4SAdam Nemet #include "llvm/Analysis/LoopAccessAnalysis.h" 17215746b4SAdam Nemet #include "llvm/Analysis/LoopInfo.h" 182910a4f6SSilviu Baranga #include "llvm/Analysis/ScalarEvolutionExpander.h" 19215746b4SAdam Nemet #include "llvm/IR/Dominators.h" 205eccf07dSAdam Nemet #include "llvm/IR/MDBuilder.h" 21215746b4SAdam Nemet #include "llvm/Transforms/Utils/BasicBlockUtils.h" 22215746b4SAdam Nemet #include "llvm/Transforms/Utils/Cloning.h" 23215746b4SAdam Nemet 24215746b4SAdam Nemet using namespace llvm; 25215746b4SAdam Nemet 265eccf07dSAdam Nemet static cl::opt<bool> 275eccf07dSAdam Nemet AnnotateNoAlias("loop-version-annotate-no-alias", cl::init(true), 285eccf07dSAdam Nemet cl::Hidden, 295eccf07dSAdam Nemet cl::desc("Add no-alias annotation for instructions that " 305eccf07dSAdam Nemet "are disambiguated by memchecks")); 315eccf07dSAdam Nemet 322910a4f6SSilviu Baranga LoopVersioning::LoopVersioning(const LoopAccessInfo &LAI, Loop *L, LoopInfo *LI, 332910a4f6SSilviu Baranga DominatorTree *DT, ScalarEvolution *SE, 342910a4f6SSilviu Baranga bool UseLAIChecks) 352910a4f6SSilviu Baranga : VersionedLoop(L), NonVersionedLoop(nullptr), LAI(LAI), LI(LI), DT(DT), 362910a4f6SSilviu Baranga SE(SE) { 37215746b4SAdam Nemet assert(L->getExitBlock() && "No single exit block"); 382e03213fSFlorian Hahn assert(L->isLoopSimplifyForm() && "Loop is not in loop-simplify form"); 392910a4f6SSilviu Baranga if (UseLAIChecks) { 402910a4f6SSilviu Baranga setAliasChecks(LAI.getRuntimePointerChecking()->getChecks()); 4194734eefSXinliang David Li setSCEVChecks(LAI.getPSE().getUnionPredicate()); 422910a4f6SSilviu Baranga } 43215746b4SAdam Nemet } 44215746b4SAdam Nemet 452910a4f6SSilviu Baranga void LoopVersioning::setAliasChecks( 46728f4448SBenjamin Kramer SmallVector<RuntimePointerChecking::PointerCheck, 4> Checks) { 472910a4f6SSilviu Baranga AliasChecks = std::move(Checks); 482910a4f6SSilviu Baranga } 492910a4f6SSilviu Baranga 502910a4f6SSilviu Baranga void LoopVersioning::setSCEVChecks(SCEVUnionPredicate Check) { 512910a4f6SSilviu Baranga Preds = std::move(Check); 52dfaeb33eSAdam Nemet } 53dfaeb33eSAdam Nemet 54e4813409SAdam Nemet void LoopVersioning::versionLoop( 55e4813409SAdam Nemet const SmallVectorImpl<Instruction *> &DefsUsedOutside) { 56215746b4SAdam Nemet Instruction *FirstCheckInst; 57215746b4SAdam Nemet Instruction *MemRuntimeCheck; 582910a4f6SSilviu Baranga Value *SCEVRuntimeCheck; 592910a4f6SSilviu Baranga Value *RuntimeCheck = nullptr; 602910a4f6SSilviu Baranga 61215746b4SAdam Nemet // Add the memcheck in the original preheader (this is empty initially). 622910a4f6SSilviu Baranga BasicBlock *RuntimeCheckBB = VersionedLoop->getLoopPreheader(); 63215746b4SAdam Nemet std::tie(FirstCheckInst, MemRuntimeCheck) = 642910a4f6SSilviu Baranga LAI.addRuntimeChecks(RuntimeCheckBB->getTerminator(), AliasChecks); 65215746b4SAdam Nemet 6694734eefSXinliang David Li const SCEVUnionPredicate &Pred = LAI.getPSE().getUnionPredicate(); 672910a4f6SSilviu Baranga SCEVExpander Exp(*SE, RuntimeCheckBB->getModule()->getDataLayout(), 682910a4f6SSilviu Baranga "scev.check"); 692910a4f6SSilviu Baranga SCEVRuntimeCheck = 702910a4f6SSilviu Baranga Exp.expandCodeForPredicate(&Pred, RuntimeCheckBB->getTerminator()); 712910a4f6SSilviu Baranga auto *CI = dyn_cast<ConstantInt>(SCEVRuntimeCheck); 722910a4f6SSilviu Baranga 732910a4f6SSilviu Baranga // Discard the SCEV runtime check if it is always true. 742910a4f6SSilviu Baranga if (CI && CI->isZero()) 752910a4f6SSilviu Baranga SCEVRuntimeCheck = nullptr; 762910a4f6SSilviu Baranga 772910a4f6SSilviu Baranga if (MemRuntimeCheck && SCEVRuntimeCheck) { 782910a4f6SSilviu Baranga RuntimeCheck = BinaryOperator::Create(Instruction::Or, MemRuntimeCheck, 79299abc10SVikram TV SCEVRuntimeCheck, "lver.safe"); 802910a4f6SSilviu Baranga if (auto *I = dyn_cast<Instruction>(RuntimeCheck)) 812910a4f6SSilviu Baranga I->insertBefore(RuntimeCheckBB->getTerminator()); 822910a4f6SSilviu Baranga } else 832910a4f6SSilviu Baranga RuntimeCheck = MemRuntimeCheck ? MemRuntimeCheck : SCEVRuntimeCheck; 842910a4f6SSilviu Baranga 852910a4f6SSilviu Baranga assert(RuntimeCheck && "called even though we don't need " 862910a4f6SSilviu Baranga "any runtime checks"); 872910a4f6SSilviu Baranga 88215746b4SAdam Nemet // Rename the block to make the IR more readable. 892910a4f6SSilviu Baranga RuntimeCheckBB->setName(VersionedLoop->getHeader()->getName() + 902910a4f6SSilviu Baranga ".lver.check"); 91215746b4SAdam Nemet 92215746b4SAdam Nemet // Create empty preheader for the loop (and after cloning for the 93215746b4SAdam Nemet // non-versioned loop). 942910a4f6SSilviu Baranga BasicBlock *PH = 952910a4f6SSilviu Baranga SplitBlock(RuntimeCheckBB, RuntimeCheckBB->getTerminator(), DT, LI); 96215746b4SAdam Nemet PH->setName(VersionedLoop->getHeader()->getName() + ".ph"); 97215746b4SAdam Nemet 98215746b4SAdam Nemet // Clone the loop including the preheader. 99215746b4SAdam Nemet // 100215746b4SAdam Nemet // FIXME: This does not currently preserve SimplifyLoop because the exit 101215746b4SAdam Nemet // block is a join between the two loops. 102215746b4SAdam Nemet SmallVector<BasicBlock *, 8> NonVersionedLoopBlocks; 103215746b4SAdam Nemet NonVersionedLoop = 1042910a4f6SSilviu Baranga cloneLoopWithPreheader(PH, RuntimeCheckBB, VersionedLoop, VMap, 1052910a4f6SSilviu Baranga ".lver.orig", LI, DT, NonVersionedLoopBlocks); 106215746b4SAdam Nemet remapInstructionsInBlocks(NonVersionedLoopBlocks, VMap); 107215746b4SAdam Nemet 108215746b4SAdam Nemet // Insert the conditional branch based on the result of the memchecks. 1092910a4f6SSilviu Baranga Instruction *OrigTerm = RuntimeCheckBB->getTerminator(); 110215746b4SAdam Nemet BranchInst::Create(NonVersionedLoop->getLoopPreheader(), 1112910a4f6SSilviu Baranga VersionedLoop->getLoopPreheader(), RuntimeCheck, OrigTerm); 112215746b4SAdam Nemet OrigTerm->eraseFromParent(); 113215746b4SAdam Nemet 114215746b4SAdam Nemet // The loops merge in the original exit block. This is now dominated by the 115215746b4SAdam Nemet // memchecking block. 1162910a4f6SSilviu Baranga DT->changeImmediateDominator(VersionedLoop->getExitBlock(), RuntimeCheckBB); 117e4813409SAdam Nemet 118e4813409SAdam Nemet // Adds the necessary PHI nodes for the versioned loops based on the 119e4813409SAdam Nemet // loop-defined values used outside of the loop. 120e4813409SAdam Nemet addPHINodes(DefsUsedOutside); 121215746b4SAdam Nemet } 122215746b4SAdam Nemet 123215746b4SAdam Nemet void LoopVersioning::addPHINodes( 124215746b4SAdam Nemet const SmallVectorImpl<Instruction *> &DefsUsedOutside) { 125215746b4SAdam Nemet BasicBlock *PHIBlock = VersionedLoop->getExitBlock(); 126215746b4SAdam Nemet assert(PHIBlock && "No single successor to loop exit block"); 127215746b4SAdam Nemet PHINode *PN; 128215746b4SAdam Nemet 12973a26957SAdam Nemet // First add a single-operand PHI for each DefsUsedOutside if one does not 13073a26957SAdam Nemet // exists yet. 13173a26957SAdam Nemet for (auto *Inst : DefsUsedOutside) { 13273a26957SAdam Nemet // See if we have a single-operand PHI with the value defined by the 133215746b4SAdam Nemet // original loop. 134215746b4SAdam Nemet for (auto I = PHIBlock->begin(); (PN = dyn_cast<PHINode>(I)); ++I) { 13557fb8989SAdam Nemet if (PN->getIncomingValue(0) == Inst) 136215746b4SAdam Nemet break; 137215746b4SAdam Nemet } 138215746b4SAdam Nemet // If not create it. 139215746b4SAdam Nemet if (!PN) { 140215746b4SAdam Nemet PN = PHINode::Create(Inst->getType(), 2, Inst->getName() + ".lver", 1415b4c837cSDuncan P. N. Exon Smith &PHIBlock->front()); 142fecef6beSBjorn Pettersson SmallVector<User*, 8> UsersToUpdate; 143fecef6beSBjorn Pettersson for (User *U : Inst->users()) 144fecef6beSBjorn Pettersson if (!VersionedLoop->contains(cast<Instruction>(U)->getParent())) 145fecef6beSBjorn Pettersson UsersToUpdate.push_back(U); 146fecef6beSBjorn Pettersson for (User *U : UsersToUpdate) 147fecef6beSBjorn Pettersson U->replaceUsesOfWith(Inst, PN); 148215746b4SAdam Nemet PN->addIncoming(Inst, VersionedLoop->getExitingBlock()); 149215746b4SAdam Nemet } 15073a26957SAdam Nemet } 15173a26957SAdam Nemet 15273a26957SAdam Nemet // Then for each PHI add the operand for the edge from the cloned loop. 15373a26957SAdam Nemet for (auto I = PHIBlock->begin(); (PN = dyn_cast<PHINode>(I)); ++I) { 15473a26957SAdam Nemet assert(PN->getNumOperands() == 1 && 15573a26957SAdam Nemet "Exit block should only have on predecessor"); 15673a26957SAdam Nemet 15773a26957SAdam Nemet // If the definition was cloned used that otherwise use the same value. 15873a26957SAdam Nemet Value *ClonedValue = PN->getIncomingValue(0); 15973a26957SAdam Nemet auto Mapped = VMap.find(ClonedValue); 16073a26957SAdam Nemet if (Mapped != VMap.end()) 16173a26957SAdam Nemet ClonedValue = Mapped->second; 16273a26957SAdam Nemet 16373a26957SAdam Nemet PN->addIncoming(ClonedValue, NonVersionedLoop->getExitingBlock()); 164215746b4SAdam Nemet } 165215746b4SAdam Nemet } 166d52ed841SAdam Nemet 1675eccf07dSAdam Nemet void LoopVersioning::prepareNoAliasMetadata() { 1685eccf07dSAdam Nemet // We need to turn the no-alias relation between pointer checking groups into 1695eccf07dSAdam Nemet // no-aliasing annotations between instructions. 1705eccf07dSAdam Nemet // 1715eccf07dSAdam Nemet // We accomplish this by mapping each pointer checking group (a set of 1725eccf07dSAdam Nemet // pointers memchecked together) to an alias scope and then also mapping each 1735eccf07dSAdam Nemet // group to the list of scopes it can't alias. 1745eccf07dSAdam Nemet 1755eccf07dSAdam Nemet const RuntimePointerChecking *RtPtrChecking = LAI.getRuntimePointerChecking(); 1765eccf07dSAdam Nemet LLVMContext &Context = VersionedLoop->getHeader()->getContext(); 1775eccf07dSAdam Nemet 1785eccf07dSAdam Nemet // First allocate an aliasing scope for each pointer checking group. 1795eccf07dSAdam Nemet // 1805eccf07dSAdam Nemet // While traversing through the checking groups in the loop, also create a 1815eccf07dSAdam Nemet // reverse map from pointers to the pointer checking group they were assigned 1825eccf07dSAdam Nemet // to. 1835eccf07dSAdam Nemet MDBuilder MDB(Context); 1845eccf07dSAdam Nemet MDNode *Domain = MDB.createAnonymousAliasScopeDomain("LVerDomain"); 1855eccf07dSAdam Nemet 1865eccf07dSAdam Nemet for (const auto &Group : RtPtrChecking->CheckingGroups) { 1875eccf07dSAdam Nemet GroupToScope[&Group] = MDB.createAnonymousAliasScope(Domain); 1885eccf07dSAdam Nemet 1895eccf07dSAdam Nemet for (unsigned PtrIdx : Group.Members) 1905eccf07dSAdam Nemet PtrToGroup[RtPtrChecking->getPointerInfo(PtrIdx).PointerValue] = &Group; 1915eccf07dSAdam Nemet } 1925eccf07dSAdam Nemet 1935eccf07dSAdam Nemet // Go through the checks and for each pointer group, collect the scopes for 1945eccf07dSAdam Nemet // each non-aliasing pointer group. 1955eccf07dSAdam Nemet DenseMap<const RuntimePointerChecking::CheckingPtrGroup *, 1965eccf07dSAdam Nemet SmallVector<Metadata *, 4>> 1975eccf07dSAdam Nemet GroupToNonAliasingScopes; 1985eccf07dSAdam Nemet 1995eccf07dSAdam Nemet for (const auto &Check : AliasChecks) 2005eccf07dSAdam Nemet GroupToNonAliasingScopes[Check.first].push_back(GroupToScope[Check.second]); 2015eccf07dSAdam Nemet 2025eccf07dSAdam Nemet // Finally, transform the above to actually map to scope list which is what 2035eccf07dSAdam Nemet // the metadata uses. 2045eccf07dSAdam Nemet 2055eccf07dSAdam Nemet for (auto Pair : GroupToNonAliasingScopes) 2065eccf07dSAdam Nemet GroupToNonAliasingScopeList[Pair.first] = MDNode::get(Context, Pair.second); 2075eccf07dSAdam Nemet } 2085eccf07dSAdam Nemet 2095eccf07dSAdam Nemet void LoopVersioning::annotateLoopWithNoAlias() { 2105eccf07dSAdam Nemet if (!AnnotateNoAlias) 2115eccf07dSAdam Nemet return; 2125eccf07dSAdam Nemet 2135eccf07dSAdam Nemet // First prepare the maps. 2145eccf07dSAdam Nemet prepareNoAliasMetadata(); 2155eccf07dSAdam Nemet 2165eccf07dSAdam Nemet // Add the scope and no-alias metadata to the instructions. 2175eccf07dSAdam Nemet for (Instruction *I : LAI.getDepChecker().getMemoryInstructions()) { 2185eccf07dSAdam Nemet annotateInstWithNoAlias(I); 2195eccf07dSAdam Nemet } 2205eccf07dSAdam Nemet } 2215eccf07dSAdam Nemet 222b0c4eae0SAdam Nemet void LoopVersioning::annotateInstWithNoAlias(Instruction *VersionedInst, 223b0c4eae0SAdam Nemet const Instruction *OrigInst) { 2245eccf07dSAdam Nemet if (!AnnotateNoAlias) 2255eccf07dSAdam Nemet return; 2265eccf07dSAdam Nemet 2275eccf07dSAdam Nemet LLVMContext &Context = VersionedLoop->getHeader()->getContext(); 228b0c4eae0SAdam Nemet const Value *Ptr = isa<LoadInst>(OrigInst) 229b0c4eae0SAdam Nemet ? cast<LoadInst>(OrigInst)->getPointerOperand() 230b0c4eae0SAdam Nemet : cast<StoreInst>(OrigInst)->getPointerOperand(); 2315eccf07dSAdam Nemet 2325eccf07dSAdam Nemet // Find the group for the pointer and then add the scope metadata. 2335eccf07dSAdam Nemet auto Group = PtrToGroup.find(Ptr); 2345eccf07dSAdam Nemet if (Group != PtrToGroup.end()) { 235b0c4eae0SAdam Nemet VersionedInst->setMetadata( 2365eccf07dSAdam Nemet LLVMContext::MD_alias_scope, 237b0c4eae0SAdam Nemet MDNode::concatenate( 238b0c4eae0SAdam Nemet VersionedInst->getMetadata(LLVMContext::MD_alias_scope), 2395eccf07dSAdam Nemet MDNode::get(Context, GroupToScope[Group->second]))); 2405eccf07dSAdam Nemet 2415eccf07dSAdam Nemet // Add the no-alias metadata. 2425eccf07dSAdam Nemet auto NonAliasingScopeList = GroupToNonAliasingScopeList.find(Group->second); 2435eccf07dSAdam Nemet if (NonAliasingScopeList != GroupToNonAliasingScopeList.end()) 244b0c4eae0SAdam Nemet VersionedInst->setMetadata( 2455eccf07dSAdam Nemet LLVMContext::MD_noalias, 246b0c4eae0SAdam Nemet MDNode::concatenate( 247b0c4eae0SAdam Nemet VersionedInst->getMetadata(LLVMContext::MD_noalias), 2485eccf07dSAdam Nemet NonAliasingScopeList->second)); 2495eccf07dSAdam Nemet } 2505eccf07dSAdam Nemet } 2515eccf07dSAdam Nemet 252d52ed841SAdam Nemet namespace { 2535f8f34e4SAdrian Prantl /// Also expose this is a pass. Currently this is only used for 254d52ed841SAdam Nemet /// unit-testing. It adds all memchecks necessary to remove all may-aliasing 255d52ed841SAdam Nemet /// array accesses from the loop. 256d52ed841SAdam Nemet class LoopVersioningPass : public FunctionPass { 257d52ed841SAdam Nemet public: 258d52ed841SAdam Nemet LoopVersioningPass() : FunctionPass(ID) { 259d52ed841SAdam Nemet initializeLoopVersioningPassPass(*PassRegistry::getPassRegistry()); 260d52ed841SAdam Nemet } 261d52ed841SAdam Nemet 262d52ed841SAdam Nemet bool runOnFunction(Function &F) override { 263d52ed841SAdam Nemet auto *LI = &getAnalysis<LoopInfoWrapperPass>().getLoopInfo(); 2647853c1ddSXinliang David Li auto *LAA = &getAnalysis<LoopAccessLegacyAnalysis>(); 265d52ed841SAdam Nemet auto *DT = &getAnalysis<DominatorTreeWrapperPass>().getDomTree(); 266d52ed841SAdam Nemet auto *SE = &getAnalysis<ScalarEvolutionWrapperPass>().getSE(); 267d52ed841SAdam Nemet 268d52ed841SAdam Nemet // Build up a worklist of inner-loops to version. This is necessary as the 269d52ed841SAdam Nemet // act of versioning a loop creates new loops and can invalidate iterators 270d52ed841SAdam Nemet // across the loops. 271d52ed841SAdam Nemet SmallVector<Loop *, 8> Worklist; 272d52ed841SAdam Nemet 273d52ed841SAdam Nemet for (Loop *TopLevelLoop : *LI) 274d52ed841SAdam Nemet for (Loop *L : depth_first(TopLevelLoop)) 275d52ed841SAdam Nemet // We only handle inner-most loops. 276d52ed841SAdam Nemet if (L->empty()) 277d52ed841SAdam Nemet Worklist.push_back(L); 278d52ed841SAdam Nemet 279d52ed841SAdam Nemet // Now walk the identified inner loops. 280d52ed841SAdam Nemet bool Changed = false; 281d52ed841SAdam Nemet for (Loop *L : Worklist) { 282bdbc5227SAdam Nemet const LoopAccessInfo &LAI = LAA->getInfo(L); 283*aa6bdf9dSMatt Arsenault if (L->isLoopSimplifyForm() && !LAI.hasConvergentOp() && 284*aa6bdf9dSMatt Arsenault (LAI.getNumRuntimePointerChecks() || 2852e03213fSFlorian Hahn !LAI.getPSE().getUnionPredicate().isAlwaysTrue())) { 286d52ed841SAdam Nemet LoopVersioning LVer(LAI, L, LI, DT, SE); 287d52ed841SAdam Nemet LVer.versionLoop(); 2885eccf07dSAdam Nemet LVer.annotateLoopWithNoAlias(); 289d52ed841SAdam Nemet Changed = true; 290d52ed841SAdam Nemet } 291d52ed841SAdam Nemet } 292d52ed841SAdam Nemet 293d52ed841SAdam Nemet return Changed; 294d52ed841SAdam Nemet } 295d52ed841SAdam Nemet 296d52ed841SAdam Nemet void getAnalysisUsage(AnalysisUsage &AU) const override { 297d52ed841SAdam Nemet AU.addRequired<LoopInfoWrapperPass>(); 298d52ed841SAdam Nemet AU.addPreserved<LoopInfoWrapperPass>(); 2997853c1ddSXinliang David Li AU.addRequired<LoopAccessLegacyAnalysis>(); 300d52ed841SAdam Nemet AU.addRequired<DominatorTreeWrapperPass>(); 301d52ed841SAdam Nemet AU.addPreserved<DominatorTreeWrapperPass>(); 302d52ed841SAdam Nemet AU.addRequired<ScalarEvolutionWrapperPass>(); 303d52ed841SAdam Nemet } 304d52ed841SAdam Nemet 305d52ed841SAdam Nemet static char ID; 306d52ed841SAdam Nemet }; 307d52ed841SAdam Nemet } 308d52ed841SAdam Nemet 309d52ed841SAdam Nemet #define LVER_OPTION "loop-versioning" 310d52ed841SAdam Nemet #define DEBUG_TYPE LVER_OPTION 311d52ed841SAdam Nemet 312d52ed841SAdam Nemet char LoopVersioningPass::ID; 313d52ed841SAdam Nemet static const char LVer_name[] = "Loop Versioning"; 314d52ed841SAdam Nemet 315d52ed841SAdam Nemet INITIALIZE_PASS_BEGIN(LoopVersioningPass, LVER_OPTION, LVer_name, false, false) 316d52ed841SAdam Nemet INITIALIZE_PASS_DEPENDENCY(LoopInfoWrapperPass) 3177853c1ddSXinliang David Li INITIALIZE_PASS_DEPENDENCY(LoopAccessLegacyAnalysis) 318d52ed841SAdam Nemet INITIALIZE_PASS_DEPENDENCY(DominatorTreeWrapperPass) 319d52ed841SAdam Nemet INITIALIZE_PASS_DEPENDENCY(ScalarEvolutionWrapperPass) 320d52ed841SAdam Nemet INITIALIZE_PASS_END(LoopVersioningPass, LVER_OPTION, LVer_name, false, false) 321d52ed841SAdam Nemet 322d52ed841SAdam Nemet namespace llvm { 323d52ed841SAdam Nemet FunctionPass *createLoopVersioningPass() { 324d52ed841SAdam Nemet return new LoopVersioningPass(); 325d52ed841SAdam Nemet } 326d52ed841SAdam Nemet } 327