llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-analysis Author: Florian Mayer (fmayer) <details> <summary>Changes</summary> We have an internal analysis that uses them, and the HTML dump would fail on the assertion. --- Full diff: https://github.com/llvm/llvm-project/pull/112457.diff 1 Files Affected: - (modified) clang/lib/Analysis/FlowSensitive/HTMLLogger.cpp (+1-2) ``````````diff diff --git a/clang/lib/Analysis/FlowSensitive/HTMLLogger.cpp b/clang/lib/Analysis/FlowSensitive/HTMLLogger.cpp index a36cb41a63dfb1..557df218837941 100644 --- a/clang/lib/Analysis/FlowSensitive/HTMLLogger.cpp +++ b/clang/lib/Analysis/FlowSensitive/HTMLLogger.cpp @@ -133,8 +133,7 @@ class ModelDumper { for (const auto &Child : RLoc->children()) JOS.attributeObject("f:" + Child.first->getNameAsString(), [&] { if (Child.second) - if (Value *Val = Env.getValue(*Child.second)) - dump(*Val); + dump(*Child.second); }); for (const auto &SyntheticField : RLoc->synthetic_fields()) `````````` </details> https://github.com/llvm/llvm-project/pull/112457 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
