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.

Sign up for GitHub

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

Jump to bottom

[release/6.0] Fix build with Clang 13 #63314

Merged
merged 2 commits into from
Jan 4, 2022
Merged

[release/6.0] Fix build with Clang 13 #63314

merged 2 commits into from
Jan 4, 2022

Commits on Jan 3, 2022

  1. Fix clang 13 induced runtime issues (dotnet#62170) 

    The clang 13 optimizer started to assume that "this" pointer is always
properly aligned. That lead to elimination of some code that was actually
needed.
It also takes pointer aliasing rules more strictly in one place in jit.
That caused the optimizer to falsely assume that a callee with an argument
passed by reference is not modifying that argument and used a stale
copy of the original value at the caller site.

This change fixes both of the issues. With this fix, runtime compiled
using clang 13 seems to be fully functional.
    @janvorli
    janvorli committed Jan 3, 2022
    Configuration menu
    Copy the full SHA
    956eec3 View commit details
    Browse the repository at this point in the history

  2. Fix build with clang 13 (dotnet#60328)

    @janvorli
    janvorli committed Jan 3, 2022
    Configuration menu
    Copy the full SHA
    62cf0e9 View commit details
    Browse the repository at this point in the history