This is an archive of the discontinued LLVM Phabricator instance.

Upstreaming changes to support Rust on Bitrig
ClosedPublic

Authored by dhuseby on Feb 10 2015, 12:59 PM.

Details

Reviewers
echristo
Summary

This patch adds a few things to LLVM that were necessary to bootstrap Rust on Bitrig.

Diff Detail

Event Timeline

dhuseby updated this revision to Diff 19704.Feb 10 2015, 12:59 PM
dhuseby retitled this revision from to Upstreaming changes to support Rust on Bitrig.
dhuseby updated this object.
dhuseby edited the test plan for this revision. (Show Details)
dhuseby added a subscriber: Unknown Object (MLST).
echristo added inline comments.
lib/Target/X86/X86Subtarget.h
396 ↗(On Diff #19704)

Seems unused?

dhuseby abandoned this revision.Feb 10 2015, 4:53 PM

I can't figure out how to update this patch so I'm abandoning it and creating a new one.

dhuseby reclaimed this revision.Feb 10 2015, 4:55 PM

nvrmnd, figured it out.

dhuseby updated this revision to Diff 19720.Feb 10 2015, 4:56 PM

this removes the unused code.

Can I get some review love on this patch? This should be straight forward. Thanks!

Some comments:

a) Don't edit configure by hand. Edit configure.ac and regenerate.
b) Why the Makefile.config.in change?

-eric

The changes to Makefile.config.in came from a a patch in the Bitrig LLVM port. I reworked it so that it was conditional and only operative when building for Bitrig.

I'll update the patch to change configure.ac instead.

dhuseby updated this revision to Diff 20601.Feb 24 2015, 9:45 AM

Updated the patch by modifying autoconf/configure.ac and re-running autoconf/AutoRegen.sh as per review feedback.

The changes to Makefile.config.in are to get the output paths for the documentation correct for Bitrig only when compiling for Bitrig.

dhuseby updated this revision to Diff 20602.Feb 24 2015, 9:47 AM

Ooops, the previous diff was in the wrong direction. This one is in the correct direction.

dhuseby updated this revision to Diff 20603.Feb 24 2015, 9:50 AM

On second thought, I removed the changes to Makefile.config.in. Bitrig can continue to patch that as part of its port. There's no need to dirty up the Makefile for a local quirk of one OS.

The rest of the patch is necessary to get llvm to properly detect building for Bitrig.

echristo accepted this revision.Feb 24 2015, 9:57 AM
echristo added a reviewer: echristo.

Thanks, I wasn't fond of the Makefile.config.in part :)

LGTM. Do you have commit access or do you need someone to do it?

-eric

This revision is now accepted and ready to land.Feb 24 2015, 9:57 AM

I do not have commit access. I need somebody to land it for me.

Thanks!

Applied here:

dzur:~/sources/llvm> git svn dcommit
Committing to https://llvm.org/svn/llvm-project/llvm/trunk ...
M autoconf/config.sub
M autoconf/configure.ac
M configure
Committed r230669

Thanks!

-eric

echristo closed this revision.Feb 26 2015, 11:49 AM