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

[NativeAOT-LLVM] Runtime tests generate a significant number of warnings #2165

Closed
SingleAccretion opened this issue Jan 11, 2023 · 1 comment
Labels
area-NativeAOT-LLVM LLVM generation for Native AOT compilation (including Web Assembly)

Comments

@SingleAccretion
Copy link

SingleAccretion commented Jan 11, 2023

Let's get at least HelloWasm clean, eleven warnings is too high a number to ignore.

Some of these are upstream problems we've picked up, but most should be actionable directly.

Current list of warnings:

1) Four MSB4011 - dotnet/runtime#70658

2) Two IL2070.

  • One DAM will solve the problem.

3) clang++ : warning : argument unused during compilation: '-I C:\Users\Accretion\source\dotnet\emsdk\upstream\emscripten\cache\sysroot\include\SDL'.

4) EXEC : warning : undefined symbol: S_P_CoreLib_Internal_Runtime_CompilerHelpers_ThrowHelpers__ThrowBodyRemoved - appeared after the recent merge.

  • We are using the wrong helper for tentative instance method stubs.

5) EXEC : warning : undefined symbol: dotnet_browser_entropy - good old dotnet_browser_entropy.

  • Presumably there because System.Native references it.

6) emcc : warning : warnings in JS library compilation [-Wjs-compiler].

  • Fallout from the undefined symbols warnings.
@SingleAccretion
Copy link
Author

FYI @yowl

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-NativeAOT-LLVM LLVM generation for Native AOT compilation (including Web Assembly)
Projects
None yet
Development

No branches or pull requests

2 participants