r210333 - Quieting a false-positive which was causing the sanitizer bots to go red.
aaron at aaronballman.com
Fri Jun 6 08:10:32 PDT 2014
Not only was this not a sanitizer issue, it was a valid warning from
clang, so I guess I'm 0/2 on this one. I've made an improved commit in
r210338 that uses llvm_unreachable to silence the warning.
On Fri, Jun 6, 2014 at 10:55 AM, Aaron Ballman <aaron at aaronballman.com> wrote:
> On Fri, Jun 6, 2014 at 10:12 AM, Kostya Serebryany <kcc at google.com> wrote:
>> Just to make sure, did you mean this:
>> This is clang build using -DLLVM_ENABLE_WERROR=ON and report comes from
>> clang, sanitizers are not involved, right?
>> If you think this is a false positive, shouldn't we fix it in clang?
> Ah, yes, I did mean that, but I see now that it's not a sanitizer
> that's at fault. Things suddenly make more sense. :-) Yes, I do think
> we should fix it in clang. I'll see if I can repro in a smaller test
> case and report it.
More information about the cfe-commits