Skip to content

Conversation

martinboehme
Copy link
Contributor

No description provided.

@martinboehme martinboehme requested a review from a team as a code owner September 5, 2023 13:00
@martinboehme martinboehme requested a review from ymand September 5, 2023 13:01
@martinboehme martinboehme merged commit be12f26 into llvm:main Sep 6, 2023
Guzhu-AMD pushed a commit to GPUOpen-Drivers/llvm-project that referenced this pull request Sep 7, 2023
Local branch amd-gfx c3c1922 Merged main:71be020dda2c into amd-gfx:a4d6f75a4aba
Remote branch main be12f26 [clang][dataflow][NFC] Remove stale comment. (llvm#65322)
avillega pushed a commit to avillega/llvm-project that referenced this pull request Sep 11, 2023
@Endilll Endilll added clang Clang issues not falling into any other category clang:analysis labels Jul 14, 2025
@llvmbot
Copy link
Member

llvmbot commented Jul 14, 2025

@llvm/pr-subscribers-clang

@llvm/pr-subscribers-clang-analysis

Author: None (martinboehme)

Changes

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

1 Files Affected:

  • (modified) clang/unittests/Analysis/FlowSensitive/TestingSupport.h (-4)
diff --git a/clang/unittests/Analysis/FlowSensitive/TestingSupport.h b/clang/unittests/Analysis/FlowSensitive/TestingSupport.h
index 44dbf27a74586..c61e9f26beff4 100644
--- a/clang/unittests/Analysis/FlowSensitive/TestingSupport.h
+++ b/clang/unittests/Analysis/FlowSensitive/TestingSupport.h
@@ -465,10 +465,6 @@ inline Value *getFieldValue(const RecordStorageLocation *Loc,
 
 /// Returns the value of a `Field` on a `Struct.
 /// Returns null if `Struct` is null.
-///
-/// Note: This function currently does not use the `Env` parameter, but it will
-/// soon be needed to look up the `Value` when `setChild()` changes to return a
-/// `StorageLocation *`.
 inline Value *getFieldValue(const RecordValue *Struct, const ValueDecl &Field,
                             const Environment &Env) {
   if (Struct == nullptr)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang:analysis clang Clang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants