-
Notifications
You must be signed in to change notification settings - Fork 65
8352481: Enforce the use of lld with clang #163
Conversation
👋 Welcome back syan! A progress list of the required criteria for merging this PR into |
❗ This change is not yet ready to be integrated. |
This backport pull request has now been updated with issue from the original commit. |
/issue add JDK-8353005 |
@sendaoYan |
/label add clean |
@sendaoYan |
@sendaoYan This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply add a new comment to the pull request. Feel free to ask for assistance if you need help with progressing this pull request towards integration! |
Hi, can anyone take look this backport PR. The two backports all are cleans. This backport is intend to fix a little bug in makefile, risk is low. |
24.0.2 is closed, and it's unlikely there will be a 24.0.3, so I'd withdraw this PR. |
@sendaoYan This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply issue a |
Okey, Thanks. I think we can close this PR. |
Hi all,
This pull request contains clean backports of commit 0935ba9a and commit 3319dcfb from the openjdk/jdk repository.
The commit being backported was authored by SendaoYan on 26 Mar 2025 and was reviewed by Magnus Ihse Bursie and Erik Joelsson.
Thanks!
Progress
Issues
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk24u.git pull/163/head:pull/163
$ git checkout pull/163
Update a local copy of the PR:
$ git checkout pull/163
$ git pull https://git.openjdk.org/jdk24u.git pull/163/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 163
View PR using the GUI difftool:
$ git pr show -t 163
Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk24u/pull/163.diff
Using Webrev
Link to Webrev Comment