[PATCH] D118987: [analyzer] Add failing test case demonstrating buggy taint propagation

Wed Feb 23 04:15:48 PST 2022

steakhal added a comment.

In D118987#3332940 <https://reviews.llvm.org/D118987#3332940>, @jrtc27 wrote:

> In D118987#3319697 <https://reviews.llvm.org/D118987#3319697>, @steakhal wrote:
>
>> It seems like the `clang-ve-ninja` doesn't really want to accept any patches from me :D
>> I hope it's not personal. Let's be friends bot, please.
>>
>> Link to the breakage: https://lab.llvm.org/buildbot/#/builders/91/builds/3818
>>
>> I'm inviting @simoll for resolving this, and the underlying issue to prevent future breakages and reverts.
>
> As stated in https://discourse.llvm.org/t/the-angry-clang-ve-ninja-build-bot/60330/4, it's because the clang-ve-ninja bot does a build without assertions enabled, so you need REQUIRES: asserts or to write the test in a way that avoids the need for -debug-only.

It seems like it worked! Thanks again.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D118987/new/

https://reviews.llvm.org/D118987