[PATCH] D55697: [analyzer] Assume that we always have a SubEngine available
Gábor Horváth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 14 02:02:05 PST 2018
xazax.hun created this revision.
xazax.hun added reviewers: NoQ, george.karpenkov.
xazax.hun added a project: clang.
Herald added subscribers: gamesh411, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware, whisperity.
I was stumbled upon some checks whether a SubEngine is available. I was wondering why do we have those checks so I removed them and wondered if the tests would break. They did not break on my machine, so I submitted this patch.
Do yo think these should be removed? If no, how could we test those code paths because apparently they are untested at the moment.
Repository:
rC Clang
https://reviews.llvm.org/D55697
Files:
lib/StaticAnalyzer/Core/ProgramState.cpp
lib/StaticAnalyzer/Core/RegionStore.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55697.178204.patch
Type: text/x-patch
Size: 4627 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181214/90f29852/attachment.bin>
More information about the cfe-commits
mailing list