Allow _ in use! bindings values

[edgarfgp]

Description

Currently if we use any pattern that is not Named or LongIdent it errors miserably pointing at the use! range.

Also it does not allow to wrap patterns In parens(I don't see the point in restricting use e.g. (x), (__), (_).)

So in this PR:

• Allows Named or LongIdent and Wild(only in preview) to be wrapped in parens
• Updates the error range to use the pat range instead of the use! range

BEFORE

counterDisposable {use! res =new Disposable()// No Erroruse! __ =new Disposable()// No Erroruse!(res1)=new Disposable()// Error^^^^use!(___)=new Disposable()// Error^^^^use! _ =new Disposable()// Error^^^^use! [| 1 |] =new Disposable()// Error^^^^return()}

AFTER

counterDisposable {use! res =new Disposable()// No Erroruse! __ =new Disposable()// No Erroruse!(res1)=new Disposable()// No Erroruse!(___)=new Disposable()// No Erroruse! _ =new Disposable()// No Erroruse! [| 1 |] =new Disposable()// Error^^^^^^^return()}

Fixes#16343

Checklist

• Test cases added
• Release notes entry updated

[github-actions]

❗ Release notes required

---

✅ Found changes and release notes in following paths:

Change path	Release notes path	Description
src/Compiler	docs/release-notes/.FSharp.Compiler.Service/9.0.300.md
LanguageFeatures.fsi	docs/release-notes/.Language/preview.md

[edgarfgp]

To avoid conflicts #18482 should be merged first.

[edgarfgp]

Fixed the merged conflict and this is green again :)