xref: /freebsd/contrib/llvm-project/llvm/lib/Target/WebAssembly/WebAssemblyFixFunctionBitcasts.cpp (revision 8bcb0991864975618c09697b1aca10683346d9f0)
10b57cec5SDimitry Andric //===-- WebAssemblyFixFunctionBitcasts.cpp - Fix function bitcasts --------===//
20b57cec5SDimitry Andric //
30b57cec5SDimitry Andric // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
40b57cec5SDimitry Andric // See https://llvm.org/LICENSE.txt for license information.
50b57cec5SDimitry Andric // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
60b57cec5SDimitry Andric //
70b57cec5SDimitry Andric //===----------------------------------------------------------------------===//
80b57cec5SDimitry Andric ///
90b57cec5SDimitry Andric /// \file
100b57cec5SDimitry Andric /// Fix bitcasted functions.
110b57cec5SDimitry Andric ///
120b57cec5SDimitry Andric /// WebAssembly requires caller and callee signatures to match, however in LLVM,
130b57cec5SDimitry Andric /// some amount of slop is vaguely permitted. Detect mismatch by looking for
140b57cec5SDimitry Andric /// bitcasts of functions and rewrite them to use wrapper functions instead.
150b57cec5SDimitry Andric ///
160b57cec5SDimitry Andric /// This doesn't catch all cases, such as when a function's address is taken in
170b57cec5SDimitry Andric /// one place and casted in another, but it works for many common cases.
180b57cec5SDimitry Andric ///
190b57cec5SDimitry Andric /// Note that LLVM already optimizes away function bitcasts in common cases by
200b57cec5SDimitry Andric /// dropping arguments as needed, so this pass only ends up getting used in less
210b57cec5SDimitry Andric /// common cases.
220b57cec5SDimitry Andric ///
230b57cec5SDimitry Andric //===----------------------------------------------------------------------===//
240b57cec5SDimitry Andric 
250b57cec5SDimitry Andric #include "WebAssembly.h"
260b57cec5SDimitry Andric #include "llvm/IR/CallSite.h"
270b57cec5SDimitry Andric #include "llvm/IR/Constants.h"
280b57cec5SDimitry Andric #include "llvm/IR/Instructions.h"
290b57cec5SDimitry Andric #include "llvm/IR/Module.h"
300b57cec5SDimitry Andric #include "llvm/IR/Operator.h"
310b57cec5SDimitry Andric #include "llvm/Pass.h"
320b57cec5SDimitry Andric #include "llvm/Support/Debug.h"
330b57cec5SDimitry Andric #include "llvm/Support/raw_ostream.h"
340b57cec5SDimitry Andric using namespace llvm;
350b57cec5SDimitry Andric 
360b57cec5SDimitry Andric #define DEBUG_TYPE "wasm-fix-function-bitcasts"
370b57cec5SDimitry Andric 
380b57cec5SDimitry Andric namespace {
390b57cec5SDimitry Andric class FixFunctionBitcasts final : public ModulePass {
400b57cec5SDimitry Andric   StringRef getPassName() const override {
410b57cec5SDimitry Andric     return "WebAssembly Fix Function Bitcasts";
420b57cec5SDimitry Andric   }
430b57cec5SDimitry Andric 
440b57cec5SDimitry Andric   void getAnalysisUsage(AnalysisUsage &AU) const override {
450b57cec5SDimitry Andric     AU.setPreservesCFG();
460b57cec5SDimitry Andric     ModulePass::getAnalysisUsage(AU);
470b57cec5SDimitry Andric   }
480b57cec5SDimitry Andric 
490b57cec5SDimitry Andric   bool runOnModule(Module &M) override;
500b57cec5SDimitry Andric 
510b57cec5SDimitry Andric public:
520b57cec5SDimitry Andric   static char ID;
530b57cec5SDimitry Andric   FixFunctionBitcasts() : ModulePass(ID) {}
540b57cec5SDimitry Andric };
550b57cec5SDimitry Andric } // End anonymous namespace
560b57cec5SDimitry Andric 
570b57cec5SDimitry Andric char FixFunctionBitcasts::ID = 0;
580b57cec5SDimitry Andric INITIALIZE_PASS(FixFunctionBitcasts, DEBUG_TYPE,
590b57cec5SDimitry Andric                 "Fix mismatching bitcasts for WebAssembly", false, false)
600b57cec5SDimitry Andric 
610b57cec5SDimitry Andric ModulePass *llvm::createWebAssemblyFixFunctionBitcasts() {
620b57cec5SDimitry Andric   return new FixFunctionBitcasts();
630b57cec5SDimitry Andric }
640b57cec5SDimitry Andric 
650b57cec5SDimitry Andric // Recursively descend the def-use lists from V to find non-bitcast users of
660b57cec5SDimitry Andric // bitcasts of V.
670b57cec5SDimitry Andric static void findUses(Value *V, Function &F,
680b57cec5SDimitry Andric                      SmallVectorImpl<std::pair<Use *, Function *>> &Uses,
690b57cec5SDimitry Andric                      SmallPtrSetImpl<Constant *> &ConstantBCs) {
700b57cec5SDimitry Andric   for (Use &U : V->uses()) {
710b57cec5SDimitry Andric     if (auto *BC = dyn_cast<BitCastOperator>(U.getUser()))
720b57cec5SDimitry Andric       findUses(BC, F, Uses, ConstantBCs);
73*8bcb0991SDimitry Andric     else if (auto *A = dyn_cast<GlobalAlias>(U.getUser()))
74*8bcb0991SDimitry Andric       findUses(A, F, Uses, ConstantBCs);
750b57cec5SDimitry Andric     else if (U.get()->getType() != F.getType()) {
760b57cec5SDimitry Andric       CallSite CS(U.getUser());
770b57cec5SDimitry Andric       if (!CS)
780b57cec5SDimitry Andric         // Skip uses that aren't immediately called
790b57cec5SDimitry Andric         continue;
800b57cec5SDimitry Andric       Value *Callee = CS.getCalledValue();
810b57cec5SDimitry Andric       if (Callee != V)
820b57cec5SDimitry Andric         // Skip calls where the function isn't the callee
830b57cec5SDimitry Andric         continue;
840b57cec5SDimitry Andric       if (isa<Constant>(U.get())) {
850b57cec5SDimitry Andric         // Only add constant bitcasts to the list once; they get RAUW'd
860b57cec5SDimitry Andric         auto C = ConstantBCs.insert(cast<Constant>(U.get()));
870b57cec5SDimitry Andric         if (!C.second)
880b57cec5SDimitry Andric           continue;
890b57cec5SDimitry Andric       }
900b57cec5SDimitry Andric       Uses.push_back(std::make_pair(&U, &F));
910b57cec5SDimitry Andric     }
920b57cec5SDimitry Andric   }
930b57cec5SDimitry Andric }
940b57cec5SDimitry Andric 
950b57cec5SDimitry Andric // Create a wrapper function with type Ty that calls F (which may have a
960b57cec5SDimitry Andric // different type). Attempt to support common bitcasted function idioms:
970b57cec5SDimitry Andric //  - Call with more arguments than needed: arguments are dropped
980b57cec5SDimitry Andric //  - Call with fewer arguments than needed: arguments are filled in with undef
990b57cec5SDimitry Andric //  - Return value is not needed: drop it
1000b57cec5SDimitry Andric //  - Return value needed but not present: supply an undef
1010b57cec5SDimitry Andric //
1020b57cec5SDimitry Andric // If the all the argument types of trivially castable to one another (i.e.
1030b57cec5SDimitry Andric // I32 vs pointer type) then we don't create a wrapper at all (return nullptr
1040b57cec5SDimitry Andric // instead).
1050b57cec5SDimitry Andric //
1060b57cec5SDimitry Andric // If there is a type mismatch that we know would result in an invalid wasm
1070b57cec5SDimitry Andric // module then generate wrapper that contains unreachable (i.e. abort at
1080b57cec5SDimitry Andric // runtime).  Such programs are deep into undefined behaviour territory,
1090b57cec5SDimitry Andric // but we choose to fail at runtime rather than generate and invalid module
1100b57cec5SDimitry Andric // or fail at compiler time.  The reason we delay the error is that we want
1110b57cec5SDimitry Andric // to support the CMake which expects to be able to compile and link programs
1120b57cec5SDimitry Andric // that refer to functions with entirely incorrect signatures (this is how
1130b57cec5SDimitry Andric // CMake detects the existence of a function in a toolchain).
1140b57cec5SDimitry Andric //
1150b57cec5SDimitry Andric // For bitcasts that involve struct types we don't know at this stage if they
1160b57cec5SDimitry Andric // would be equivalent at the wasm level and so we can't know if we need to
1170b57cec5SDimitry Andric // generate a wrapper.
1180b57cec5SDimitry Andric static Function *createWrapper(Function *F, FunctionType *Ty) {
1190b57cec5SDimitry Andric   Module *M = F->getParent();
1200b57cec5SDimitry Andric 
1210b57cec5SDimitry Andric   Function *Wrapper = Function::Create(Ty, Function::PrivateLinkage,
1220b57cec5SDimitry Andric                                        F->getName() + "_bitcast", M);
1230b57cec5SDimitry Andric   BasicBlock *BB = BasicBlock::Create(M->getContext(), "body", Wrapper);
1240b57cec5SDimitry Andric   const DataLayout &DL = BB->getModule()->getDataLayout();
1250b57cec5SDimitry Andric 
1260b57cec5SDimitry Andric   // Determine what arguments to pass.
1270b57cec5SDimitry Andric   SmallVector<Value *, 4> Args;
1280b57cec5SDimitry Andric   Function::arg_iterator AI = Wrapper->arg_begin();
1290b57cec5SDimitry Andric   Function::arg_iterator AE = Wrapper->arg_end();
1300b57cec5SDimitry Andric   FunctionType::param_iterator PI = F->getFunctionType()->param_begin();
1310b57cec5SDimitry Andric   FunctionType::param_iterator PE = F->getFunctionType()->param_end();
1320b57cec5SDimitry Andric   bool TypeMismatch = false;
1330b57cec5SDimitry Andric   bool WrapperNeeded = false;
1340b57cec5SDimitry Andric 
1350b57cec5SDimitry Andric   Type *ExpectedRtnType = F->getFunctionType()->getReturnType();
1360b57cec5SDimitry Andric   Type *RtnType = Ty->getReturnType();
1370b57cec5SDimitry Andric 
1380b57cec5SDimitry Andric   if ((F->getFunctionType()->getNumParams() != Ty->getNumParams()) ||
1390b57cec5SDimitry Andric       (F->getFunctionType()->isVarArg() != Ty->isVarArg()) ||
1400b57cec5SDimitry Andric       (ExpectedRtnType != RtnType))
1410b57cec5SDimitry Andric     WrapperNeeded = true;
1420b57cec5SDimitry Andric 
1430b57cec5SDimitry Andric   for (; AI != AE && PI != PE; ++AI, ++PI) {
1440b57cec5SDimitry Andric     Type *ArgType = AI->getType();
1450b57cec5SDimitry Andric     Type *ParamType = *PI;
1460b57cec5SDimitry Andric 
1470b57cec5SDimitry Andric     if (ArgType == ParamType) {
1480b57cec5SDimitry Andric       Args.push_back(&*AI);
1490b57cec5SDimitry Andric     } else {
1500b57cec5SDimitry Andric       if (CastInst::isBitOrNoopPointerCastable(ArgType, ParamType, DL)) {
1510b57cec5SDimitry Andric         Instruction *PtrCast =
1520b57cec5SDimitry Andric             CastInst::CreateBitOrPointerCast(AI, ParamType, "cast");
1530b57cec5SDimitry Andric         BB->getInstList().push_back(PtrCast);
1540b57cec5SDimitry Andric         Args.push_back(PtrCast);
1550b57cec5SDimitry Andric       } else if (ArgType->isStructTy() || ParamType->isStructTy()) {
1560b57cec5SDimitry Andric         LLVM_DEBUG(dbgs() << "createWrapper: struct param type in bitcast: "
1570b57cec5SDimitry Andric                           << F->getName() << "\n");
1580b57cec5SDimitry Andric         WrapperNeeded = false;
1590b57cec5SDimitry Andric       } else {
1600b57cec5SDimitry Andric         LLVM_DEBUG(dbgs() << "createWrapper: arg type mismatch calling: "
1610b57cec5SDimitry Andric                           << F->getName() << "\n");
1620b57cec5SDimitry Andric         LLVM_DEBUG(dbgs() << "Arg[" << Args.size() << "] Expected: "
1630b57cec5SDimitry Andric                           << *ParamType << " Got: " << *ArgType << "\n");
1640b57cec5SDimitry Andric         TypeMismatch = true;
1650b57cec5SDimitry Andric         break;
1660b57cec5SDimitry Andric       }
1670b57cec5SDimitry Andric     }
1680b57cec5SDimitry Andric   }
1690b57cec5SDimitry Andric 
1700b57cec5SDimitry Andric   if (WrapperNeeded && !TypeMismatch) {
1710b57cec5SDimitry Andric     for (; PI != PE; ++PI)
1720b57cec5SDimitry Andric       Args.push_back(UndefValue::get(*PI));
1730b57cec5SDimitry Andric     if (F->isVarArg())
1740b57cec5SDimitry Andric       for (; AI != AE; ++AI)
1750b57cec5SDimitry Andric         Args.push_back(&*AI);
1760b57cec5SDimitry Andric 
1770b57cec5SDimitry Andric     CallInst *Call = CallInst::Create(F, Args, "", BB);
1780b57cec5SDimitry Andric 
1790b57cec5SDimitry Andric     Type *ExpectedRtnType = F->getFunctionType()->getReturnType();
1800b57cec5SDimitry Andric     Type *RtnType = Ty->getReturnType();
1810b57cec5SDimitry Andric     // Determine what value to return.
1820b57cec5SDimitry Andric     if (RtnType->isVoidTy()) {
1830b57cec5SDimitry Andric       ReturnInst::Create(M->getContext(), BB);
1840b57cec5SDimitry Andric     } else if (ExpectedRtnType->isVoidTy()) {
1850b57cec5SDimitry Andric       LLVM_DEBUG(dbgs() << "Creating dummy return: " << *RtnType << "\n");
1860b57cec5SDimitry Andric       ReturnInst::Create(M->getContext(), UndefValue::get(RtnType), BB);
1870b57cec5SDimitry Andric     } else if (RtnType == ExpectedRtnType) {
1880b57cec5SDimitry Andric       ReturnInst::Create(M->getContext(), Call, BB);
1890b57cec5SDimitry Andric     } else if (CastInst::isBitOrNoopPointerCastable(ExpectedRtnType, RtnType,
1900b57cec5SDimitry Andric                                                     DL)) {
1910b57cec5SDimitry Andric       Instruction *Cast =
1920b57cec5SDimitry Andric           CastInst::CreateBitOrPointerCast(Call, RtnType, "cast");
1930b57cec5SDimitry Andric       BB->getInstList().push_back(Cast);
1940b57cec5SDimitry Andric       ReturnInst::Create(M->getContext(), Cast, BB);
1950b57cec5SDimitry Andric     } else if (RtnType->isStructTy() || ExpectedRtnType->isStructTy()) {
1960b57cec5SDimitry Andric       LLVM_DEBUG(dbgs() << "createWrapper: struct return type in bitcast: "
1970b57cec5SDimitry Andric                         << F->getName() << "\n");
1980b57cec5SDimitry Andric       WrapperNeeded = false;
1990b57cec5SDimitry Andric     } else {
2000b57cec5SDimitry Andric       LLVM_DEBUG(dbgs() << "createWrapper: return type mismatch calling: "
2010b57cec5SDimitry Andric                         << F->getName() << "\n");
2020b57cec5SDimitry Andric       LLVM_DEBUG(dbgs() << "Expected: " << *ExpectedRtnType
2030b57cec5SDimitry Andric                         << " Got: " << *RtnType << "\n");
2040b57cec5SDimitry Andric       TypeMismatch = true;
2050b57cec5SDimitry Andric     }
2060b57cec5SDimitry Andric   }
2070b57cec5SDimitry Andric 
2080b57cec5SDimitry Andric   if (TypeMismatch) {
2090b57cec5SDimitry Andric     // Create a new wrapper that simply contains `unreachable`.
2100b57cec5SDimitry Andric     Wrapper->eraseFromParent();
2110b57cec5SDimitry Andric     Wrapper = Function::Create(Ty, Function::PrivateLinkage,
2120b57cec5SDimitry Andric                                F->getName() + "_bitcast_invalid", M);
2130b57cec5SDimitry Andric     BasicBlock *BB = BasicBlock::Create(M->getContext(), "body", Wrapper);
2140b57cec5SDimitry Andric     new UnreachableInst(M->getContext(), BB);
2150b57cec5SDimitry Andric     Wrapper->setName(F->getName() + "_bitcast_invalid");
2160b57cec5SDimitry Andric   } else if (!WrapperNeeded) {
2170b57cec5SDimitry Andric     LLVM_DEBUG(dbgs() << "createWrapper: no wrapper needed: " << F->getName()
2180b57cec5SDimitry Andric                       << "\n");
2190b57cec5SDimitry Andric     Wrapper->eraseFromParent();
2200b57cec5SDimitry Andric     return nullptr;
2210b57cec5SDimitry Andric   }
2220b57cec5SDimitry Andric   LLVM_DEBUG(dbgs() << "createWrapper: " << F->getName() << "\n");
2230b57cec5SDimitry Andric   return Wrapper;
2240b57cec5SDimitry Andric }
2250b57cec5SDimitry Andric 
2260b57cec5SDimitry Andric // Test whether a main function with type FuncTy should be rewritten to have
2270b57cec5SDimitry Andric // type MainTy.
2280b57cec5SDimitry Andric static bool shouldFixMainFunction(FunctionType *FuncTy, FunctionType *MainTy) {
2290b57cec5SDimitry Andric   // Only fix the main function if it's the standard zero-arg form. That way,
2300b57cec5SDimitry Andric   // the standard cases will work as expected, and users will see signature
2310b57cec5SDimitry Andric   // mismatches from the linker for non-standard cases.
2320b57cec5SDimitry Andric   return FuncTy->getReturnType() == MainTy->getReturnType() &&
2330b57cec5SDimitry Andric          FuncTy->getNumParams() == 0 &&
2340b57cec5SDimitry Andric          !FuncTy->isVarArg();
2350b57cec5SDimitry Andric }
2360b57cec5SDimitry Andric 
2370b57cec5SDimitry Andric bool FixFunctionBitcasts::runOnModule(Module &M) {
2380b57cec5SDimitry Andric   LLVM_DEBUG(dbgs() << "********** Fix Function Bitcasts **********\n");
2390b57cec5SDimitry Andric 
2400b57cec5SDimitry Andric   Function *Main = nullptr;
2410b57cec5SDimitry Andric   CallInst *CallMain = nullptr;
2420b57cec5SDimitry Andric   SmallVector<std::pair<Use *, Function *>, 0> Uses;
2430b57cec5SDimitry Andric   SmallPtrSet<Constant *, 2> ConstantBCs;
2440b57cec5SDimitry Andric 
2450b57cec5SDimitry Andric   // Collect all the places that need wrappers.
2460b57cec5SDimitry Andric   for (Function &F : M) {
2470b57cec5SDimitry Andric     findUses(&F, F, Uses, ConstantBCs);
2480b57cec5SDimitry Andric 
2490b57cec5SDimitry Andric     // If we have a "main" function, and its type isn't
2500b57cec5SDimitry Andric     // "int main(int argc, char *argv[])", create an artificial call with it
2510b57cec5SDimitry Andric     // bitcasted to that type so that we generate a wrapper for it, so that
2520b57cec5SDimitry Andric     // the C runtime can call it.
2530b57cec5SDimitry Andric     if (F.getName() == "main") {
2540b57cec5SDimitry Andric       Main = &F;
2550b57cec5SDimitry Andric       LLVMContext &C = M.getContext();
2560b57cec5SDimitry Andric       Type *MainArgTys[] = {Type::getInt32Ty(C),
2570b57cec5SDimitry Andric                             PointerType::get(Type::getInt8PtrTy(C), 0)};
2580b57cec5SDimitry Andric       FunctionType *MainTy = FunctionType::get(Type::getInt32Ty(C), MainArgTys,
2590b57cec5SDimitry Andric                                                /*isVarArg=*/false);
2600b57cec5SDimitry Andric       if (shouldFixMainFunction(F.getFunctionType(), MainTy)) {
2610b57cec5SDimitry Andric         LLVM_DEBUG(dbgs() << "Found `main` function with incorrect type: "
2620b57cec5SDimitry Andric                           << *F.getFunctionType() << "\n");
2630b57cec5SDimitry Andric         Value *Args[] = {UndefValue::get(MainArgTys[0]),
2640b57cec5SDimitry Andric                          UndefValue::get(MainArgTys[1])};
2650b57cec5SDimitry Andric         Value *Casted =
2660b57cec5SDimitry Andric             ConstantExpr::getBitCast(Main, PointerType::get(MainTy, 0));
2670b57cec5SDimitry Andric         CallMain = CallInst::Create(MainTy, Casted, Args, "call_main");
2680b57cec5SDimitry Andric         Use *UseMain = &CallMain->getOperandUse(2);
2690b57cec5SDimitry Andric         Uses.push_back(std::make_pair(UseMain, &F));
2700b57cec5SDimitry Andric       }
2710b57cec5SDimitry Andric     }
2720b57cec5SDimitry Andric   }
2730b57cec5SDimitry Andric 
2740b57cec5SDimitry Andric   DenseMap<std::pair<Function *, FunctionType *>, Function *> Wrappers;
2750b57cec5SDimitry Andric 
2760b57cec5SDimitry Andric   for (auto &UseFunc : Uses) {
2770b57cec5SDimitry Andric     Use *U = UseFunc.first;
2780b57cec5SDimitry Andric     Function *F = UseFunc.second;
2790b57cec5SDimitry Andric     auto *PTy = cast<PointerType>(U->get()->getType());
2800b57cec5SDimitry Andric     auto *Ty = dyn_cast<FunctionType>(PTy->getElementType());
2810b57cec5SDimitry Andric 
2820b57cec5SDimitry Andric     // If the function is casted to something like i8* as a "generic pointer"
2830b57cec5SDimitry Andric     // to be later casted to something else, we can't generate a wrapper for it.
2840b57cec5SDimitry Andric     // Just ignore such casts for now.
2850b57cec5SDimitry Andric     if (!Ty)
2860b57cec5SDimitry Andric       continue;
2870b57cec5SDimitry Andric 
2880b57cec5SDimitry Andric     auto Pair = Wrappers.insert(std::make_pair(std::make_pair(F, Ty), nullptr));
2890b57cec5SDimitry Andric     if (Pair.second)
2900b57cec5SDimitry Andric       Pair.first->second = createWrapper(F, Ty);
2910b57cec5SDimitry Andric 
2920b57cec5SDimitry Andric     Function *Wrapper = Pair.first->second;
2930b57cec5SDimitry Andric     if (!Wrapper)
2940b57cec5SDimitry Andric       continue;
2950b57cec5SDimitry Andric 
2960b57cec5SDimitry Andric     if (isa<Constant>(U->get()))
2970b57cec5SDimitry Andric       U->get()->replaceAllUsesWith(Wrapper);
2980b57cec5SDimitry Andric     else
2990b57cec5SDimitry Andric       U->set(Wrapper);
3000b57cec5SDimitry Andric   }
3010b57cec5SDimitry Andric 
3020b57cec5SDimitry Andric   // If we created a wrapper for main, rename the wrapper so that it's the
3030b57cec5SDimitry Andric   // one that gets called from startup.
3040b57cec5SDimitry Andric   if (CallMain) {
3050b57cec5SDimitry Andric     Main->setName("__original_main");
3060b57cec5SDimitry Andric     auto *MainWrapper =
3070b57cec5SDimitry Andric         cast<Function>(CallMain->getCalledValue()->stripPointerCasts());
3080b57cec5SDimitry Andric     delete CallMain;
3090b57cec5SDimitry Andric     if (Main->isDeclaration()) {
3100b57cec5SDimitry Andric       // The wrapper is not needed in this case as we don't need to export
3110b57cec5SDimitry Andric       // it to anyone else.
3120b57cec5SDimitry Andric       MainWrapper->eraseFromParent();
3130b57cec5SDimitry Andric     } else {
3140b57cec5SDimitry Andric       // Otherwise give the wrapper the same linkage as the original main
3150b57cec5SDimitry Andric       // function, so that it can be called from the same places.
3160b57cec5SDimitry Andric       MainWrapper->setName("main");
3170b57cec5SDimitry Andric       MainWrapper->setLinkage(Main->getLinkage());
3180b57cec5SDimitry Andric       MainWrapper->setVisibility(Main->getVisibility());
3190b57cec5SDimitry Andric     }
3200b57cec5SDimitry Andric   }
3210b57cec5SDimitry Andric 
3220b57cec5SDimitry Andric   return true;
3230b57cec5SDimitry Andric }
324