ash mis-compiles with clang

[Yunlian Jiang]

Yes, this patch works.

Thanks

On Thu, Jul 27, 2017 at 6:32 AM, Denys Vlasenko
<vda.linux at googlemail.com> wrote:
> On Wed, Jul 26, 2017 at 10:40 PM, Yunlian Jiang <yunlian at google.com> wrote:
>> Hi,
>>    When I try to build busybox with clang, I got some errors on ash at
>> runtime.
>>    https://bugs.chromium.org/p/chromium/issues/detail?id=659834
>>
>>    It seems that root cause is clang adds the attribute __noreturn__ to
>> the function evaltree.
>>
>>    Can we fix this to make it build with clang? I can provide a patch to
>> remove
>> the attribute  __noreturn__ from evaltreenr if the compiler is clang.
>>
>>    Does that sounds a reasonable fix?
>
> Tried that, it throws more warning about uninited variables.
>
> Does this patch help?