[PATCH] D36919: [ThinLTO] Fix ThinLTO crash

[David Li via Phabricator via llvm-commits]

davidxl added inline comments.


================
Comment at: test/Transforms/FunctionImport/Inputs/funcimport_var2.ll:2
+target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-unknown-linux-gnu"
+
----------------
Are these necessary?


================
Comment at: test/Transforms/FunctionImport/funcimport_var.ll:8
+; RUN: llvm-lto2 run %t.bc %t2.bc -o %t.out \
+; RUN:   -r %t.bc,_Z4LinkPKcS0_,plx \
+; RUN:   -r %t.bc,link,l \
----------------
line wrap intended?


================
Comment at: test/Transforms/FunctionImport/funcimport_var.ll:10
+; RUN:   -r %t.bc,link,l \
+; RUN:   -r %t2.bc,get_link,plx
+
----------------
what is the difference beween llvm-lto2 and llvm-lto? what are these  options for?


https://reviews.llvm.org/D36919