[llvm] r220055 - [Stackmaps] Enable invoking the patchpoint intrinsic.

Rafael Espíndola rafael.espindola at gmail.com
Tue Oct 21 07:38:56 PDT 2014


You should probably update the langref. It still has:

The llvm.donothing intrinsic doesn’t perform any operation. It’s the
only intrinsic that can be called with an invoke instruction.

On 17 October 2014 13:39, Juergen Ributzka <juergen at apple.com> wrote:
> Author: ributzka
> Date: Fri Oct 17 12:39:00 2014
> New Revision: 220055
>
> URL: http://llvm.org/viewvc/llvm-project?rev=220055&view=rev
> Log:
> [Stackmaps] Enable invoking the patchpoint intrinsic.
>
> Patch by Kevin Modzelewski
> Reviewers: atrick, ributzka
> Reviewed By: ributzka
> Subscribers: llvm-commits, reames
>
> Differential Revision: http://reviews.llvm.org/D5634
>
> Added:
>     llvm/trunk/test/CodeGen/X86/patchpoint-invoke.ll
> Modified:
>     llvm/trunk/include/llvm/IR/Intrinsics.td
>     llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
>     llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.h
>     llvm/trunk/lib/IR/Verifier.cpp
>     llvm/trunk/test/Verifier/invoke.ll
>
> Modified: llvm/trunk/include/llvm/IR/Intrinsics.td
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/IR/Intrinsics.td?rev=220055&r1=220054&r2=220055&view=diff
> ==============================================================================
> --- llvm/trunk/include/llvm/IR/Intrinsics.td (original)
> +++ llvm/trunk/include/llvm/IR/Intrinsics.td Fri Oct 17 12:39:00 2014
> @@ -483,11 +483,13 @@ def int_experimental_stackmap : Intrinsi
>  def int_experimental_patchpoint_void : Intrinsic<[],
>                                                   [llvm_i64_ty, llvm_i32_ty,
>                                                    llvm_ptr_ty, llvm_i32_ty,
> -                                                  llvm_vararg_ty]>;
> +                                                  llvm_vararg_ty],
> +                                                  [Throws]>;
>  def int_experimental_patchpoint_i64 : Intrinsic<[llvm_i64_ty],
>                                                  [llvm_i64_ty, llvm_i32_ty,
>                                                   llvm_ptr_ty, llvm_i32_ty,
> -                                                 llvm_vararg_ty]>;
> +                                                 llvm_vararg_ty],
> +                                                 [Throws]>;
>
>  //===-------------------------- Other Intrinsics --------------------------===//
>  //
>
> Modified: llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp?rev=220055&r1=220054&r2=220055&view=diff
> ==============================================================================
> --- llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp (original)
> +++ llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp Fri Oct 17 12:39:00 2014
> @@ -2008,8 +2008,17 @@ void SelectionDAGBuilder::visitInvoke(co
>    if (isa<InlineAsm>(Callee))
>      visitInlineAsm(&I);
>    else if (Fn && Fn->isIntrinsic()) {
> -    assert(Fn->getIntrinsicID() == Intrinsic::donothing);
> -    // Ignore invokes to @llvm.donothing: jump directly to the next BB.
> +    switch (Fn->getIntrinsicID()) {
> +    default:
> +      llvm_unreachable("Cannot invoke this intrinsic");
> +    case Intrinsic::donothing:
> +      // Ignore invokes to @llvm.donothing: jump directly to the next BB.
> +      break;
> +    case Intrinsic::experimental_patchpoint_void:
> +    case Intrinsic::experimental_patchpoint_i64:
> +      visitPatchpoint(&I, LandingPad);
> +      break;
> +    }
>    } else
>      LowerCallTo(&I, getValue(Callee), false, LandingPad);
>
> @@ -5429,7 +5438,7 @@ SelectionDAGBuilder::visitIntrinsicCall(
>    }
>    case Intrinsic::experimental_patchpoint_void:
>    case Intrinsic::experimental_patchpoint_i64: {
> -    visitPatchpoint(I);
> +    visitPatchpoint(&I);
>      return nullptr;
>    }
>    }
> @@ -6786,18 +6795,18 @@ void SelectionDAGBuilder::visitVACopy(co
>  /// convention or require stack pointer adjustment. Only a subset of the
>  /// intrinsic's operands need to participate in the calling convention.
>  std::pair<SDValue, SDValue>
> -SelectionDAGBuilder::LowerCallOperands(const CallInst &CI, unsigned ArgIdx,
> +SelectionDAGBuilder::lowerCallOperands(ImmutableCallSite CS, unsigned ArgIdx,
>                                         unsigned NumArgs, SDValue Callee,
> -                                       bool useVoidTy) {
> +                                       bool UseVoidTy,
> +                                       MachineBasicBlock *LandingPad) {
>    TargetLowering::ArgListTy Args;
>    Args.reserve(NumArgs);
>
>    // Populate the argument list.
>    // Attributes for args start at offset 1, after the return attribute.
> -  ImmutableCallSite CS(&CI);
>    for (unsigned ArgI = ArgIdx, ArgE = ArgIdx + NumArgs, AttrI = ArgIdx + 1;
>         ArgI != ArgE; ++ArgI) {
> -    const Value *V = CI.getOperand(ArgI);
> +    const Value *V = CS->getOperand(ArgI);
>
>      assert(!V->getType()->isEmptyTy() && "Empty type passed to intrinsic.");
>
> @@ -6808,13 +6817,13 @@ SelectionDAGBuilder::LowerCallOperands(c
>      Args.push_back(Entry);
>    }
>
> -  Type *retTy = useVoidTy ? Type::getVoidTy(*DAG.getContext()) : CI.getType();
> +  Type *retTy = UseVoidTy ? Type::getVoidTy(*DAG.getContext()) : CS->getType();
>    TargetLowering::CallLoweringInfo CLI(DAG);
>    CLI.setDebugLoc(getCurSDLoc()).setChain(getRoot())
> -    .setCallee(CI.getCallingConv(), retTy, Callee, std::move(Args), NumArgs)
> -    .setDiscardResult(!CI.use_empty());
> +    .setCallee(CS.getCallingConv(), retTy, Callee, std::move(Args), NumArgs)
> +    .setDiscardResult(CS->use_empty());
>
> -  return lowerInvokable(CLI, nullptr);
> +  return lowerInvokable(CLI, LandingPad);
>  }
>
>  /// \brief Add a stack map intrinsic call's live variable operands to a stackmap
> @@ -6834,11 +6843,11 @@ SelectionDAGBuilder::LowerCallOperands(c
>  /// assumption made by the llvm.gcroot intrinsic). If the alloca's location were
>  /// only available in a register, then the runtime would need to trap when
>  /// execution reaches the StackMap in order to read the alloca's location.
> -static void addStackMapLiveVars(const CallInst &CI, unsigned StartIdx,
> +static void addStackMapLiveVars(ImmutableCallSite CS, unsigned StartIdx,
>                                  SmallVectorImpl<SDValue> &Ops,
>                                  SelectionDAGBuilder &Builder) {
> -  for (unsigned i = StartIdx, e = CI.getNumArgOperands(); i != e; ++i) {
> -    SDValue OpVal = Builder.getValue(CI.getArgOperand(i));
> +  for (unsigned i = StartIdx, e = CS.arg_size(); i != e; ++i) {
> +    SDValue OpVal = Builder.getValue(CS.getArgument(i));
>      if (ConstantSDNode *C = dyn_cast<ConstantSDNode>(OpVal)) {
>        Ops.push_back(
>          Builder.DAG.getTargetConstant(StackMaps::ConstantOp, MVT::i64));
> @@ -6889,7 +6898,7 @@ void SelectionDAGBuilder::visitStackmap(
>                    cast<ConstantSDNode>(NBytesVal)->getZExtValue(), MVT::i32));
>
>    // Push live variables for the stack map.
> -  addStackMapLiveVars(CI, 2, Ops, *this);
> +  addStackMapLiveVars(&CI, 2, Ops, *this);
>
>    // We are not pushing any register mask info here on the operands list,
>    // because the stackmap doesn't clobber anything.
> @@ -6916,7 +6925,8 @@ void SelectionDAGBuilder::visitStackmap(
>  }
>
>  /// \brief Lower llvm.experimental.patchpoint directly to its target opcode.
> -void SelectionDAGBuilder::visitPatchpoint(const CallInst &CI) {
> +void SelectionDAGBuilder::visitPatchpoint(ImmutableCallSite CS,
> +                                          MachineBasicBlock *LandingPad) {
>    // void|i64 @llvm.experimental.patchpoint.void|i64(i64 <id>,
>    //                                                 i32 <numBytes>,
>    //                                                 i8* <target>,
> @@ -6924,29 +6934,29 @@ void SelectionDAGBuilder::visitPatchpoin
>    //                                                 [Args...],
>    //                                                 [live variables...])
>
> -  CallingConv::ID CC = CI.getCallingConv();
> -  bool isAnyRegCC = CC == CallingConv::AnyReg;
> -  bool hasDef = !CI.getType()->isVoidTy();
> -  SDValue Callee = getValue(CI.getOperand(2)); // <target>
> +  CallingConv::ID CC = CS.getCallingConv();
> +  bool IsAnyRegCC = CC == CallingConv::AnyReg;
> +  bool HasDef = !CS->getType()->isVoidTy();
> +  SDValue Callee = getValue(CS->getOperand(2)); // <target>
>
>    // Get the real number of arguments participating in the call <numArgs>
> -  SDValue NArgVal = getValue(CI.getArgOperand(PatchPointOpers::NArgPos));
> +  SDValue NArgVal = getValue(CS.getArgument(PatchPointOpers::NArgPos));
>    unsigned NumArgs = cast<ConstantSDNode>(NArgVal)->getZExtValue();
>
>    // Skip the four meta args: <id>, <numNopBytes>, <target>, <numArgs>
>    // Intrinsics include all meta-operands up to but not including CC.
>    unsigned NumMetaOpers = PatchPointOpers::CCPos;
> -  assert(CI.getNumArgOperands() >= NumMetaOpers + NumArgs &&
> +  assert(CS.arg_size() >= NumMetaOpers + NumArgs &&
>           "Not enough arguments provided to the patchpoint intrinsic");
>
>    // For AnyRegCC the arguments are lowered later on manually.
> -  unsigned NumCallArgs = isAnyRegCC ? 0 : NumArgs;
> +  unsigned NumCallArgs = IsAnyRegCC ? 0 : NumArgs;
>    std::pair<SDValue, SDValue> Result =
> -    LowerCallOperands(CI, NumMetaOpers, NumCallArgs, Callee, isAnyRegCC);
> +    lowerCallOperands(CS, NumMetaOpers, NumCallArgs, Callee, IsAnyRegCC,
> +                      LandingPad);
>
> -  SDValue Chain = Result.second;
> -  SDNode *CallEnd = Chain.getNode();
> -  if (hasDef && (CallEnd->getOpcode() == ISD::CopyFromReg))
> +  SDNode *CallEnd = Result.second.getNode();
> +  if (HasDef && (CallEnd->getOpcode() == ISD::CopyFromReg))
>      CallEnd = CallEnd->getOperand(0).getNode();
>
>    /// Get a call instruction from the call sequence chain.
> @@ -6954,16 +6964,16 @@ void SelectionDAGBuilder::visitPatchpoin
>    assert(CallEnd->getOpcode() == ISD::CALLSEQ_END &&
>           "Expected a callseq node.");
>    SDNode *Call = CallEnd->getOperand(0).getNode();
> -  bool hasGlue = Call->getGluedNode();
> +  bool HasGlue = Call->getGluedNode();
>
>    // Replace the target specific call node with the patchable intrinsic.
>    SmallVector<SDValue, 8> Ops;
>
>    // Add the <id> and <numBytes> constants.
> -  SDValue IDVal = getValue(CI.getOperand(PatchPointOpers::IDPos));
> +  SDValue IDVal = getValue(CS->getOperand(PatchPointOpers::IDPos));
>    Ops.push_back(DAG.getTargetConstant(
>                    cast<ConstantSDNode>(IDVal)->getZExtValue(), MVT::i64));
> -  SDValue NBytesVal = getValue(CI.getOperand(PatchPointOpers::NBytesPos));
> +  SDValue NBytesVal = getValue(CS->getOperand(PatchPointOpers::NBytesPos));
>    Ops.push_back(DAG.getTargetConstant(
>                    cast<ConstantSDNode>(NBytesVal)->getZExtValue(), MVT::i32));
>
> @@ -6976,8 +6986,8 @@ void SelectionDAGBuilder::visitPatchpoin
>    // Adjust <numArgs> to account for any arguments that have been passed on the
>    // stack instead.
>    // Call Node: Chain, Target, {Args}, RegMask, [Glue]
> -  unsigned NumCallRegArgs = Call->getNumOperands() - (hasGlue ? 4 : 3);
> -  NumCallRegArgs = isAnyRegCC ? NumArgs : NumCallRegArgs;
> +  unsigned NumCallRegArgs = Call->getNumOperands() - (HasGlue ? 4 : 3);
> +  NumCallRegArgs = IsAnyRegCC ? NumArgs : NumCallRegArgs;
>    Ops.push_back(DAG.getTargetConstant(NumCallRegArgs, MVT::i32));
>
>    // Add the calling convention
> @@ -6985,20 +6995,20 @@ void SelectionDAGBuilder::visitPatchpoin
>
>    // Add the arguments we omitted previously. The register allocator should
>    // place these in any free register.
> -  if (isAnyRegCC)
> +  if (IsAnyRegCC)
>      for (unsigned i = NumMetaOpers, e = NumMetaOpers + NumArgs; i != e; ++i)
> -      Ops.push_back(getValue(CI.getArgOperand(i)));
> +      Ops.push_back(getValue(CS.getArgument(i)));
>
>    // Push the arguments from the call instruction up to the register mask.
> -  SDNode::op_iterator e = hasGlue ? Call->op_end()-2 : Call->op_end()-1;
> +  SDNode::op_iterator e = HasGlue ? Call->op_end()-2 : Call->op_end()-1;
>    for (SDNode::op_iterator i = Call->op_begin()+2; i != e; ++i)
>      Ops.push_back(*i);
>
>    // Push live variables for the stack map.
> -  addStackMapLiveVars(CI, NumMetaOpers + NumArgs, Ops, *this);
> +  addStackMapLiveVars(CS, NumMetaOpers + NumArgs, Ops, *this);
>
>    // Push the register mask info.
> -  if (hasGlue)
> +  if (HasGlue)
>      Ops.push_back(*(Call->op_end()-2));
>    else
>      Ops.push_back(*(Call->op_end()-1));
> @@ -7008,15 +7018,15 @@ void SelectionDAGBuilder::visitPatchpoin
>    Ops.push_back(*(Call->op_begin()));
>
>    // Push the glue flag (last operand).
> -  if (hasGlue)
> +  if (HasGlue)
>      Ops.push_back(*(Call->op_end()-1));
>
>    SDVTList NodeTys;
> -  if (isAnyRegCC && hasDef) {
> +  if (IsAnyRegCC && HasDef) {
>      // Create the return types based on the intrinsic definition
>      const TargetLowering &TLI = DAG.getTargetLoweringInfo();
>      SmallVector<EVT, 3> ValueVTs;
> -    ComputeValueVTs(TLI, CI.getType(), ValueVTs);
> +    ComputeValueVTs(TLI, CS->getType(), ValueVTs);
>      assert(ValueVTs.size() == 1 && "Expected only one return value type.");
>
>      // There is always a chain and a glue type at the end
> @@ -7031,18 +7041,18 @@ void SelectionDAGBuilder::visitPatchpoin
>                                           getCurSDLoc(), NodeTys, Ops);
>
>    // Update the NodeMap.
> -  if (hasDef) {
> -    if (isAnyRegCC)
> -      setValue(&CI, SDValue(MN, 0));
> +  if (HasDef) {
> +    if (IsAnyRegCC)
> +      setValue(CS.getInstruction(), SDValue(MN, 0));
>      else
> -      setValue(&CI, Result.first);
> +      setValue(CS.getInstruction(), Result.first);
>    }
>
>    // Fixup the consumers of the intrinsic. The chain and glue may be used in the
>    // call sequence. Furthermore the location of the chain and glue can change
>    // when the AnyReg calling convention is used and the intrinsic returns a
>    // value.
> -  if (isAnyRegCC && hasDef) {
> +  if (IsAnyRegCC && HasDef) {
>      SDValue From[] = {SDValue(Call, 0), SDValue(Call, 1)};
>      SDValue To[] = {SDValue(MN, 1), SDValue(MN, 2)};
>      DAG.ReplaceAllUsesOfValuesWith(From, To, 2);
>
> Modified: llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.h
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.h?rev=220055&r1=220054&r2=220055&view=diff
> ==============================================================================
> --- llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.h (original)
> +++ llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.h Fri Oct 17 12:39:00 2014
> @@ -634,11 +634,13 @@ public:
>    void LowerCallTo(ImmutableCallSite CS, SDValue Callee, bool IsTailCall,
>                     MachineBasicBlock *LandingPad = nullptr);
>
> -  std::pair<SDValue, SDValue> LowerCallOperands(const CallInst &CI,
> -                                                unsigned ArgIdx,
> -                                                unsigned NumArgs,
> -                                                SDValue Callee,
> -                                                bool useVoidTy = false);
> +  std::pair<SDValue, SDValue> lowerCallOperands(
> +          ImmutableCallSite CS,
> +          unsigned ArgIdx,
> +          unsigned NumArgs,
> +          SDValue Callee,
> +          bool UseVoidTy = false,
> +          MachineBasicBlock *LandingPad = nullptr);
>
>    /// UpdateSplitBlock - When an MBB was split during scheduling, update the
>    /// references that need to refer to the last resulting block.
> @@ -778,7 +780,8 @@ private:
>    void visitVAEnd(const CallInst &I);
>    void visitVACopy(const CallInst &I);
>    void visitStackmap(const CallInst &I);
> -  void visitPatchpoint(const CallInst &I);
> +  void visitPatchpoint(ImmutableCallSite CS,
> +                       MachineBasicBlock *LandingPad = nullptr);
>
>    void visitUserOp1(const Instruction &I) {
>      llvm_unreachable("UserOp1 should not exist at instruction selection time!");
>
> Modified: llvm/trunk/lib/IR/Verifier.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/IR/Verifier.cpp?rev=220055&r1=220054&r2=220055&view=diff
> ==============================================================================
> --- llvm/trunk/lib/IR/Verifier.cpp (original)
> +++ llvm/trunk/lib/IR/Verifier.cpp Fri Oct 17 12:39:00 2014
> @@ -2212,11 +2212,15 @@ void Verifier::visitInstruction(Instruct
>      if (Function *F = dyn_cast<Function>(I.getOperand(i))) {
>        // Check to make sure that the "address of" an intrinsic function is never
>        // taken.
> -      Assert1(!F->isIntrinsic() || i == (isa<CallInst>(I) ? e-1 : 0),
> +      Assert1(!F->isIntrinsic() || i == (isa<CallInst>(I) ? e-1 :
> +                                         isa<InvokeInst>(I) ? e-3 : 0),
>                "Cannot take the address of an intrinsic!", &I);
>        Assert1(!F->isIntrinsic() || isa<CallInst>(I) ||
> -              F->getIntrinsicID() == Intrinsic::donothing,
> -              "Cannot invoke an intrinsinc other than donothing", &I);
> +              F->getIntrinsicID() == Intrinsic::donothing ||
> +              F->getIntrinsicID() == Intrinsic::experimental_patchpoint_void ||
> +              F->getIntrinsicID() == Intrinsic::experimental_patchpoint_i64,
> +              "Cannot invoke an intrinsinc other than"
> +              " donothing or patchpoint", &I);
>        Assert1(F->getParent() == M, "Referencing function in another module!",
>                &I);
>      } else if (BasicBlock *OpBB = dyn_cast<BasicBlock>(I.getOperand(i))) {
>
> Added: llvm/trunk/test/CodeGen/X86/patchpoint-invoke.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/patchpoint-invoke.ll?rev=220055&view=auto
> ==============================================================================
> --- llvm/trunk/test/CodeGen/X86/patchpoint-invoke.ll (added)
> +++ llvm/trunk/test/CodeGen/X86/patchpoint-invoke.ll Fri Oct 17 12:39:00 2014
> @@ -0,0 +1,65 @@
> +; RUN: llc -mtriple=x86_64-unknown-linux -mcpu=corei7                             < %s | FileCheck %s
> +
> +; Test invoking of patchpoints
> +;
> +define i64 @patchpoint_invoke(i64 %p1, i64 %p2) {
> +entry:
> +; CHECK-LABEL: patchpoint_invoke:
> +; CHECK-NEXT: .cfi_startproc
> +; CHECK:      [[FUNC_BEGIN:.L.*]]:
> +; CHECK:      .cfi_lsda 3, [[EXCEPTION_LABEL:.L[^ ]*]]
> +; CHECK:      pushq %rbp
> +
> +; Unfortunately, hardcode the name of the label that begins the patchpoint:
> +; CHECK:      .Ltmp0:
> +; CHECK:      movabsq $-559038736, %r11
> +; CHECK-NEXT: callq *%r11
> +; CHECK-NEXT: xchgw %ax, %ax
> +; CHECK-NEXT: [[PP_END:.L.*]]:
> +; CHECK:      ret
> +  %resolveCall = inttoptr i64 -559038736 to i8*
> +  %result = invoke i64 (i64, i32, i8*, i32, ...)* @llvm.experimental.patchpoint.i64(i64 2, i32 15, i8* %resolveCall, i32 1, i64 %p1, i64 %p2)
> +            to label %success unwind label %threw
> +
> +success:
> +  ret i64 %result
> +
> +threw:
> +  %0 = landingpad { i8*, i32 } personality i8* bitcast (i32 (...)* @__gxx_personality_v0 to i8*)
> +          catch i8* null
> +  ret i64 0
> +}
> +
> +; Verify that the exception table was emitted:
> +; CHECK:      [[EXCEPTION_LABEL]]:
> +; CHECK-NEXT: .byte 255
> +; CHECK-NEXT: .byte 3
> +; CHECK-NEXT: .byte 21
> +; CHECK-NEXT: .byte 3
> +; CHECK-NEXT: .byte 13
> +; Verify that the unwind data covers the entire patchpoint region:
> +; CHECK-NEXT: [[RANGE_OFFSET:.L[^ ]*]] = .Ltmp0-[[FUNC_BEGIN]]
> +; CHECK-NEXT: .long [[RANGE_OFFSET]]
> +; CHECK-NEXT: [[RANGE_LENGTH:.L[^ ]*]] = [[PP_END]]-.Ltmp0
> +; CHECK-NEXT: .long [[RANGE_LENGTH]]
> +
> +
> +; Verify that the stackmap section got emitted:
> +; CHECK-LABEL: __LLVM_StackMaps:
> +; Header
> +; CHECK-NEXT:   .byte 1
> +; CHECK-NEXT:   .byte 0
> +; CHECK-NEXT:   .short 0
> +; Num Functions
> +; CHECK-NEXT:   .long 1
> +; Num LargeConstants
> +; CHECK-NEXT:   .long 0
> +; Num Callsites
> +; CHECK-NEXT:   .long 1
> +; CHECK-NEXT:   .quad patchpoint_invoke
> +
> +
> +declare void @llvm.experimental.stackmap(i64, i32, ...)
> +declare void @llvm.experimental.patchpoint.void(i64, i32, i8*, i32, ...)
> +declare i64 @llvm.experimental.patchpoint.i64(i64, i32, i8*, i32, ...)
> +declare i32 @__gxx_personality_v0(...)
>
> Modified: llvm/trunk/test/Verifier/invoke.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Verifier/invoke.ll?rev=220055&r1=220054&r2=220055&view=diff
> ==============================================================================
> --- llvm/trunk/test/Verifier/invoke.ll (original)
> +++ llvm/trunk/test/Verifier/invoke.ll Fri Oct 17 12:39:00 2014
> @@ -46,7 +46,7 @@ contb:
>
>  define i8 @f2() {
>  entry:
> -; CHECK: Cannot invoke an intrinsinc other than donothing
> +; CHECK: Cannot invoke an intrinsinc other than donothing or patchpoint
>    invoke void @llvm.trap()
>    to label %cont unwind label %lpad
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list