[PATCH] D26173: [lli/COFF] Set the correct alignment for common symbols
    Reid Kleckner via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Nov  2 10:15:53 PDT 2016
    
    
  
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
================
Comment at: test/ExecutionEngine/MCJIT/coff-alignment.ll:5-8
+define void @main() {
+  %tmp23 = load i32, i32* @o, align 4
+  ret void
+}
----------------
Want to return the i32 from main so that this isn't fragile to codegen changes?
https://reviews.llvm.org/D26173
    
    
More information about the llvm-commits
mailing list