]> granicus.if.org Git - clang/commit
Enhance the CFG construction to detect no-return destructors for
authorChandler Carruth <chandlerc@gmail.com>
Tue, 13 Sep 2011 06:09:01 +0000 (06:09 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Tue, 13 Sep 2011 06:09:01 +0000 (06:09 +0000)
commitc8cfc74bdcc999828bc232294d937fb191940d5b
tree7a8b66ff927fe4f6e10e4d1610dc1537fa6ac4cd
parent6c11f0b3106263600af2ea0438ebb372821514aa
Enhance the CFG construction to detect no-return destructors for
temporary objects and local variables. When detected, these split the
block, marking the new one as having only the exit block as a successor.
This prevents a large number of false positives in warnings sensitive to
no-return constructs such as -Wreturn-type, and fixes the remainder of
PR10063 along with several variations of this bug that had not been
reported. The test cases are extended across the board to cover these
patterns.

This also checks in a stress test for these types of CFGs. The stress
test declares some 32k variables, a mixture of no-return and normal
destructors. Previously, this resulted in roughly 2500 CFG blocks, but
didn't model any of the no-return destructors. With this patch, it
results in over 33k blocks, many of them now unreachable.

The nice thing about how the analyzer is set up? This causes *no*
regression in performance of building the CFG. It actually in some cases
makes it faster, as best I can benchmark. The analysis for -Wreturn-type
(and any other that cares about no-return code paths) is technically
slower now as it has to look at many more candidate blocks, but it
computes the correct answer. I have more test cases to follow, I think
they all work now. Also I have further work that should dramatically
simplify analyses in the presence of no-return.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@139586 91177308-0d34-0410-b5e6-96231b3b80d8
INPUTS/cfg-nested-var-scopes.cpp [new file with mode: 0644]
include/clang/Analysis/CFG.h
lib/Analysis/CFG.cpp
test/SemaCXX/return-noreturn.cpp