Merge pull request rust-lang#31 from TheDan64/versioned_travis

Support for multiple LLVM versions, starting with 3.6

Author: TheDan64

.travis.yml

@@ -1,37 +1,63 @@
 language: rust
 
 sudo: false
+cache:
+  - apt
+  - cargo
 
-rust:
-  - stable
-  - beta
-  - nightly
+include_base: &BASE
+  addons:
+    apt:
+      sources: &BASE_SOURCES
+        - ubuntu-toolchain-r-test
+      packages: &BASE_PACKAGES
+        - libelf-dev
+        - lib32z1-dev
+        - libedit-dev
+        - libdw-dev
+        - binutils-dev
+        - libiberty-dev
+  before_install:
+    - export LLVM_VERSION_DASH=${LLVM_VERSION/./-}
+    - export LLVM_VERSION_SHORT=${LLVM_VERSION/./}
+    - export PATH=/usr/lib/llvm-${LLVM_VERSION}/bin/:$HOME/.local/bin:$PATH
+    - export LLVM_PATH=/usr/share/llvm-${LLVM_VERSION}/cmake/
+    - sed -i -e 's/llvm-sys = ".*"/llvm-sys = "'${LLVM_VERSION_SHORT}'"/' Cargo.toml
+  script:
+    - cargo test --no-default-features --features "llvm${LLVM_VERSION_DASH}" --verbose
 
 matrix:
   allow_failures:
     - rust: nightly
   fast_finish: true
+  include:
+    - env:
+        - LLVM_VERSION="3.6"
+      <<: *BASE
+      addons:
+        apt:
+          sources:
+            - *BASE_SOURCES
+            - llvm-toolchain-precise-3.6
+          packages:
+            - *BASE_PACKAGES
+            - llvm-3.6-dev
+
+    - env:
+        - LLVM_VERSION="3.7"
+      <<: *BASE
+      addons:
+        apt:
+          sources:
+            - *BASE_SOURCES
+            - llvm-toolchain-precise-3.7
+          packages:
+            - *BASE_PACKAGES
+            - llvm-3.7-dev
 
 env:
   global:
-  - RUSTFLAGS="-C link-dead-code"
-
-addons:
-  apt:
-    sources:
-      - ubuntu-toolchain-r-test
-      - llvm-toolchain-precise-3.7
-    packages:
-      - llvm-3.7-dev
-      - libelf-dev
-      - lib32z1-dev
-      - libedit-dev
-      - libdw-dev
-      - binutils-dev
-      - libiberty-dev
-
-before_install:
-  - export PATH=/usr/lib/llvm-3.7/bin/:$HOME/.local/bin:$PATH
+    - RUSTFLAGS="-C link-dead-code -C target-cpu=native"
 
 after_success: |
   wget https://github.com/SimonKagstrom/kcov/archive/master.tar.gz &&

Cargo.toml

@@ -10,11 +10,16 @@ keywords = ["llvm", "safe", "wrapper"]
 license = "Apache-2.0"
 categories = ["development-tools::ffi"]
 
+[features]
+default = []
+llvm3-6 = []
+llvm3-7 = []
+
 [dependencies]
-either = "1.1.0"
-enum-methods = "0.0.7"
+either = "1.4.0"
+enum-methods = "0.0.8"
 libc = "*"
-llvm-sys = "37.0.5" # TODO: Configure based on feature toggle. GH#1
+llvm-sys = "37"
 
 [[example]]
 name = "kaleidoscope"

README.md

@@ -14,18 +14,25 @@ Inkwell aims to help you pen your own programming languages by safely wrapping l
 
 * Any Rust version released in the last year or so
 * Rust Stable, Beta, or Nightly
-* LLVM 3.7 (3.6 and 3.8+ support is planned: [#1](https://github.com/TheDan64/inkwell/issues/1))
+* LLVM 3.6 or 3.7 (3.8+ support is planned: [#1](https://github.com/TheDan64/inkwell/issues/1))
 
 ## Usage
 
-You'll need to point your Cargo.toml to the master branch of Inkwell like so:
+You'll need to point your Cargo.toml to a branch and use a feature flag corresponding to a supported LLVM version:
 
 ```toml
 [dependencies]
-inkwell = { git = "https://github.com/TheDan64/inkwell", branch = "master" }
+inkwell = { git = "https://github.com/TheDan64/inkwell", branch = "llvm3-7", features = ["llvm3-7"] }
 ```
 
-In the root of your source code you will likely have to add an extern crate to begin using Inkwell:
+Supported versions:
+
+| GitHub Branch | Feature Flag |
+| :-----------: | :----------: |
+| llvm3-6       | llvm3-6      |
+| llvm3-7       | llvm3-7      |
+
+In the root of your source code you will have to add an extern crate to begin using Inkwell:
 
 ```rust
 extern crate inkwell;

src/lib.rs

@@ -23,6 +23,9 @@ use llvm_sys::support::LLVMLoadLibraryPermanently;
 
 use std::ffi::CString;
 
+#[cfg(not(any(feature = "llvm3-6", feature = "llvm3-7")))]
+compile_error!("A LLVM feature flag must be provided. See the README for more details.");
+
 // TODO: Probably move into error handling module
 pub fn enable_llvm_pretty_stack_trace() {
     // use llvm_sys::error_handling::LLVMEnablePrettyStackTrace; // v3.8

src/passes.rs

@@ -1,14 +1,18 @@
 use llvm_sys::core::{LLVMDisposePassManager, LLVMInitializeFunctionPassManager, LLVMFinalizeFunctionPassManager, LLVMRunFunctionPassManager, LLVMRunPassManager, LLVMCreatePassManager, LLVMCreateFunctionPassManagerForModule, LLVMGetGlobalPassRegistry};
 use llvm_sys::initialization::{LLVMInitializeCore, LLVMInitializeTransformUtils, LLVMInitializeScalarOpts, LLVMInitializeObjCARCOpts, LLVMInitializeVectorization, LLVMInitializeInstCombine, LLVMInitializeIPO, LLVMInitializeInstrumentation, LLVMInitializeAnalysis, LLVMInitializeIPA, LLVMInitializeCodeGen, LLVMInitializeTarget};
 use llvm_sys::prelude::{LLVMPassManagerRef, LLVMPassRegistryRef};
+#[cfg(not(feature = "llvm3-6"))]
 use llvm_sys::target::LLVMAddTargetData;
 use llvm_sys::transforms::ipo::{LLVMAddArgumentPromotionPass, LLVMAddConstantMergePass, LLVMAddDeadArgEliminationPass, LLVMAddFunctionAttrsPass, LLVMAddFunctionInliningPass, LLVMAddAlwaysInlinerPass, LLVMAddGlobalDCEPass, LLVMAddGlobalOptimizerPass, LLVMAddIPConstantPropagationPass, LLVMAddIPSCCPPass, LLVMAddInternalizePass, LLVMAddStripDeadPrototypesPass, LLVMAddPruneEHPass, LLVMAddStripSymbolsPass};
 use llvm_sys::transforms::pass_manager_builder::{LLVMPassManagerBuilderRef, LLVMPassManagerBuilderCreate, LLVMPassManagerBuilderDispose, LLVMPassManagerBuilderSetOptLevel, LLVMPassManagerBuilderSetSizeLevel, LLVMPassManagerBuilderSetDisableUnitAtATime, LLVMPassManagerBuilderSetDisableUnrollLoops, LLVMPassManagerBuilderSetDisableSimplifyLibCalls, LLVMPassManagerBuilderUseInlinerWithThreshold, LLVMPassManagerBuilderPopulateFunctionPassManager, LLVMPassManagerBuilderPopulateModulePassManager, LLVMPassManagerBuilderPopulateLTOPassManager};
-use llvm_sys::transforms::scalar::{LLVMAddAggressiveDCEPass, LLVMAddMemCpyOptPass, LLVMAddBitTrackingDCEPass, LLVMAddAlignmentFromAssumptionsPass, LLVMAddCFGSimplificationPass, LLVMAddDeadStoreEliminationPass, LLVMAddScalarizerPass, LLVMAddMergedLoadStoreMotionPass, LLVMAddGVNPass, LLVMAddIndVarSimplifyPass, LLVMAddInstructionCombiningPass, LLVMAddJumpThreadingPass, LLVMAddLICMPass, LLVMAddLoopDeletionPass, LLVMAddLoopIdiomPass, LLVMAddLoopRotatePass, LLVMAddLoopRerollPass, LLVMAddLoopUnrollPass, LLVMAddLoopUnswitchPass, LLVMAddPartiallyInlineLibCallsPass, LLVMAddLowerSwitchPass, LLVMAddPromoteMemoryToRegisterPass, LLVMAddSCCPPass, LLVMAddScalarReplAggregatesPass, LLVMAddScalarReplAggregatesPassSSA, LLVMAddScalarReplAggregatesPassWithThreshold, LLVMAddSimplifyLibCallsPass, LLVMAddTailCallEliminationPass, LLVMAddConstantPropagationPass, LLVMAddDemoteMemoryToRegisterPass, LLVMAddVerifierPass, LLVMAddCorrelatedValuePropagationPass, LLVMAddEarlyCSEPass, LLVMAddLowerExpectIntrinsicPass, LLVMAddTypeBasedAliasAnalysisPass, LLVMAddScopedNoAliasAAPass, LLVMAddBasicAliasAnalysisPass, LLVMAddReassociatePass};
+use llvm_sys::transforms::scalar::{LLVMAddAggressiveDCEPass, LLVMAddMemCpyOptPass, LLVMAddAlignmentFromAssumptionsPass, LLVMAddCFGSimplificationPass, LLVMAddDeadStoreEliminationPass, LLVMAddScalarizerPass, LLVMAddMergedLoadStoreMotionPass, LLVMAddGVNPass, LLVMAddIndVarSimplifyPass, LLVMAddInstructionCombiningPass, LLVMAddJumpThreadingPass, LLVMAddLICMPass, LLVMAddLoopDeletionPass, LLVMAddLoopIdiomPass, LLVMAddLoopRotatePass, LLVMAddLoopRerollPass, LLVMAddLoopUnrollPass, LLVMAddLoopUnswitchPass, LLVMAddPartiallyInlineLibCallsPass, LLVMAddLowerSwitchPass, LLVMAddPromoteMemoryToRegisterPass, LLVMAddSCCPPass, LLVMAddScalarReplAggregatesPass, LLVMAddScalarReplAggregatesPassSSA, LLVMAddScalarReplAggregatesPassWithThreshold, LLVMAddSimplifyLibCallsPass, LLVMAddTailCallEliminationPass, LLVMAddConstantPropagationPass, LLVMAddDemoteMemoryToRegisterPass, LLVMAddVerifierPass, LLVMAddCorrelatedValuePropagationPass, LLVMAddEarlyCSEPass, LLVMAddLowerExpectIntrinsicPass, LLVMAddTypeBasedAliasAnalysisPass, LLVMAddScopedNoAliasAAPass, LLVMAddBasicAliasAnalysisPass, LLVMAddReassociatePass};
+#[cfg(not(feature = "llvm3-6"))]
+use llvm_sys::transforms::scalar::LLVMAddBitTrackingDCEPass;
 use llvm_sys::transforms::vectorize::{LLVMAddBBVectorizePass, LLVMAddLoopVectorizePass, LLVMAddSLPVectorizePass};
 
 use OptimizationLevel;
 use module::Module;
+#[cfg(not(feature = "llvm3-6"))]
 use targets::TargetData;
 use values::{AsValueRef, FunctionValue};
 
@@ -159,6 +163,7 @@ impl PassManager {
         }
     }
 
+    #[cfg(not(feature = "llvm3-6"))]
     pub fn add_target_data(&self, target_data: &TargetData) {
         unsafe {
             LLVMAddTargetData(target_data.target_data, self.pass_manager)
@@ -274,7 +279,7 @@ impl PassManager {
         }
     }
 
-    // TODO: 3.7+ only
+    #[cfg(not(feature = "llvm3-6"))]
     pub fn add_bit_tracking_dce_pass(&self) {
         unsafe {
             LLVMAddBitTrackingDCEPass(self.pass_manager)

src/targets.rs

@@ -210,6 +210,36 @@ impl Target {
         }
     }
 
+    // TODOC: Called AMDGPU in 3.7+
+    #[cfg(feature = "llvm3-6")]
+    pub fn initialize_r600(config: &InitializationConfig) {
+        use llvm_sys::target::{LLVMInitializeR600Target, LLVMInitializeR600TargetInfo, LLVMInitializeR600TargetMC, LLVMInitializeR600AsmPrinter, LLVMInitializeR600AsmParser};
+
+        unsafe {
+            if config.base {
+                LLVMInitializeR600Target()
+            }
+
+            if config.info {
+                LLVMInitializeR600TargetInfo()
+            }
+
+            if config.asm_printer {
+                LLVMInitializeR600AsmPrinter()
+            }
+
+            if config.asm_parser {
+                LLVMInitializeR600AsmParser()
+            }
+
+            if config.machine_code {
+                LLVMInitializeR600TargetMC()
+            }
+        }
+    }
+
+    // TODOC: Called R600 in 3.6
+    #[cfg(not(feature = "llvm3-6"))]
     pub fn initialize_amd_gpu(config: &InitializationConfig) {
         use llvm_sys::target::{LLVMInitializeAMDGPUTarget, LLVMInitializeAMDGPUTargetInfo, LLVMInitializeAMDGPUTargetMC, LLVMInitializeAMDGPUAsmPrinter, LLVMInitializeAMDGPUAsmParser};
 
@@ -314,6 +344,7 @@ impl Target {
         }
     }
 
+    #[cfg(not(feature = "llvm3-6"))]
     pub fn initialize_cpp_backend(config: &InitializationConfig) {
         use llvm_sys::target::{LLVMInitializeCppBackendTarget, LLVMInitializeCppBackendTargetInfo, LLVMInitializeCppBackendTargetMC};
 
@@ -441,6 +472,7 @@ impl Target {
     }
 
     // TODOC: Disassembler only supported in LLVM 4.0+
+    #[cfg(not(feature = "llvm3-6"))]
     pub fn initialize_bpf(config: &InitializationConfig) {
         use llvm_sys::target::{LLVMInitializeBPFTarget, LLVMInitializeBPFTargetInfo, LLVMInitializeBPFTargetMC, LLVMInitializeBPFAsmPrinter};
 
@@ -860,7 +892,7 @@ impl TargetData {
         TargetData::new(target_data)
     }
 
-    // REVIEW: Maybe this should be pass_manager.add_target_data()?
+    // REVIEW: Maybe this should be pass_manager.add_target_data(&target_data)?
     pub fn add_target_data(&self, pass_manager: &PassManager) {
         unsafe {
             LLVMAddTargetData(self.target_data, pass_manager.pass_manager)

src/types/array_type.rs

@@ -86,6 +86,7 @@ impl ArrayType {
         self.array_type.print_to_string()
     }
 
+    #[cfg(not(feature = "llvm3-6"))]
     pub fn print_to_stderr(&self) {
         self.array_type.print_to_stderr()
     }

src/types/float_type.rs

@@ -126,6 +126,7 @@ impl FloatType {
         self.float_type.print_to_string()
     }
 
+    #[cfg(not(feature = "llvm3-6"))]
     pub fn print_to_stderr(&self) {
         self.float_type.print_to_stderr()
     }

src/types/fn_type.rs

@@ -65,6 +65,7 @@ impl FunctionType {
         self.fn_type.print_to_string()
     }
 
+    #[cfg(not(feature = "llvm3-6"))]
     pub fn print_to_stderr(&self) {
         self.fn_type.print_to_stderr()
     }

src/types/int_type.rs

@@ -311,6 +311,7 @@ impl IntType {
         self.int_type.print_to_string()
     }
 
+    #[cfg(not(feature = "llvm3-6"))]
     pub fn print_to_stderr(&self) {
         self.int_type.print_to_stderr()
     }