This is an archive of the discontinued LLVM Phabricator instance.

Clang counterpart change for fuzzer FreeBSD support
ClosedPublic

Authored by devnexen on Jan 7 2018, 10:02 AM.

Diff Detail

Event Timeline

devnexen created this revision.Jan 7 2018, 10:02 AM
kimgr requested changes to this revision.Jan 7 2018, 11:08 AM
kimgr added a subscriber: kimgr.

Typo in the commit title: buzzer :)

This revision now requires changes to proceed.Jan 7 2018, 11:08 AM
devnexen retitled this revision from Clang counterpart change for buzzer FreeBSD support to Clang counterpart change for fuzzer FreeBSD support.Jan 7 2018, 11:11 AM
devnexen updated this revision to Diff 129329.Jan 10 2018, 1:13 PM
devnexen updated this revision to Diff 129364.Jan 10 2018, 4:05 PM

Typo in the commit title: buzzer :)

Changed ... just to follow-up the now accepted change of this one https://reviews.llvm.org/D41642

Looks good, please improve the description of the commit (and not your authorship that will be preserved in the commit message).

Once accepted by another developer, I can commit it for you.

devnexen edited the summary of this revision. (Show Details)Jan 12 2018, 10:30 AM
sbc100 removed a reviewer: sbc100.Jan 12 2018, 11:30 AM
martell accepted this revision.Jan 13 2018, 5:47 PM

Given that https://reviews.llvm.org/rL322380 has been merged this seems good to me.

This revision was not accepted when it landed; it landed in state Needs Review.Jan 14 2018, 9:58 PM
This revision was automatically updated to reflect the committed changes.

Committed as SVN r. 322469.