[PATCH] D42720: [CodeGen] Switch non-SJLJ EH encoding to uleb128

Ryan Prichard via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 8 16:48:36 PST 2018


Can you commit these changes for me? I don't have commit access.

Is there anything I can do to help test them? So far, I've built a stage 2
toolchain and verified that check-llvm, check-libcxx, and check-libcxxabi
all pass. It wasn't clear to me that any of the check-llvm tests tested
SJLJ, so I manually inspected the compiler output of "-target arm-darwin".

Thanks for reviewing the changes.

-Ryan


On Wed, Feb 7, 2018 at 2:27 PM, Ryan Prichard <rprichard at google.com> wrote:

> On Wed, Feb 7, 2018 at 9:49 AM, Rafael Avila de Espindola <
> rafael.espindola at gmail.com> wrote:
>
>> With this change AsmPrinter::EmitPaddedULEB128 is dead, please delete
>> it.
>>
>
> I noticed that MCStreamer::EmitPaddedULEB128IntValue was also dead, so I
> removed both it and AsmPrinter::EmitPaddedULEB128.
>
> -Ryan
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180208/8f620c94/attachment.html>


More information about the llvm-commits mailing list