Remove libunwind disabled warnings.

[AaronRobinsonMSFT]

Contributes to #66154

/cc @GrabYourPitchforks @jkotas @am11

[danmoseley]

if it compiles..

[ghost]

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Contributes to #66154

/cc @GrabYourPitchforks @jkotas @am11

Author:	AaronRobinsonMSFT
Assignees:	AaronRobinsonMSFT
Labels:	area-Infrastructure
Milestone:	7.0.0

[am11]

@AaronRobinsonMSFT, sorry I have just noticed this 🤦:
We should revert this file as it is not the one we are using for compilation anymore.

The idea is, src/native/external/{dependency-name}/ directory will be untouched third-party tree. "Our" config files (cmake etc.) will live outside of vendor's directory. Any patch that we apply in vendors tree will have some note in src/native/external/{dependency-name}-version.txt file.

In this case, we are now using src/native/external/libunwind.cmake and src/native/external/libunwind_extras/CMakeLists.txt.

[AaronRobinsonMSFT]

@am11 Are you saying this is the official libunwind version?

[am11]

Yup src/native/external/libunwind/CMakeLists.txt is the copy of upstream: https://github.com/libunwind/libunwind/blob/v1.6-stable/CMakeLists.txt which we do not use anymore (since #64043).

[AaronRobinsonMSFT]

Okay. I've been updating this in the official libunwind repo too - see libunwind/libunwind#333. I assume this means they will be back in sync?

[am11]

Ah great, libunwind-version.txt has libunwind/libunwind#333 already listed. All good. 👍