Index: llvm/trunk/lib/FuzzMutate/Operations.cpp =================================================================== --- llvm/trunk/lib/FuzzMutate/Operations.cpp +++ llvm/trunk/lib/FuzzMutate/Operations.cpp @@ -216,8 +216,9 @@ static SourcePred matchScalarInAggregate() { auto Pred = [](ArrayRef Cur, const Value *V) { - if (isa(Cur[0]->getType())) - return V->getType() == Cur[0]->getType(); + if (auto *ArrayT = dyn_cast(Cur[0]->getType())) + return V->getType() == ArrayT->getElementType(); + auto *STy = cast(Cur[0]->getType()); for (int I = 0, E = STy->getNumElements(); I < E; ++I) if (STy->getTypeAtIndex(I) == V->getType()) @@ -225,8 +226,9 @@ return false; }; auto Make = [](ArrayRef Cur, ArrayRef) { - if (isa(Cur[0]->getType())) - return makeConstantsWithType(Cur[0]->getType()); + if (auto *ArrayT = dyn_cast(Cur[0]->getType())) + return makeConstantsWithType(ArrayT->getElementType()); + std::vector Result; auto *STy = cast(Cur[0]->getType()); for (int I = 0, E = STy->getNumElements(); I < E; ++I) Index: llvm/trunk/unittests/FuzzMutate/RandomIRBuilderTest.cpp =================================================================== --- llvm/trunk/unittests/FuzzMutate/RandomIRBuilderTest.cpp +++ llvm/trunk/unittests/FuzzMutate/RandomIRBuilderTest.cpp @@ -164,4 +164,40 @@ } } +TEST(RandomIRBuilderTest, InsertValueArray) { + // Check that we can generate insertvalue for the vector operations + + LLVMContext Ctx; + const char *SourceCode = + "define void @test() {\n" + " %A = alloca [8 x i32]\n" + " %L = load [8 x i32], [8 x i32]* %A" + " ret void\n" + "}"; + auto M = parseAssembly(SourceCode, Ctx); + + fuzzerop::OpDescriptor Descr = fuzzerop::insertValueDescriptor(1); + + std::vector Types = + {Type::getInt8Ty(Ctx), Type::getInt32Ty(Ctx), Type::getInt64Ty(Ctx)}; + RandomIRBuilder IB(Seed, Types); + + // Get first basic block of the first function + Function &F = *M->begin(); + BasicBlock &BB = *F.begin(); + + // Pick first source + Instruction *Source = &*std::next(BB.begin()); + ASSERT_TRUE(Descr.SourcePreds[0].matches({}, Source)); + + SmallVector Srcs(2); + + // Check that we can always pick the last two operands. + for (int i = 0; i < 10; ++i) { + Srcs[0] = Source; + Srcs[1] = IB.findOrCreateSource(BB, {Source}, Srcs, Descr.SourcePreds[1]); + IB.findOrCreateSource(BB, {}, Srcs, Descr.SourcePreds[2]); + } +} + }