[clang] [clang][Sema] Add missing scope flags to Scope::dumpImpl (PR #94529)

via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 5 12:51:01 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Krzysztof Parzyszek (kparzysz)

<details>
<summary>Changes</summary>

There were a handlful of scope flags that were not handled in the dump function, would would then lead to an assert.

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


1 Files Affected:

- (modified) clang/lib/Sema/Scope.cpp (+4) 


``````````diff
diff --git a/clang/lib/Sema/Scope.cpp b/clang/lib/Sema/Scope.cpp
index c08073e80ff3d..5bc7e79a68186 100644
--- a/clang/lib/Sema/Scope.cpp
+++ b/clang/lib/Sema/Scope.cpp
@@ -228,7 +228,11 @@ void Scope::dumpImpl(raw_ostream &OS) const {
       {CompoundStmtScope, "CompoundStmtScope"},
       {ClassInheritanceScope, "ClassInheritanceScope"},
       {CatchScope, "CatchScope"},
+      {ConditionVarScope, "ConditionVarScope"},
+      {OpenMPOrderClauseScope, "OpenMPOrderClauseScope"},
+      {LambdaScope, "LambdaScope"},
       {OpenACCComputeConstructScope, "OpenACCComputeConstructScope"},
+      {TypeAliasScope, "TypeAliasScope"},
       {FriendScope, "FriendScope"},
   };
 

``````````

</details>


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


More information about the cfe-commits mailing list