[clang] [clang][ExtractAPI] Add ability to create multiple symbol graphs (PR #86676)

via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 26 07:58:30 PDT 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 72c729f354d71697a1402720c90b57ff521b6739 cc5542353b1fa36a1d2c2637526af3dad522f7a0 -- clang/include/clang/ExtractAPI/APIRecords.inc clang/include/clang/ExtractAPI/Serialization/APISetVisitor.h clang/test/ExtractAPI/metadata_and_module.c clang/include/clang/ExtractAPI/API.h clang/include/clang/ExtractAPI/DeclarationFragments.h clang/include/clang/ExtractAPI/ExtractAPIActionBase.h clang/include/clang/ExtractAPI/ExtractAPIVisitor.h clang/include/clang/ExtractAPI/FrontendActions.h clang/include/clang/ExtractAPI/Serialization/SymbolGraphSerializer.h clang/include/clang/Frontend/FrontendOptions.h clang/lib/Driver/ToolChains/Clang.cpp clang/lib/ExtractAPI/API.cpp clang/lib/ExtractAPI/DeclarationFragments.cpp clang/lib/ExtractAPI/ExtractAPIConsumer.cpp clang/lib/ExtractAPI/Serialization/SymbolGraphSerializer.cpp clang/lib/ExtractAPI/TypedefUnderlyingTypeResolver.cpp clang/lib/FrontendTool/ExecuteCompilerInvocation.cpp clang/test/ExtractAPI/anonymous_record_no_typedef.c clang/test/ExtractAPI/availability.c clang/test/ExtractAPI/bool.c clang/test/ExtractAPI/bool.cpp clang/test/ExtractAPI/class.cpp clang/test/ExtractAPI/class_template.cpp clang/test/ExtractAPI/class_template_param_inheritance.cpp clang/test/ExtractAPI/class_template_partial_spec.cpp clang/test/ExtractAPI/class_template_spec.cpp clang/test/ExtractAPI/concept.cpp clang/test/ExtractAPI/constructor_destructor.cpp clang/test/ExtractAPI/conversions.cpp clang/test/ExtractAPI/emit-symbol-graph/multi_file.c clang/test/ExtractAPI/emit-symbol-graph/single_file.c clang/test/ExtractAPI/enum.c clang/test/ExtractAPI/field_template.cpp clang/test/ExtractAPI/function_noexcepts.cpp clang/test/ExtractAPI/global_func_template.cpp clang/test/ExtractAPI/global_func_template_spec.cpp clang/test/ExtractAPI/global_record.c clang/test/ExtractAPI/global_record_multifile.c clang/test/ExtractAPI/global_var_template.cpp clang/test/ExtractAPI/global_var_template_partial_spec.cpp clang/test/ExtractAPI/global_var_template_spec.cpp clang/test/ExtractAPI/known_files_only.c clang/test/ExtractAPI/language.c clang/test/ExtractAPI/macro_undefined.c clang/test/ExtractAPI/macros.c clang/test/ExtractAPI/method_template.cpp clang/test/ExtractAPI/method_template_spec.cpp clang/test/ExtractAPI/methods.cpp clang/test/ExtractAPI/multiple_inheritance.cpp clang/test/ExtractAPI/namespace.cpp clang/test/ExtractAPI/nested_namespaces.cpp clang/test/ExtractAPI/operator_overload.cpp clang/test/ExtractAPI/simple_inheritance.cpp clang/test/ExtractAPI/struct.c clang/test/ExtractAPI/typedef.c clang/test/ExtractAPI/typedef_anonymous_record.c clang/test/ExtractAPI/typedef_chain.c clang/test/ExtractAPI/typedef_struct_enum.c clang/test/ExtractAPI/underscored.c clang/test/ExtractAPI/union.c clang/tools/libclang/CXExtractAPI.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/clang/include/clang/ExtractAPI/API.h b/clang/include/clang/ExtractAPI/API.h
index 90ee1e8dbb..9a34334686 100644
--- a/clang/include/clang/ExtractAPI/API.h
+++ b/clang/include/clang/ExtractAPI/API.h
@@ -339,7 +339,8 @@ public:
     // This doesn't strictly meet the iterator requirements, but it's the
     // behavior we want here.
     value_type operator->() const { return Current; }
-    record_iterator &operator++() { Current = Current->getNextInContex();
+    record_iterator &operator++() {
+      Current = Current->getNextInContex();
       return *this;
     }
     record_iterator operator++(int) {
@@ -1421,8 +1422,8 @@ APISet::createRecord(StringRef USR, StringRef Name,
 
   // Create the record if it does not already exist
   if (Result.second) {
-    Record = new (Allocator) RecordTy(USRString, copyString(Name),
-                                std::forward<CtorArgsContTy>(CtorArgs)...);
+    Record = new (Allocator) RecordTy(
+        USRString, copyString(Name), std::forward<CtorArgsContTy>(CtorArgs)...);
     // Store the record in the record lookup map
     Result.first->second = APIRecordStoredPtr(Record);
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/86676


More information about the cfe-commits mailing list