Re: r361340 - [Analysis] Link library dependencies to Analysis plugins

2019-05-25 Thread Don Hinton via cfe-commits
I've submitted a patch to fix this, https://reviews.llvm.org/D62445. Without it, non of the bots can build/test any plugins. On Fri, May 24, 2019 at 5:47 PM Akira Hatanaka via cfe-commits < cfe-commits@lists.llvm.org> wrote: > I reverted the patch in r361685 to make the bot green again. > > >

Re: r361340 - [Analysis] Link library dependencies to Analysis plugins

2019-05-24 Thread Akira Hatanaka via cfe-commits
I reverted the patch in r361685 to make the bot green again. > On May 23, 2019, at 5:22 PM, Akira Hatanaka wrote: > > Hi Petr, > > This seems to have caused Analysis/checker-plugins.c to fail. Can you > investigate it or revert your commit? > >

Re: r361340 - [Analysis] Link library dependencies to Analysis plugins

2019-05-23 Thread Akira Hatanaka via cfe-commits
Hi Petr, This seems to have caused Analysis/checker-plugins.c to fail. Can you investigate it or revert your commit? http://green.lab.llvm.org/green/job/clang-stage2-cmake-RgSan/6381/consoleFull#-5275661368254eaf0-7326-4999-85b0-388101f2d404 > On May 21, 2019, at 5:47 PM, Petr Hosek via

r361340 - [Analysis] Link library dependencies to Analysis plugins

2019-05-21 Thread Petr Hosek via cfe-commits
Author: phosek Date: Tue May 21 17:47:37 2019 New Revision: 361340 URL: http://llvm.org/viewvc/llvm-project?rev=361340=rev Log: [Analysis] Link library dependencies to Analysis plugins These are needed to avoid undefined symbols which aren't satisfied by Clang itself. Differential Revision: