This patch adds a few things to LLVM that were necessary to bootstrap Rust on Bitrig.
Details
Diff Detail
Event Timeline
lib/Target/X86/X86Subtarget.h | ||
---|---|---|
396 ↗ | (On Diff #19704) | Seems unused? |
I can't figure out how to update this patch so I'm abandoning it and creating a new one.
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.
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.
Ooops, the previous diff was in the wrong direction. This one is in the correct direction.
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.
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
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