[PATCH] D33333: Emit warning when throw exception in destruct or dealloc functions which has a (possible implicit) noexcept specifier

[Jennifer Yu via Phabricator via cfe-commits]

jyu2 marked 8 inline comments as done.
jyu2 added inline comments.


================
Comment at: lib/Sema/AnalysisBasedWarnings.cpp:334
+      continue;
+    else
+      HasThrowOutFunc = true;
----------------
aaron.ballman wrote:
> You can drop the `else` here and just set `HasThrowOutFunc` to true.
Can not do that, don't if block has throw expression yet. 


https://reviews.llvm.org/D33333