- Added checks for broken control flow
- Added unittests
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
- Build Status
Buildable 37501 Build 37500: arc lint + arc unit
Event Timeline
clang-tools-extra/clangd/refactor/tweaks/ExtractFunction.cpp | ||
---|---|---|
437 | rather than writing this twice, take an int Increment parameter? | |
462 | (NumberOfLoops > 0 || NumberOfSwitch = 0) | |
479 | NestedLoops would be clearer I think | |
clang-tools-extra/clangd/unittests/TweakTests.cpp | ||
523 | please add these to a new test case e.g. TEST_F(ExtractFunctionTest, ControlFlow) to avoid each getting too long |
rather than writing this twice, take an int Increment parameter?