[PATCH] D52880: [clang-tidy] fix PR39167, bugprone-exception-escape hangs-up

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 4 08:47:42 PDT 2018


JonasToth added a comment.

In https://reviews.llvm.org/D52880#1255250, @alexfh wrote:

> In https://reviews.llvm.org/D52880#1255249, @alexfh wrote:
>
> > Have you figured out why exactly does the check hang? Disabling it for -fno-exceptions may just hide a logical problem in the check.
>
>
> (disabling it for these cases seems to be the right thing to do anyways, but it might be nice to use the opportunity to investigate the possible issue deeper)


I did not investigate further, as the fix was so straight forward. I did verify it works though and that it actually fixes the issue :)

Deeper analysis could be done by @baloghadamsoftware	as the original author?


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D52880





More information about the cfe-commits mailing list