Add LLVM compatibility to CONTRIBUTING.md

[manishwalia15]

No description provided.

[rustbot]

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Mark-Simulacrum (or someone else) some time within the next two weeks.

Please see the contribution instructions for more information. Namely, in order to ensure the minimum review times lag, PR authors and assigned reviewers should ensure that the review label (S-waiting-on-review and S-waiting-on-author) stays updated, invoking these commands when appropriate:

• @rustbot author: the review is finished, PR author should check the comments and take action accordingly
• @rustbot review: the author is ready for a review, this PR will be queued again in the reviewer's queue

[Mark-Simulacrum]

r? llvm

My sense is that this is probably not going to be kept up to date, which makes it not very useful. I'm also not sure we have any real policy on this -- my sense is that the versions are really just "whatever is tested in our CI"?

[rustbot]

Failed to set assignee to llvm: cannot assign: response: {"message":"Not Found","documentation_url":"https://docs.github.com/rest/issues/assignees#add-assignees-to-an-issue","status":"404"}

Note: Only org members with at least the repository "read" role, users with write permissions, or people who have commented on the PR may be assigned.

[workingjubilee]

@manishwalia15 Please do not open a new PR for every revision.

[slanterns]

#131619 (comment)

[workingjubilee]

This is still not the right version, we don't support LLVM 17 anymore. Where are you getting this information?

[manishwalia15]

[workingjubilee]

Link?

[workingjubilee]

Where is this?

[manishwalia15]

https://www.tublian.com/project/724712/131588

[workingjubilee]

#131588 ?

[manishwalia15]

mybe

[manishwalia15]

Kindly tell where is the mistake

[workingjubilee]

See: #130487

The minimum was raised to LLVM 18.

[manishwalia15]

shall i update to 18..rest is fine ?
pls tell @workingjubilee

[slanterns]

Maybe we should figure out if it's truly helpful to add this to CONTRIBUTING.md first.

[manishwalia15]

Please do tell

[nikic]

https://github.com/rust-lang/rust/blob/master/INSTALL.md#dependencies would be a more appropriate place.

[manishwalia15]

@nikic #131627
i hv done kindly see

[slanterns]

You can update your PR instead of opening a new one each time.

[manishwalia15]

sorry @slanterns i didn knew
next time i wont
please allow for this time