<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Sep 3, 2014 at 12:57 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 3 September 2014 10:32, Iain Sandoe <<a href="mailto:iain@codesourcery.com">iain@codesourcery.com</a>> wrote:<br>

> Hi Gents,<br>
><br>
> For those of us with out-of-tree backends which are not 32bit, the msp430 backend is a useful vehicle for examining changes and testing out ideas.<br>
><br>
> So I was wondering about enabling debug output on the MSP430 backend so that I can illustrate a few issues to Adrian and you on the variable pieces side.  (there doesn't appear to be any specific person claiming the msp430 code right now).<br>

><br>
> I have a question tho.<br>
><br>
> the following patch :<br>
><br>
> diff --git a/llvm/lib/Target/MSP430/MCTargetDesc/MSP430MCAsmInfo.cpp b/llvm/lib/Target/MSP430/MCTargetDesc/MSP430MCAsmInfo.cpp<br>
> index df1aa1a..844db93 100644<br>
> --- a/llvm/lib/Target/MSP430/MCTargetDesc/MSP430MCAsmInfo.cpp<br>
> +++ b/llvm/lib/Target/MSP430/MCTargetDesc/MSP430MCAsmInfo.cpp<br>
> @@ -24,4 +24,6 @@ MSP430MCAsmInfo::MSP430MCAsmInfo(StringRef TT) {<br>
><br>
>    AlignmentIsInBytes = false;<br>
>    UsesELFSectionDirectiveForBSS = true;<br>
> +<br>
> +  SupportsDebugInformation = true;<br>
>  }<br>
><br>
><br>
> does, indeed enable debug - but NOT .debug_frame (and IIRC some of the variable range info goes missing).<br>
><br>
> it needs:<br>
><br>
> diff --git a/llvm/lib/Target/MSP430/MCTargetDesc/MSP430MCAsmInfo.cpp b/llvm/lib/Target/MSP430/MCTargetDesc/MSP430MCAsmInfo.cpp<br>
> index df1aa1a..e7e0550 100644<br>
> --- a/llvm/lib/Target/MSP430/MCTargetDesc/MSP430MCAsmInfo.cpp<br>
> +++ b/llvm/lib/Target/MSP430/MCTargetDesc/MSP430MCAsmInfo.cpp<br>
> @@ -24,4 +24,7 @@ MSP430MCAsmInfo::MSP430MCAsmInfo(StringRef TT) {<br>
><br>
>    AlignmentIsInBytes = false;<br>
>    UsesELFSectionDirectiveForBSS = true;<br>
> +<br>
> +  SupportsDebugInformation = true;<br>
> +  ExceptionsType = ExceptionHandling::DwarfCFI;<br>
>  }<br>
><br>
> to make the debug_frame stuff happen.<br>
><br>
> Is that intentional?<br>
> (it's not a big deal, just a bit non-intuitive for backends which have no exception support).<br>
<br>
</div></div>I think that is the case. ExceptionsType is probably just a bad name<br>
for the frame unwind format.<br></blockquote><div><br></div><div>Yes. This is the case.</div><div><br></div><div>-eric </div></div></div></div>