Skip to content

Commit

Permalink
Set LLVM issue tracker URL to Julia's issue tracker (#53128)
Browse files Browse the repository at this point in the history 
This patch sets the default issue tracker URL for LLVM to Julia's issue
tracker. LLVM issues coming from Julia should probably go through
Julia's issue tracker first and then can be reported upstream if there
is an issue with something in upstream LLVM. Having this set to the
default is moderately confusing for users as it refers to the LLVM issue
tracker, which probably isn't wanted.
  • Loading branch information
@boomanaiden154
boomanaiden154 authored Feb 13, 2024
1 parent 69f16a2 commit 38065a4
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions deps/llvm.mk
View file
Original file line number Diff line number Diff line change
Expand Up @@ -210,6 +210,9 @@ LLVM_CMAKE += -DCMAKE_EXE_LINKER_FLAGS="$(LLVM_LDFLAGS)" \
LLVM_CMAKE += -DLLVM_VERSION_SUFFIX:STRING="jl"
LLVM_CMAKE += -DLLVM_SHLIB_SYMBOL_VERSION:STRING="JL_LLVM_$(LLVM_VER_SHORT)"

# Change the default bug report URL to Julia's issue tracker
LLVM_CMAKE += -DBUG_REPORT_URL="https://github.com/julialang/julia"

# Apply version-specific LLVM patches sequentially
LLVM_PATCH_PREV :=
define LLVM_PATCH
Expand Down

6 comments on commit 38065a4

@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(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 job failed. Consult the server logs for more details (cc @maleadt).

@maleadt
Copy link
Member

Choose a reason for hiding this comment

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

@nanosoldier runtests(isdaily = true)

@vtjnash
Copy link
Member

Choose a reason for hiding this comment

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

@nanosoldier runbenchmarks(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 job failed.

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

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

The package evaluation job you requested has completed - possible new issues were detected.
The full report is available.

Please sign in to comment.