Page MenuHomePhabricator

alexfh (Alexander Kornienko)
User

Projects

User does not belong to any projects.

User Details

User Since
Jul 10 2012, 10:35 AM (353 w, 3 d)

Recent Activity

Thu, Apr 18

alexfh created D60868: [clang-format] Fix an assertion failure.
Thu, Apr 18, 7:36 AM · Restricted Project
alexfh accepted D60857: [clang-tidy] Address post-commit comments.

There's one more nit. Otherwise good to go.

Thu, Apr 18, 6:31 AM · Restricted Project
alexfh added inline comments to D60857: [clang-tidy] Address post-commit comments.
Thu, Apr 18, 5:51 AM · Restricted Project

Wed, Apr 17

alexfh committed rG68be3229879e: [clang-format] Remove unused Environment constructor. (authored by alexfh).
[clang-format] Remove unused Environment constructor.
Wed, Apr 17, 5:40 PM
alexfh committed rC358630: [clang-format] Remove unused Environment constructor..
[clang-format] Remove unused Environment constructor.
Wed, Apr 17, 5:40 PM
alexfh committed rL358630: [clang-format] Remove unused Environment constructor..
[clang-format] Remove unused Environment constructor.
Wed, Apr 17, 5:39 PM
alexfh committed rG4c177038e096: [clang-tidy] Don't issue cppcoreguidelines-macro-usage on builtin macros (authored by alexfh).
[clang-tidy] Don't issue cppcoreguidelines-macro-usage on builtin macros
Wed, Apr 17, 3:36 PM
alexfh committed rL358621: [clang-tidy] Don't issue cppcoreguidelines-macro-usage on builtin macros.
[clang-tidy] Don't issue cppcoreguidelines-macro-usage on builtin macros
Wed, Apr 17, 3:36 PM
alexfh committed rCTE358621: [clang-tidy] Don't issue cppcoreguidelines-macro-usage on builtin macros.
[clang-tidy] Don't issue cppcoreguidelines-macro-usage on builtin macros
Wed, Apr 17, 3:36 PM
alexfh accepted D60808: [analyzer] pr41335: NoStoreFuncVisitor: Fix crash when no-store event is in a body-farmed function..

LG with a couple of nits.

Wed, Apr 17, 2:58 PM · Restricted Project
alexfh added a comment to D60670: [analyzer] [NFC] PathDiagnostic: Create PathDiagnosticPopUpPiece.
In D60670#1469494, @NoQ wrote:

Let's decide whether we want to display the new notes in the text output. @alexfh: Given that clang-tidy is the primary consumer of the analyzer's text output mode, would you rather have or rather not have these new note:s in the text output? (scroll to the bottom of D53076 to see how they look). In my opinion, i'd rather have them in text output (because text output is hard to navigate, and having the necessary information exactly where you need it would probably be helpful), but rather not have them in tools that parse clang-tidy notes and stuff them into a source code view in some GUI (such tools do exist, right?), so im confused. I'd also prefer to have them be off in text output in tests, because such tests usually test checker-specific notes and having more checker-inspecific notes in the tests is more annoying than useful. But that's pretty minor; i'll be happy to write them down every time i write a test if they are believed to be a good thing in text output in general.

Wed, Apr 17, 2:52 PM · Restricted Project
alexfh added inline comments to D60629: [clang-tidy] Change the namespace for llvm checkers from 'llvm' to 'llvm_check'.
Wed, Apr 17, 12:41 PM · Restricted Project, Restricted Project
alexfh accepted D59812: [analyzer] PR41185: Fix regression where __builtin_* functions weren't recognized.

Awesome, thanks!
LG

Wed, Apr 17, 12:21 PM · Restricted Project, Restricted Project
alexfh added a comment to rL358576: [clang-tidy] Add fix descriptions to clang-tidy checks..

A few post-commit comments.

Wed, Apr 17, 10:45 AM
alexfh added a comment to D59812: [analyzer] PR41185: Fix regression where __builtin_* functions weren't recognized.

Artem, I'd appreciate, if you found time to finish this fix. The bug is causing most failures in our setup. Thanks!

Wed, Apr 17, 10:03 AM · Restricted Project, Restricted Project
alexfh committed rGe31fe508e890: [clang-tidy] Fix invalid location in readability-misleading-indentation… (authored by alexfh).
[clang-tidy] Fix invalid location in readability-misleading-indentation…
Wed, Apr 17, 9:24 AM
alexfh committed rCTE358589: [clang-tidy] Fix invalid location in readability-misleading-indentation….
[clang-tidy] Fix invalid location in readability-misleading-indentation…
Wed, Apr 17, 9:18 AM
alexfh committed rL358589: [clang-tidy] Fix invalid location in readability-misleading-indentation….
[clang-tidy] Fix invalid location in readability-misleading-indentation…
Wed, Apr 17, 9:18 AM

Tue, Apr 16

alexfh added inline comments to D60629: [clang-tidy] Change the namespace for llvm checkers from 'llvm' to 'llvm_check'.
Tue, Apr 16, 1:35 PM · Restricted Project, Restricted Project
alexfh requested changes to D60776: [clang-tidy] Add test support for the fix description..
Tue, Apr 16, 1:07 PM · Restricted Project

Mon, Apr 15

alexfh accepted D59932: [clang-tidy] Add fix descriptions to clang-tidy checks..

Thanks! The change looks good now.

Mon, Apr 15, 7:51 AM · Restricted Project, Restricted Project

Wed, Apr 10

alexfh committed rG8eae988b89a1: Fix a typo (authored by alexfh).
Fix a typo
Wed, Apr 10, 8:42 AM
alexfh committed rL358092: Fix a typo.
Fix a typo
Wed, Apr 10, 8:41 AM
alexfh added a comment to D60507: [clang-tidy] new check: bugprone-unhandled-self-assignment.

Thanks for the useful check! I have a few comments, see inline.

Wed, Apr 10, 6:15 AM · Restricted Project, Restricted Project

Mon, Apr 8

alexfh committed rG10d6008f8523: Remove a bogus sed option in test. (authored by alexfh).
Remove a bogus sed option in test.
Mon, Apr 8, 9:37 AM
alexfh committed rC357922: Remove a bogus sed option in test..
Remove a bogus sed option in test.
Mon, Apr 8, 9:37 AM
alexfh committed rL357922: Remove a bogus sed option in test..
Remove a bogus sed option in test.
Mon, Apr 8, 9:37 AM
alexfh committed rGfb7338b6a579: Remove a useless assertion in clang-check. (authored by alexfh).
Remove a useless assertion in clang-check.
Mon, Apr 8, 9:29 AM
alexfh committed rL357921: Remove a useless assertion in clang-check..
Remove a useless assertion in clang-check.
Mon, Apr 8, 9:28 AM
alexfh committed rC357921: Remove a useless assertion in clang-check..
Remove a useless assertion in clang-check.
Mon, Apr 8, 9:28 AM
alexfh added a comment to D59135: Add check for matching HeaderFilter before emitting Diagnostic.

Hello @alexfh ,
Let me extend your example

$ cat a.cc 
#include "b.h"
#include "d.h"
int main(){check(nullptr);}
$ cat b.h 
#include "c.h"
inline void b() { c(/*y=*/42); }
$ cat c.h 
void c(int x);
$ cat d.h 
inline char* check(char* buffer)
{
	*buffer++=1; // Should be clang-analyzer-core.NullDereference
	return buffer;
}

Now an additional warning is found and shown (=not suppressed):

$ clang-tidy -checks=-*,clang-*,bugprone-argument-comment a.cc --
2 warnings generated.
/home/default/Temp/clang-tidy-test/d.h:3:11: warning: Dereference of null pointer [clang-analyzer-core.NullDereference]
        *buffer++=1; // Should be clang-analyzer-core.NullDereference
                 ^
/home/default/Temp/clang-tidy-test/a.cc:3:12: note: Calling 'check'
int main(){check(0);}
           ^
/home/default/Temp/clang-tidy-test/d.h:3:3: note: Null pointer value stored to 'buffer'
        *buffer++=1; // Should be clang-analyzer-core.NullDereference
         ^
/home/default/Temp/clang-tidy-test/d.h:3:11: note: Dereference of null pointer
        *buffer++=1; // Should be clang-analyzer-core.NullDereference
                 ^
Suppressed 1 warnings (1 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
Mon, Apr 8, 8:06 AM · Restricted Project
alexfh committed rGc70d38e997b6: Remove a useless assertion in clang-check. (authored by alexfh).
Remove a useless assertion in clang-check.
Mon, Apr 8, 7:17 AM
alexfh committed rC357915: Remove a useless assertion in clang-check..
Remove a useless assertion in clang-check.
Mon, Apr 8, 7:17 AM
alexfh committed rL357915: Remove a useless assertion in clang-check..
Remove a useless assertion in clang-check.
Mon, Apr 8, 7:17 AM
alexfh added a comment to D59977: [Lexer] Fix an off-by-one bug in Lexer::getAsCharRange() - NFC..

It looks like there's a number of users of this function beyond what you've mentioned:
clang-tidy/readability/IsolateDeclarationCheck.cpp:210
clang-tidy/modernize/ConcatNestedNamespacesCheck.cpp:43
clang-tidy/readability/NamespaceCommentCheck.cpp:106
clang/lib/ARCMigrate/PlistReporter.cpp:110
clang/lib/StaticAnalyzer/Core/PlistDiagnostics.cpp:192

Mon, Apr 8, 1:24 AM · Restricted Project, Restricted Project

Fri, Apr 5

alexfh added a comment to D60139: [clang-tidy] Add bugprone-placement-new-target-type-mismatch check.

Looks like this check would fit better into the bugprone module.

Fri, Apr 5, 5:57 AM · Restricted Project, Restricted Project
alexfh requested changes to D56644: [clang-tidy] readability-container-size-empty handle std::string length().
Fri, Apr 5, 5:54 AM · Restricted Project, Restricted Project
alexfh removed a reviewer for D59408: [clang-format] [PR25010] Extend AllowShortIfStatementsOnASingleLine not working if an "else" statement is present: alexfh.
Fri, Apr 5, 5:51 AM · Restricted Project
alexfh added inline comments to D59932: [clang-tidy] Add fix descriptions to clang-tidy checks..
Fri, Apr 5, 5:51 AM · Restricted Project, Restricted Project
alexfh requested changes to D59859: [clang-tidy] FIXIT for implicit bool conversion now obeys upper case suffixes if enforced..
Fri, Apr 5, 5:51 AM · Restricted Project, Restricted Project
alexfh added a comment to D59746: [CommandLineParser] Add DefaultOption flag.

Can you give a specific example of how this problem manifests?

Fri, Apr 5, 5:48 AM · Restricted Project, Restricted Project
alexfh requested changes to D59135: Add check for matching HeaderFilter before emitting Diagnostic.
Fri, Apr 5, 5:45 AM · Restricted Project
alexfh added inline comments to D59932: [clang-tidy] Add fix descriptions to clang-tidy checks..
Fri, Apr 5, 5:06 AM · Restricted Project, Restricted Project
alexfh added a comment to D59540: [clang-tidy] [PR41119] readability-identifier-naming incorrectly fixes lambda capture.

friendly ping

Fri, Apr 5, 5:02 AM · Restricted Project
alexfh added a comment to D59540: [clang-tidy] [PR41119] readability-identifier-naming incorrectly fixes lambda capture.

Thanks, looks better now, but still a few comments, mostly nits.

Fri, Apr 5, 5:02 AM · Restricted Project

Thu, Apr 4

alexfh added a comment to D60151: [clang-tidy] Rename llvm checkers to llvm-project .
  • Rename llvm directory to llvm_project.
  • Change llvm- to llvm-project-.
  • Rename files.
Thu, Apr 4, 5:32 AM · Restricted Project

Wed, Apr 3

alexfh added a comment to D59812: [analyzer] PR41185: Fix regression where __builtin_* functions weren't recognized.

Thanks for addressing this! Please add the test cases from https://bugs.llvm.org/show_bug.cgi?id=41185

Wed, Apr 3, 10:35 AM · Restricted Project, Restricted Project
alexfh added inline comments to D60151: [clang-tidy] Rename llvm checkers to llvm-project .
Wed, Apr 3, 10:28 AM · Restricted Project
alexfh added a comment to D59932: [clang-tidy] Add fix descriptions to clang-tidy checks..

This looks like a more promising direction. Thanks for the readiness to experiment with this.

Wed, Apr 3, 10:15 AM · Restricted Project, Restricted Project
alexfh added a comment to D59449: [clang-tidy] Integrate clang-tidy-diff.py machinery into run-clang-tidy.py.

Why not just use clang-tidy-diff.py? The clang-tidy-diff.py script has a distinct and somewhat self-documenting name and a very specific purpose (find clang-tidy regressions in a patch), while run-clang-tidy.py is more focused on running over larger bodies of code with a purpose of analyzing or cleaning up. Is there any benefit in having all functionality in run-clang-tidy.py?

Both scripts have almost the same implementation, almost the same syntax (at least after -j and -timeout we introduced). So why not merge them.

Wed, Apr 3, 7:50 AM · Restricted Project, Restricted Project
alexfh added inline comments to D33841: [clang-tidy] redundant keyword check.
Wed, Apr 3, 7:14 AM · Restricted Project
alexfh committed rGee737a84d75b: [clang-tidy] Remove the old ClangTidyCheck::registerPPCallbacks method (authored by alexfh).
[clang-tidy] Remove the old ClangTidyCheck::registerPPCallbacks method
Wed, Apr 3, 7:03 AM
alexfh committed rL357582: [clang-tidy] Remove the old ClangTidyCheck::registerPPCallbacks method.
[clang-tidy] Remove the old ClangTidyCheck::registerPPCallbacks method
Wed, Apr 3, 7:03 AM
alexfh committed rCTE357582: [clang-tidy] Remove the old ClangTidyCheck::registerPPCallbacks method.
[clang-tidy] Remove the old ClangTidyCheck::registerPPCallbacks method
Wed, Apr 3, 7:03 AM
alexfh closed D60197: [clang-tidy] Remove the old ClangTidyCheck::registerPPCallbacks method.
Wed, Apr 3, 7:03 AM · Restricted Project
alexfh added a comment to D59963: [clang-tidy] Add a module for the Linux kernel..

Can you verify that the add_new_check.py script works fine with this new module?
Thanks!

Wed, Apr 3, 6:25 AM · Restricted Project, Restricted Project
alexfh accepted D60151: [clang-tidy] Rename llvm checkers to llvm-project .

The change looks fine, but I don't understand the description of this revision. Could you clarify which checkers you're talking about and which bug you observe?

Wed, Apr 3, 6:20 AM · Restricted Project
alexfh created D60197: [clang-tidy] Remove the old ClangTidyCheck::registerPPCallbacks method.
Wed, Apr 3, 6:15 AM · Restricted Project

Tue, Apr 2

alexfh committed rG07ef786652e7: [clang-tidy] make getLangOpts return a const ref (authored by alexfh).
[clang-tidy] make getLangOpts return a const ref
Tue, Apr 2, 4:33 AM
alexfh committed rL357468: [clang-tidy] make getLangOpts return a const ref.
[clang-tidy] make getLangOpts return a const ref
Tue, Apr 2, 4:33 AM
alexfh committed rCTE357468: [clang-tidy] make getLangOpts return a const ref.
[clang-tidy] make getLangOpts return a const ref
Tue, Apr 2, 4:33 AM
alexfh requested changes to D59932: [clang-tidy] Add fix descriptions to clang-tidy checks..
Tue, Apr 2, 2:27 AM · Restricted Project, Restricted Project
alexfh edited reviewers for D60115: Adding 'CLion integration' to clang-format doc, added: krasimir; removed: alexfh.
Tue, Apr 2, 2:16 AM · Restricted Project

Mon, Apr 1

alexfh added inline comments to D59859: [clang-tidy] FIXIT for implicit bool conversion now obeys upper case suffixes if enforced..
Mon, Apr 1, 9:45 AM · Restricted Project, Restricted Project
alexfh added a comment to D59859: [clang-tidy] FIXIT for implicit bool conversion now obeys upper case suffixes if enforced..

I'm not sure why we want this? What is wrong with simply applying clang-tidy twice?

It doubles the execution time of checking a large project (which may be unacceptably slow), and there's no guarantee that twice will be enough in the general case (one set of fixes may trigger a different check's diagnostics, which may get fixed and trigger another check's diagnostics, etc).

Mon, Apr 1, 8:28 AM · Restricted Project, Restricted Project
alexfh added inline comments to D59859: [clang-tidy] FIXIT for implicit bool conversion now obeys upper case suffixes if enforced..
Mon, Apr 1, 8:28 AM · Restricted Project, Restricted Project
alexfh added a comment to D59449: [clang-tidy] Integrate clang-tidy-diff.py machinery into run-clang-tidy.py.

How is this functionality different from the clang-tidy-diff.py script with the -j option being added in the other patch?

It's the same.

Mon, Apr 1, 4:46 AM · Restricted Project, Restricted Project

Fri, Mar 29

alexfh committed rG397ee70180c9: [clang-tidy] Fix PR28406 (authored by alexfh).
[clang-tidy] Fix PR28406
Fri, Mar 29, 1:54 PM
alexfh committed rL357312: [clang-tidy] Fix PR28406.
[clang-tidy] Fix PR28406
Fri, Mar 29, 1:54 PM
alexfh committed rCTE357312: [clang-tidy] Fix PR28406.
[clang-tidy] Fix PR28406
Fri, Mar 29, 1:54 PM
alexfh added a comment to D59963: [clang-tidy] Add a module for the Linux kernel..

Looks reasonable in general, but we usually add modules with at least one check. Let's do the same here.

Fri, Mar 29, 10:14 AM · Restricted Project, Restricted Project

Thu, Mar 28

alexfh added a comment to D59901: [analyzer] PR41239: Fix a crash on invalid source location in NoStoreFuncVisitor..

I've posted another test case to https://bugs.llvm.org/show_bug.cgi?id=41239. It may already be covered by the fix. Could you check?

Thu, Mar 28, 8:27 AM · Restricted Project, Restricted Project

Wed, Mar 27

alexfh accepted D59857: [analyzer] PR37501: Disable the assertion for reverse-engineering logical op short circuits..

LG

Wed, Mar 27, 2:14 AM · Restricted Project

Mon, Mar 25

alexfh accepted D59622: [analyzer] C++17: PR41142: Ignore transparent InitListExprs in ExprEngine as well..

LG. This fixes (at least, most of) the crashes we observe.

Mon, Mar 25, 9:01 AM · Restricted Project
alexfh committed rG629186886405: [clang-tidy] ClangTidy.h -> ClangTidyCheck.h (authored by alexfh).
[clang-tidy] ClangTidy.h -> ClangTidyCheck.h
Mon, Mar 25, 7:46 AM
alexfh committed rCTE356902: [clang-tidy] ClangTidy.h -> ClangTidyCheck.h.
[clang-tidy] ClangTidy.h -> ClangTidyCheck.h
Mon, Mar 25, 7:45 AM
alexfh committed rL356902: [clang-tidy] ClangTidy.h -> ClangTidyCheck.h.
[clang-tidy] ClangTidy.h -> ClangTidyCheck.h
Mon, Mar 25, 7:45 AM
alexfh committed rG478fc5c83e93: [clang-tidy] Switch checks to #include "ClangTidyCheck.h" (authored by alexfh).
[clang-tidy] Switch checks to #include "ClangTidyCheck.h"
Mon, Mar 25, 5:40 AM
alexfh committed rL356892: [clang-tidy] Switch checks to #include "ClangTidyCheck.h".
[clang-tidy] Switch checks to #include "ClangTidyCheck.h"
Mon, Mar 25, 5:40 AM
alexfh committed rCTE356892: [clang-tidy] Switch checks to #include "ClangTidyCheck.h".
[clang-tidy] Switch checks to #include "ClangTidyCheck.h"
Mon, Mar 25, 5:40 AM
alexfh committed rGe9087fe75c86: [clang-tidy] Separate the check-facing interface (authored by alexfh).
[clang-tidy] Separate the check-facing interface
Mon, Mar 25, 5:36 AM
alexfh committed rL356890: [clang-tidy] Separate the check-facing interface.
[clang-tidy] Separate the check-facing interface
Mon, Mar 25, 5:35 AM
alexfh committed rCTE356890: [clang-tidy] Separate the check-facing interface.
[clang-tidy] Separate the check-facing interface
Mon, Mar 25, 5:35 AM
alexfh closed D59714: [clang-tidy] Separate the check-facing interface.
Mon, Mar 25, 5:35 AM · Restricted Project, Restricted Project

Fri, Mar 22

alexfh updated the diff for D59714: [clang-tidy] Separate the check-facing interface.
  • Reverted the <Switch checks to include "ClangTidyCheck.h"> part to simplify the review.
Fri, Mar 22, 1:31 PM · Restricted Project, Restricted Project
alexfh created D59714: [clang-tidy] Separate the check-facing interface.
Fri, Mar 22, 1:28 PM · Restricted Project, Restricted Project
alexfh accepted D59360: [clang-tidy] Fix more false positives for bugprone-string-integer-assignment.

LG. Thanks for improving the check!

Fri, Mar 22, 12:04 PM · Restricted Project, Restricted Project
alexfh committed rGb6c4db9981ce: [clang-tidy] Move all checks to the new registerPPCallbacks API (authored by alexfh).
[clang-tidy] Move all checks to the new registerPPCallbacks API
Fri, Mar 22, 11:57 AM
alexfh committed rL356796: [clang-tidy] Move all checks to the new registerPPCallbacks API.
[clang-tidy] Move all checks to the new registerPPCallbacks API
Fri, Mar 22, 11:57 AM
alexfh committed rCTE356796: [clang-tidy] Move all checks to the new registerPPCallbacks API.
[clang-tidy] Move all checks to the new registerPPCallbacks API
Fri, Mar 22, 11:57 AM
alexfh committed rGb719245a946e: Fix clang-move test. (authored by alexfh).
Fix clang-move test.
Fri, Mar 22, 11:52 AM
alexfh committed rL356795: Fix clang-move test..
Fix clang-move test.
Fri, Mar 22, 11:51 AM
alexfh committed rCTE356795: Fix clang-move test..
Fix clang-move test.
Fri, Mar 22, 11:51 AM
alexfh committed rGa0006211e213: [clang-tidy] anyOf(hasName(..), hasName(..)) -> hasAnyName (authored by alexfh).
[clang-tidy] anyOf(hasName(..), hasName(..)) -> hasAnyName
Fri, Mar 22, 11:37 AM
alexfh committed rL356792: [clang-tidy] anyOf(hasName(..), hasName(..)) -> hasAnyName.
[clang-tidy] anyOf(hasName(..), hasName(..)) -> hasAnyName
Fri, Mar 22, 11:36 AM
alexfh committed rCTE356792: [clang-tidy] anyOf(hasName(..), hasName(..)) -> hasAnyName.
[clang-tidy] anyOf(hasName(..), hasName(..)) -> hasAnyName
Fri, Mar 22, 11:36 AM
alexfh committed rG5e381fb11a01: [clangd] Call the new ClangTidyCheck::registerPPCallbacks overload (authored by alexfh).
[clangd] Call the new ClangTidyCheck::registerPPCallbacks overload
Fri, Mar 22, 11:17 AM
alexfh committed rCTE356788: [clangd] Call the new ClangTidyCheck::registerPPCallbacks overload.
[clangd] Call the new ClangTidyCheck::registerPPCallbacks overload
Fri, Mar 22, 11:17 AM
alexfh committed rL356788: [clangd] Call the new ClangTidyCheck::registerPPCallbacks overload.
[clangd] Call the new ClangTidyCheck::registerPPCallbacks overload
Fri, Mar 22, 11:16 AM
alexfh committed rG1ae5c63f35ef: [clang-tidy] Fix a compiler warning. (authored by alexfh).
[clang-tidy] Fix a compiler warning.
Fri, Mar 22, 8:07 AM