Skip to content

[clang][ExprConst][NFC] Remove outdated comments - #213587

Open
tbaederr wants to merge 2 commits into
llvm:mainfrom
tbaederr:docs3
Open

[clang][ExprConst][NFC] Remove outdated comments#213587
tbaederr wants to merge 2 commits into
llvm:mainfrom
tbaederr:docs3

Conversation

@tbaederr

@tbaederr tbaederr commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Those parameters don't exist anymore.

Those parameters don't exist anymore.
@llvmorg-github-actions llvmorg-github-actions Bot added clang Clang issues not falling into any other category clang:frontend Language frontend issues, e.g. anything involving "Sema" labels Aug 3, 2026
@llvmorg-github-actions

Copy link
Copy Markdown

@llvm/pr-subscribers-clang

Author: Timm Baeder (tbaederr)

Changes

Those parameters don't exist anymore.


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

1 Files Affected:

  • (modified) clang/lib/AST/ExprConstant.cpp (+1-5)
diff --git a/clang/lib/AST/ExprConstant.cpp b/clang/lib/AST/ExprConstant.cpp
index 9d69de2a7c6fd..ebed083146a42 100644
--- a/clang/lib/AST/ExprConstant.cpp
+++ b/clang/lib/AST/ExprConstant.cpp
@@ -16778,11 +16778,7 @@ static bool determineEndOffset(EvalInfo &Info, SourceLocation ExprLoc,
   return true;
 }
 
-/// Tries to evaluate the __builtin_object_size for @p E. If successful,
-/// returns true and stores the result in @p Size.
-///
-/// If @p WasError is non-null, this will report whether the failure to evaluate
-/// is to be treated as an Error in IntExprEvaluator.
+/// Tries to evaluate the __builtin_object_size for @p E.
 ///
 /// If @p IsDynamic is true (i.e. we're evaluating
 /// __builtin_dynamic_object_size) and the operand designates a flexible array

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

🐧 Linux x64 Test Results

  • 121160 tests passed
  • 4977 tests skipped

✅ The build succeeded and all tests passed.

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.

1 participant