<div dir="ltr">Yes, I think we should continue emit unwind info into .eh_frame on x86 by default, even for C code that doesn't use EH. It's nice to have unwind information in the binary so that you can unwind the stack anywhere in the program.<div><br></div><div>I think that's the right default, I'm not sure why it's even architecture specific, though. I'd rather enable -funwind-tables by default on more architectures. Maybe the default -funwind-tables logic in clang should be a blacklist of legacy ISAs that don't want .eh_frame by default instead of whitelist of where to enable it by default.</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 27, 2018 at 7:59 AM Stefan Pintilie via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><font size="2" face="sans-serif">Hi All, </font><br><br><font size="2" face="sans-serif">Recently I fixed a bug that was exposed
on PowerPC relating to missing unwind tables.</font><br><br><font size="2" face="sans-serif">Bug:</font><br><a href="https://bugs.llvm.org/show_bug.cgi?id=32611" target="_blank"><font size="2" color="blue" face="sans-serif">https://bugs.llvm.org/show_bug.cgi?id=32611</font></a><br><br><font size="2" face="sans-serif">Patch For Fix:</font><br><a href="https://reviews.llvm.org/D50987" target="_blank"><font size="2" color="blue" face="sans-serif">https://reviews.llvm.org/D50987</font></a><br><br><font size="2" face="sans-serif">The bug was not exposed on x86 because
the option -funwind-tables is on by default and so the uwtable attribute
is added to all functions on x86. With the fix an unwind table is added
to any function that has an error handling personality. My questions: 1)
Why is -funwind-tables default on x86? 2) Does it still need to be default
now that this fix is in?</font><br><br><font size="2" face="sans-serif">Thank you, </font><br><font size="2" face="sans-serif">Stefan </font><br><br>_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>