Skip to content

Commit

Permalink
Merge pull request #35270 from JuliaLang/vc/llvm10
Browse files Browse the repository at this point in the history
add patch list for LLVM 10
  • Loading branch information
vchuravy committed Mar 29, 2020
2 parents f371270 + 614f644 commit ac22a6d
Show file tree
Hide file tree
Showing 4 changed files with 17 additions and 9 deletions.
1 change: 1 addition & 0 deletions deps/checksums/llvm-10.0.0.src.tar.xz/md5
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
693cefdc49d618f828144486a18b473f
1 change: 1 addition & 0 deletions deps/checksums/llvm-10.0.0.src.tar.xz/sha512
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
7dc961aacee3a01ecc002ff2b688a2ef50661856d2abd5ecc90566ffcad7566e4976736cd339ea96592e452cd5a17aaceba9712b2effec805661cca8ff020ee7
18 changes: 12 additions & 6 deletions deps/llvm.mk
Original file line number Diff line number Diff line change
Expand Up @@ -182,13 +182,7 @@ endif

ifneq ($(LLVM_VER),svn)
ifeq (,$(findstring rc,$(LLVM_VER)))
ifeq ($(shell [ x"$(LLVM_VER)" = x"8.0.1" ]; echo $$?),0)
LLVM_SRC_URL := https://github.com/llvm/llvm-project/releases/download/llvmorg-$(LLVM_VER)
else ifeq ($(shell [ x"$(LLVM_VER)" = x"9.0.1" ]; echo $$?),0)
LLVM_SRC_URL := https://github.com/llvm/llvm-project/releases/download/llvmorg-$(LLVM_VER)
else
LLVM_SRC_URL := http:https://releases.llvm.org/$(LLVM_VER)
endif
else
LLVM_VER_SPLIT := $(subst rc, ,$(LLVM_VER))
LLVM_SRC_URL := https://prereleases.llvm.org/$(word 1,$(LLVM_VER_SPLIT))/rc$(word 2,$(LLVM_VER_SPLIT))
Expand Down Expand Up @@ -417,6 +411,18 @@ $(eval $(call LLVM_PATCH,llvm-D75072-SCEV-add-type))
$(eval $(call LLVM_PATCH,llvm-9.0-D65174-limit-merge-stores)) # remove for 10.0
endif # LLVM_VER 9.0

ifeq ($(LLVM_VER_SHORT),10.0)
$(eval $(call LLVM_PATCH,llvm-D27629-AArch64-large_model_6.0.1))
$(eval $(call LLVM_PATCH,llvm8-D34078-vectorize-fdiv))
$(eval $(call LLVM_PATCH,llvm-6.0-NVPTX-addrspaces)) # NVPTX -- warning: this fails check-llvm-codegen-nvptx
$(eval $(call LLVM_PATCH,llvm-7.0-D44650)) # mingw32 build fix
$(eval $(call LLVM_PATCH,llvm9-D50010-VNCoercion-ni))
$(eval $(call LLVM_PATCH,llvm8-WASM-addrspaces)) # WebAssembly
$(eval $(call LLVM_PATCH,llvm-exegesis-mingw)) # mingw build
$(eval $(call LLVM_PATCH,llvm-test-plugin-mingw)) # mingw build
$(eval $(call LLVM_PATCH,llvm7-revert-D44485))
$(eval $(call LLVM_PATCH,llvm-D75072-SCEV-add-type))
endif # LLVM_VER 10.0

# Add a JL prefix to the version map. DO NOT REMOVE
ifneq ($(LLVM_VER), svn)
Expand Down
6 changes: 3 additions & 3 deletions src/aotcompile.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -848,7 +848,7 @@ addPassesToGenerateCode(LLVMTargetMachine *TM, PassManagerBase &PM) {
TargetPassConfig *PassConfig = TM->createPassConfig(PM);
PassConfig->setDisableVerify(false);
PM.add(PassConfig);
#if JL_LLVM_VERSION >= 110000
#if JL_LLVM_VERSION >= 100000
MachineModuleInfoWrapperPass *MMIWP =
new MachineModuleInfoWrapperPass(TM);
PM.add(MMIWP);
Expand All @@ -860,7 +860,7 @@ addPassesToGenerateCode(LLVMTargetMachine *TM, PassManagerBase &PM) {
return NULL;
PassConfig->addMachinePasses();
PassConfig->setInitialized();
#if JL_LLVM_VERSION >= 110000
#if JL_LLVM_VERSION >= 100000
return &MMIWP->getMMI().getContext();
#else
return &MMI->getContext();
Expand Down Expand Up @@ -905,7 +905,7 @@ jl_value_t *jl_dump_llvm_asm(void *F, const char* asm_variant, const char *debug
std::unique_ptr<MCAsmBackend> MAB(TM->getTarget().createMCAsmBackend(
STI, MRI, TM->Options.MCOptions));
std::unique_ptr<MCCodeEmitter> MCE;
#if JL_LLVM_VERSION >= 110000
#if JL_LLVM_VERSION >= 100000
auto FOut = std::make_unique<formatted_raw_ostream>(asmfile);
#else
auto FOut = llvm::make_unique<formatted_raw_ostream>(asmfile);
Expand Down

2 comments on commit ac22a6d

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Executing the daily package evaluation, I will reply here when finished:

@nanosoldier runtests(ALL, isdaily = true)

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your package evaluation job has completed - no new issues were detected. A full report can be found here. cc @maleadt

Please sign in to comment.