Skip to content

Conversation

amykhuang
Copy link
Collaborator

@amykhuang amykhuang commented Sep 22, 2023

Previous change in #65675 broke the docs generation.

@llvmbot llvmbot added clang Clang issues not falling into any other category clang:frontend Language frontend issues, e.g. anything involving "Sema" labels Sep 22, 2023
@amykhuang amykhuang merged commit 44f2db7 into llvm:main Sep 22, 2023
@llvmbot
Copy link
Member

llvmbot commented Sep 22, 2023

@llvm/pr-subscribers-clang

Changes

Previous change in #65675 broke the docs generation.


Full diff: https://github.com/llvm/llvm-project/pull/67195.diff

1 Files Affected:

  • (modified) clang/include/clang/Basic/Attr.td (-1)
diff --git a/clang/include/clang/Basic/Attr.td b/clang/include/clang/Basic/Attr.td
index bb8d204f33dc531..35eb6ff63132629 100644
--- a/clang/include/clang/Basic/Attr.td
+++ b/clang/include/clang/Basic/Attr.td
@@ -1803,7 +1803,6 @@ def NoUniqueAddress : InheritableAttr {
   // No spellings because instances of this attribute are created by
   // MSNoUniqueAddress and ItaniumNoUniqueAddress
   let Spellings = [];
-  let Documentation = [NoUniqueAddressDocs];
 }
 
 def MSNoUniqueAddress : InheritableAttr, TargetSpecificAttr<TargetMicrosoftCXXABI> {

MaskRay added a commit that referenced this pull request Sep 22, 2023
This reverts commit 44f2db7.

It caused
```
error: Record `NoUniqueAddress', field `Documentation' exists but does not have a list value
```
@amykhuang amykhuang deleted the no-unique-addr-docs-fix branch April 2, 2025 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang:frontend Language frontend issues, e.g. anything involving "Sema" clang Clang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants