[llvm] r179809 - Assert if we're trying to generate a null compact unwind entry.

[Eric Christopher]

Testcase? :)

-eric

On Thu, Apr 18, 2013 at 3:56 PM, Bill Wendling <isanbard at gmail.com> wrote:
> Author: void
> Date: Thu Apr 18 17:56:05 2013
> New Revision: 179809
>
> URL: http://llvm.org/viewvc/llvm-project?rev=179809&view=rev
> Log:
> Assert if we're trying to generate a null compact unwind entry.
>
> Modified:
>     llvm/trunk/lib/MC/MCDwarf.cpp
>
> Modified: llvm/trunk/lib/MC/MCDwarf.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/MCDwarf.cpp?rev=179809&r1=179808&r2=179809&view=diff
> ==============================================================================
> --- llvm/trunk/lib/MC/MCDwarf.cpp (original)
> +++ llvm/trunk/lib/MC/MCDwarf.cpp Thu Apr 18 17:56:05 2013
> @@ -1170,8 +1170,7 @@ void FrameEmitterImpl::EmitCompactUnwind
>    //   .quad except_tab1
>
>    uint32_t Encoding = Frame.CompactUnwindEncoding;
> -  if (!Encoding) return;
> -
> +  assert(Encoding && "There should never be a null compact unwind encoding!");
>    bool DwarfEHFrameOnly = (Encoding == MOFI->getCompactUnwindDwarfEHFrameOnly());
>
>    // The encoding needs to know we have an LSDA.
> @@ -1486,8 +1485,7 @@ void MCDwarfFrameEmitter::Emit(MCStreame
>    if (IsEH && MOFI->getCompactUnwindSection())
>      for (unsigned i = 0, n = Streamer.getNumFrameInfos(); i < n; ++i) {
>        const MCDwarfFrameInfo &Frame = Streamer.getFrameInfo(i);
> -      if (Frame.CompactUnwindEncoding)
> -        Emitter.EmitCompactUnwind(Streamer, Frame);
> +      Emitter.EmitCompactUnwind(Streamer, Frame);
>      }
>
>    const MCSection &Section = IsEH ? *MOFI->getEHFrameSection() :
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits