Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reland " Upgrade to LLVM 14.0.2" #45679

Merged
merged 6 commits into from
Jun 17, 2022
Merged

Reland " Upgrade to LLVM 14.0.2" #45679

merged 6 commits into from
Jun 17, 2022

Conversation

vchuravy
Copy link
Sponsor Member

  • Upgrade to LLVM 14.0.2
  • Replace debuginfo GVs for Win64 with assembly
  • Help the static analyzer
  • fix llvmpasses for 14
  • Adjust simplifycfg options

#45195 contained many assertions in the package ecosystem
that were accidentially overlooked by me.

So let's calmly reland this once we figure those out.

vchuravy and others added 5 commits June 14, 2022 11:07
https://reviews.llvm.org/D100944 introduces will split sections if the
metadata does not match. The GlobalVariables are `RW_` and the `.text`
section is `R_X`. Currently there is no facility to actually mark the
GV as `R_X` and we only need to write them during code-emission before
we flip the permissions.
@maleadt
Copy link
Member

maleadt commented Jun 14, 2022

PkgEval run comparing assertions on the previous version of this PR vs. the commit before: 6c97a48#commitcomment-76075908.

@maleadt
Copy link
Member

maleadt commented Jun 15, 2022

Your package evaluation job has completed - possible new issues were detected. A full report can be found here.

So no new assertion failures.

@vchuravy vchuravy merged commit 7303b45 into master Jun 17, 2022
@vchuravy vchuravy deleted the vc/llvm_14.0.2 branch June 17, 2022 17:45
@@ -101,8 +101,14 @@ void jl_init_stack_limits(int ismaster, void **stack_lo, void **stack_hi)
struct rlimit rl;
getrlimit(RLIMIT_STACK, &rl);
size_t stacksize = rl.rlim_cur;
// We intentionally leak a stack address here core.StackAddressEscape
Copy link
Contributor

Choose a reason for hiding this comment

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

Related: #45617 🙂

inkydragon added a commit to inkydragon/julia that referenced this pull request Jun 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants