[llvm-dev] llvm-pdbutil status?

89 views
Skip to first unread message

George Karpenkov via llvm-dev

unread,
Aug 22, 2017, 5:45:43 PM8/22/17
to llvm-dev, Justin Bogner, Reid Kleckner, Kostya Serebrianyi, Vitaly Buka
Hi All,

As a part of a recent move of libFuzzer from LLVM to compiler-rt I am looking into updating the build code
for the libraries which use libFuzzer.

I have looked into tools/llvm-pdbutil, and “ninja llvm-pdbutil-fuzzer” does not build at all,
and the error refers to non-existent headers.
The last update to the codebase is reasonably recent, and was done in June 2017.
Is anyone using it?

Thus I am not sure what should be done with this.
I think we should either:

a) Move “tools/llvm-pdbutil/fuzzer” to a repository outside of LLVM default git checkout if no one is maintaining it,
OR
b) Make sure it compiles and setup a buildbot for it.

Suggestions?

Regards,
George
_______________________________________________
LLVM Developers mailing list
llvm...@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev

Kostya Serebryany via llvm-dev

unread,
Aug 22, 2017, 5:48:26 PM8/22/17
to George Karpenkov, Zachary Turner, llvm-dev
+zturner

Reid Kleckner via llvm-dev

unread,
Aug 22, 2017, 5:49:13 PM8/22/17
to George Karpenkov, David Majnemer, llvm-dev
I think we should delete it. David implemented it a long time ago when he was working on llvm-pdbdump, but he's moved on. Zach was never able to use it effectively because he works on Windows, so the reports were not actionable.

On Tue, Aug 22, 2017 at 2:45 PM, George Karpenkov <ekarp...@apple.com> wrote:

Zachary Turner via llvm-dev

unread,
Aug 22, 2017, 5:50:51 PM8/22/17
to Reid Kleckner, George Karpenkov, David Majnemer, llvm-dev
+1

_______________________________________________

David Blaikie via llvm-dev

unread,
Aug 22, 2017, 5:51:51 PM8/22/17
to Reid Kleckner, George Karpenkov, David Majnemer, llvm-dev
What makes the reports not actionable on windows?

_______________________________________________

Zachary Turner via llvm-dev

unread,
Aug 22, 2017, 5:54:33 PM8/22/17
to David Blaikie, Reid Kleckner, George Karpenkov, David Majnemer, llvm-dev
libfuzzer has historically not supported Windows, so the reports were not actionable.  Marcos got it working a few months ago and we had it up on a bot, but the bot was turned down recently as we don't have time / headcount to keep it working in the presence of other invasive changes happening on the sanitizer side.

George Karpenkov via llvm-dev

unread,
Aug 22, 2017, 6:43:06 PM8/22/17
to Zachary Turner, llvm-dev, David Majnemer
Proposed llvm-pdbutil/fuzzer for deletion in https://reviews.llvm.org/D37032
Reply all
Reply to author
Forward
0 new messages