[PATCH] D35701: Break up Targets.cpp into a header/impl pair per target type[NFCI]

Kamil Rytarowski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 20 15:28:36 PDT 2017


krytarowski added a comment.

In https://reviews.llvm.org/D35701#816703, @erichkeane wrote:

> In https://reviews.llvm.org/D35701#816680, @krytarowski wrote:
>
> > Bitrig has to be eliminated - it's dead upstream and will be removed from LLVM. (confirmed from ex developers, nobody needs it any more)
>
>
> I just did a bit of looking about, and noticed that the "Bitrig" token is in a bunch of places.  I suspect what we need is for a single pair of patches (llvm & clang) dedicated to removing it.  I'd prefer to do those in a dedicated patchset if thats OK.


Please go for it.

Bitrig code has been merged back to OpenBSD.


https://reviews.llvm.org/D35701





More information about the cfe-commits mailing list