diff options
| author | Etienne Bergeron <etienneb@google.com> | 2016-04-07 14:52:52 +0000 |
|---|---|---|
| committer | Etienne Bergeron <etienneb@google.com> | 2016-04-07 14:52:52 +0000 |
| commit | 53f7c0ea8e02e2a49f13e713ec6bdbfe067e8383 (patch) | |
| tree | d8e08212b3d22601042d12c70636df0c955f49f0 /clang-tools-extra/clang-tidy/misc/ForwardDeclarationNamespaceCheck.cpp | |
| parent | d37630e461cb6e1db4fdf221f6ff978c8d7b0bc1 (diff) | |
| download | bcm5719-llvm-53f7c0ea8e02e2a49f13e713ec6bdbfe067e8383.tar.gz bcm5719-llvm-53f7c0ea8e02e2a49f13e713ec6bdbfe067e8383.zip | |
[clang-tidy] Fix infinite loop in MisplacedWideningCastCheck.
Summary:
In Release mode, the check was infinite looping over chromium code base.
It seems there is something strange with the creation of the Maps.
I believe the compiler is making some assumption with the implicit conversion from enum <-> int.
By moving the map to a standard switch/cases, we no longer allocate memory and we can keep the same behavior. For a small amount of elements, this is fine.
Reviewers: alexfh
Subscribers: cfe-commits
Differential Revision: http://reviews.llvm.org/D18833
llvm-svn: 265679
Diffstat (limited to 'clang-tools-extra/clang-tidy/misc/ForwardDeclarationNamespaceCheck.cpp')
0 files changed, 0 insertions, 0 deletions
