[compiler-rt] r248653 - sanitizer: fix -pedantic warnings

[Saleem Abdulrasool via llvm-commits]

On Sun, Sep 27, 2015 at 9:25 AM, Saleem Abdulrasool <compnerd at compnerd.org>
wrote:

> On Sat, Sep 26, 2015 at 11:16 AM, Renato Golin <renato.golin at linaro.org>
> wrote:
>
>> On 26 September 2015 at 05:40, Saleem Abdulrasool via llvm-commits
>> <llvm-commits at lists.llvm.org> wrote:
>> > Author: compnerd
>> > Date: Fri Sep 25 23:40:42 2015
>> > New Revision: 248653
>> >
>> > URL: http://llvm.org/viewvc/llvm-project?rev=248653&view=rev
>> > Log:
>> > sanitizer: fix -pedantic warnings
>> >
>> > Silence a few "warning: extra ';'" warnings from GCC.
>>
>> Hi Saleem,
>>
>> I believe one of yours, not necessarily this one, got a twitch on
>> AddressSanitizer-armhf-linux::debug_stacks.cc
>>
>
> Yeah, rather unlikely to be this commit would cause this unless its a
> flakey test.
>
>
>>
>> http://lab.llvm.org:8011/builders/clang-cmake-thumbv7-a15-full-sh/builds/2390
>>
>> You didn't get the email because the bot was broken due to your
>> previous commit, which got fixed in that build.
>>
>> This could have something to do with the dual library hard/soft.
>>
>
> Plausibly.
>

Im not convinced that the dual library commit is related.  I don't seem to
be able to reproduce the failure locally.

<stdin>:17:13: note: with variable "FREE_FRAME_1" equal to "0xffffffff"

Seems to indicate that `__asan_get_free_stack` seems to be reporting -1 for
the PC, which is really unlikely to be correct.  It would be interesting to
get the full output from the failure.


> Can you have a look?
>>
>
> Sure.
>
>
>> cheers,
>> --renato
>>
>
>
>
> --
> Saleem Abdulrasool
> compnerd (at) compnerd (dot) org
>



-- 
Saleem Abdulrasool
compnerd (at) compnerd (dot) org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150927/897a0f49/attachment.html>