I'll revert the change. On Wed, Apr 3, 2019 at 1:00 PM Yitzhak Mandelbaum <yitzh...@google.com> wrote:
> https://reviews.llvm.org/D60213 for the record > > On Wed, Apr 3, 2019 at 12:55 PM Yitzhak Mandelbaum <yitzh...@google.com> > wrote: > >> Thanks. Do you have build command I can run before/after to verify my >> fix before I submit? >> >> On Wed, Apr 3, 2019 at 12:43 PM Alexey Bataev via Phabricator < >> revi...@reviews.llvm.org> wrote: >> >>> ABataev added a comment. >>> >>> Patch breaks the build with the shared libraries, for example, >>> http://lab.llvm.org:8011/builders/clang-ppc64le-linux-multistage/builds/9498. >>> Seems to me, you need to add an extra dependency on clangTooling. >>> clangToolingCore is not enough. >>> >>> >>> Repository: >>> rC Clang >>> >>> CHANGES SINCE LAST ACTION >>> https://reviews.llvm.org/D59376/new/ >>> >>> https://reviews.llvm.org/D59376 >>> >>> >>> >>>
smime.p7s
Description: S/MIME Cryptographic Signature
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits