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

Missing Clang 16 support #525

Closed
andreasfertig opened this issue Apr 14, 2023 · 1 comment
Closed

Missing Clang 16 support #525

andreasfertig opened this issue Apr 14, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@andreasfertig
Copy link
Owner

Clang 16 support for C++ Insights is missing at the moment due to the following reasons:

  1. LLVM seems to no longer provide pre-built binaries for macOS Intel
  2. Some API changes in LLVM/Clang (as usual) need adjustments to the C++ Insights code.
  3. Some changes require additional work in C++ Insights and it's unclear whether the change is intended. For example, compiler-explorer.com/z/3WhM9odP9:
struct Test
{
    using size_type = int;

    size_type size() { return 5; }
};

auto X()
{
    Test s{};

    return s.size();
}

int main() {
    X();
}

The return type of X is Test::size_type in Clang 15 and size_type in Clang 16.

I got 1 and 2 covered by running my own build, and the API changes were also fixable, but I haven't had time to look into 3.

Andreas

@andreasfertig
Copy link
Owner Author

Fixed by #569.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant