[clang] [clang-tools-extra] [Clang][AST] Move NamespaceDecl bits to DeclContext (PR #98567)

2024-07-12 Thread Krystian Stasiowski via cfe-commits
https://github.com/sdkrystian updated https://github.com/llvm/llvm-project/pull/98567 >From 63a9c69cc5c4d6f8048918addfb01de6dc212524 Mon Sep 17 00:00:00 2001 From: Krystian Stasiowski Date: Thu, 20 Jun 2024 11:44:53 -0400 Subject: [PATCH 1/2] [Clang][AST] Move NamespaceDecl bits to DeclContext

[clang] [clang-tools-extra] [Clang][AST] Move NamespaceDecl bits to DeclContext (PR #98567)

2024-07-12 Thread Krystian Stasiowski via cfe-commits
https://github.com/sdkrystian updated https://github.com/llvm/llvm-project/pull/98567 >From 63a9c69cc5c4d6f8048918addfb01de6dc212524 Mon Sep 17 00:00:00 2001 From: Krystian Stasiowski Date: Thu, 20 Jun 2024 11:44:53 -0400 Subject: [PATCH 1/3] [Clang][AST] Move NamespaceDecl bits to DeclContext

[clang] [clang-tools-extra] [Clang][AST] Move NamespaceDecl bits to DeclContext (PR #98567)

2024-07-12 Thread Krystian Stasiowski via cfe-commits
sdkrystian wrote: @Endilll Should I go ahead with this change anyways? https://github.com/llvm/llvm-project/pull/98567 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-tools-extra] [Clang][AST] Move NamespaceDecl bits to DeclContext (PR #98567)

2024-07-12 Thread Vlad Serebrennikov via cfe-commits
Endilll wrote: > @Endilll Should I go ahead with this change anyways? I would prefer if you can reduce the scope of the PR by avoiding the introduction of `NamespaceDeclBits` (it looks like you do more than just that). But if you feel strongly about it, I don't want to hold you off because of

[clang] [clang-tools-extra] [Clang][AST] Move NamespaceDecl bits to DeclContext (PR #98567)

2024-07-12 Thread Matheus Izvekov via cfe-commits
https://github.com/mizvekov approved this pull request. LGTM, as a clean up, and I see no notable performance implications. Is there any follow up work for this? https://github.com/llvm/llvm-project/pull/98567 ___ cfe-commits mailing list cfe-commits@

[clang] [clang-tools-extra] [Clang][AST] Move NamespaceDecl bits to DeclContext (PR #98567)

2024-07-15 Thread Krystian Stasiowski via cfe-commits
sdkrystian wrote: > Is there any follow up work for this? @mizvekov Yes, but it's not directly related. I'm going to submit a patch in the future which stores the `SourceLocation` of the `inline` keyword for inline namespace, and perhaps another patch that extends `NestedNameSpecifier` such t

[clang] [clang-tools-extra] [Clang][AST] Move NamespaceDecl bits to DeclContext (PR #98567)

2024-07-15 Thread Krystian Stasiowski via cfe-commits
https://github.com/sdkrystian closed https://github.com/llvm/llvm-project/pull/98567 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-tools-extra] [Clang][AST] Move NamespaceDecl bits to DeclContext (PR #98567)

2024-07-15 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `sanitizer-x86_64-linux` running on `sanitizer-buildbot2` while building `clang-tools-extra,clang` at step 2 "annotate". Full details are available at: https://lab.llvm.org/buildbot/#/builders/66/builds/1531 Here is the relev