[PATCH] D85351: [Analyzer] Fix for `ExprEngine::computeObjectUnderConstruction()` for base and delegating consturctor initializers

Balogh, Ádám via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 31 01:27:25 PDT 2020


baloghadamsoftware added a comment.
Herald added a subscriber: danielkiss.

In D85351#2215547 <https://reviews.llvm.org/D85351#2215547>, @Szelethus wrote:

> Shouldn't we create a new test care for this, instead of expanding an existing one? Btw, this looks great, but I lack the confidence to accept.

Why should we? This is just a fix for cases not covered, but it is the same functionality (retrieving the return value under construction). I added the missed cases to the test of this exact functionality.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85351/new/

https://reviews.llvm.org/D85351



More information about the cfe-commits mailing list