This is an archive of the discontinued LLVM Phabricator instance.

[zorg] Delete two unused functions from UnifiedTreeBuilder
AbandonedPublic

Authored by Hahnfeld on Jan 13 2017, 1:04 AM.

Details

Reviewers
gkistanova

Diff Detail

Event Timeline

Hahnfeld updated this revision to Diff 84253.Jan 13 2017, 1:04 AM
Hahnfeld retitled this revision from to [zorg] Delete two unused functions from UnifiedTreeBuilder.
Hahnfeld updated this object.
Hahnfeld added a reviewer: gkistanova.
Hahnfeld added a subscriber: llvm-commits.
gkistanova requested changes to this revision.Jan 27 2017, 12:35 PM

These are unused yet and there for purpose.
Do these cause any problem for you? If so, could you elaborate, please?

This revision now requires changes to proceed.Jan 27 2017, 12:35 PM

No, they don't cause me any trouble directly. I just wanted to clean up a bit and remove some dead code as it's currently quite confusing which build factories get used.

Hahnfeld abandoned this revision.Feb 16 2017, 5:11 AM