[PATCH] D63128: Fixed google-readability-casting test to work in c++17

SureYeaah created this revision. SureYeaah added reviewers: gribozavr, hokein. Herald added a project: clang. Fixed google-readability-casting.cpp to get tests working in c++17 Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D63128 Files: clang-tools-extra/clang-tidy/google/Av

[PATCH] D63221: Fixed clangd diagnostics priority

SureYeaah created this revision. Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D63221 Files: clang-tools-extra/clangd/Diagnostics.cpp clang-tools-extra

[PATCH] D63222: [Clangd] Fixed clangd diagnostics priority

SureYeaah created this revision. SureYeaah added reviewers: sammccall, kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. - Fixed diagnostics where zero width inserted ranges were being used instead of the whole token - Ad

[PATCH] D63222: [Clangd] Fixed clangd diagnostics priority

SureYeaah updated this revision to Diff 204351. SureYeaah added a comment. Remove extra newline Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63222/new/ https://reviews.llvm.org/D63222 Files: clang-tools-extra/clangd/Diagnostics.cpp clang-tools

[PATCH] D63222: [Clangd] Fixed clangd diagnostics priority

SureYeaah updated this revision to Diff 204467. SureYeaah added a comment. Refactored code as pointed out by @kadircet Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63222/new/ https://reviews.llvm.org/D63222 Files: clang-tools-extra/clangd/Diagno

[PATCH] D63222: [Clangd] Fixed clangd diagnostics priority

SureYeaah updated this revision to Diff 204489. SureYeaah marked 4 inline comments as done. SureYeaah added a comment. Simplified logic for diagnostics range. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63222/new/ https://reviews.llvm.org/D63222

[PATCH] D63222: [Clangd] Fixed clangd diagnostics priority

SureYeaah updated this revision to Diff 204494. SureYeaah marked 2 inline comments as done. SureYeaah added a comment. Formatted code Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63222/new/ https://reviews.llvm.org/D63222 Files: clang-tools-extr

[PATCH] D64562: [clangd] Fixed toHalfOpenFileRange

SureYeaah updated this revision to Diff 209363. SureYeaah marked 12 inline comments as done. SureYeaah added a comment. Added comments and minor changes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64562/new/ https://reviews.llvm.org/D64562 Files

[PATCH] D64562: [clangd] Fixed toHalfOpenFileRange

SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/unittests/SourceCodeTests.cpp:410 +// Test for functions toHalfOpenFileRange and getHalfOpenFileRange +// FIXME: Need better testing support to be able to check more than just Decls. +TEST(SourceCodeTests, Half

[PATCH] D64562: [clangd] Fixed toHalfOpenFileRange

This revision was automatically updated to reflect the committed changes. Closed by commit rL365894: [clangd] Fixed toHalfOpenFileRange (authored by SureYeaah, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.or

[PATCH] D64329: [Clangd] Fixed SelectionTree bug for macros

SureYeaah updated this revision to Diff 209506. SureYeaah added a comment. Added a test case Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64329/new/ https://reviews.llvm.org/D64329 Files: clang-tools-extra/clangd/Selection.cpp clang-tools-ext

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah created this revision. SureYeaah added reviewers: sammccall, kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. - Modified ExtractVariable to stop extraction for MemberExprs and Assignment Expr - Fixed unittests

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah updated this revision to Diff 209809. SureYeaah added a comment. Fixed comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64717/new/ https://reviews.llvm.org/D64717 Files: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah updated this revision to Diff 209844. SureYeaah marked 5 inline comments as done. SureYeaah added a comment. Removed unrelated changes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64717/new/ https://reviews.llvm.org/D64717 Files: clan

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/unittests/TweakTests.cpp:299 // return statement return ^1; } kadircet wrote: > left out this one ? Since that's an IntegerLiteral, this patch doesn't apply to it?

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah added a comment. In D64717#1585512 , @sammccall wrote: > Are you sure we want to disable extraction here, rather than just do the > extraction at a higher level? > > E.g. if `bar(1,2,3, f[[o]]o(4,5));` seems like it should extract the call too

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah marked an inline comment as done. SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/unittests/TweakTests.cpp:323 while(a < ^1) -^a++; +[[a++]]; // do while kadircet wrote: > I thought extractor didn't h

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah updated this revision to Diff 209853. SureYeaah marked 2 inline comments as done. SureYeaah added a comment. Added comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64717/new/ https://reviews.llvm.org/D64717 Files: clang-tools-ext

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah added a comment. In D64717#1585632 , @sammccall wrote: > In D64717#1585542 , @SureYeaah wrote: > > > In D64717#1585512 , @sammccall > > wrote: > > > > > Are you su

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah updated this revision to Diff 209922. SureYeaah added a comment. Added fix for selecting the callExpr of a MemberExpr/Function DeclRefExpr Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64717/new/ https://reviews.llvm.org/D64717 Files:

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah updated this revision to Diff 210045. SureYeaah added a comment. Fixed wrong function name Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64717/new/ https://reviews.llvm.org/D64717 Files: clang-tools-extra/clangd/refactor/tweaks/Extract

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah updated this revision to Diff 210047. SureYeaah added a comment. Fixed crash Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64717/new/ https://reviews.llvm.org/D64717 Files: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp

[PATCH] D64912: [Clangd] Changed ExtractVariable to only work on non empty selections

SureYeaah created this revision. SureYeaah added reviewers: kadircet, sammccall. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. - For now, we don't trigger in any case if it's an empty selection - Fixed unittests Repository: rG

[PATCH] D64912: [Clangd] Changed ExtractVariable to only work on non empty selections

SureYeaah updated this revision to Diff 210526. SureYeaah added a comment. Added tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64912/new/ https://reviews.llvm.org/D64912 Files: clang-tools-extra/clangd/refactor/Tweak.cpp clang-tools-extr

[PATCH] D64912: [Clangd] Changed ExtractVariable to only work on non empty selections

SureYeaah marked an inline comment as done. SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/refactor/Tweak.h:51 +/// The Length of the selection +unsigned SelectionLength; /// The AST nodes that were selected. kadircet wrote: >

[PATCH] D64912: [Clangd] Changed ExtractVariable to only work on non empty selections

SureYeaah updated this revision to Diff 210536. SureYeaah added a comment. Added selectionend Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64912/new/ https://reviews.llvm.org/D64912 Files: clang-tools-extra/clangd/refactor/Tweak.cpp clang-too

[PATCH] D64912: [Clangd] Changed ExtractVariable to only work on non empty selections

SureYeaah updated this revision to Diff 210537. SureYeaah added a comment. Fixed comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64912/new/ https://reviews.llvm.org/D64912 Files: clang-tools-extra/clangd/refactor/Tweak.cpp clang-tools-ex

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah marked 5 inline comments as done. SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:208 const SelectionTree::Node *N = Inputs.ASTSelection.commonAncestor(); - if (!N) -return false; - Target = llvm::make_u

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah updated this revision to Diff 210545. SureYeaah added a comment. Refactored code Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64717/new/ https://reviews.llvm.org/D64717 Files: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.c

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:292 + Target = llvm::make_unique(TargetNode, SM, Ctx); + return true; +} kadircet wrote: > `return Target->InsertionPoint`? Changed to checking if target i

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah updated this revision to Diff 210560. SureYeaah marked 4 inline comments as done. SureYeaah added a comment. Fixed tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64717/new/ https://reviews.llvm.org/D64717 Files: clang-tools-extra/

[PATCH] D64912: [Clangd] Changed ExtractVariable to only work on non empty selections

SureYeaah updated this revision to Diff 210577. SureYeaah added a comment. Added SelectionBegin and SelectionEnd to Tweak::Selection Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64912/new/ https://reviews.llvm.org/D64912 Files: clang-tools-extr

[PATCH] D64912: [Clangd] Changed ExtractVariable to only work on non empty selections

This revision was automatically updated to reflect the committed changes. Closed by commit rL366451: [Clangd] Changed ExtractVariable to only work on non empty selections (authored by SureYeaah, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior t

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah updated this revision to Diff 210590. SureYeaah added a comment. Merged with trigger on non empty selection only Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64717/new/ https://reviews.llvm.org/D64717 Files: clang-tools-extra/clangd/r

[PATCH] D64329: [Clangd] Fixed SelectionTree bug for macros

SureYeaah updated this revision to Diff 210783. SureYeaah marked 2 inline comments as done. SureYeaah added a comment. Updated comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64329/new/ https://reviews.llvm.org/D64329 Files: clang-tools-e

[PATCH] D64329: [Clangd] Fixed SelectionTree bug for macros

This revision was automatically updated to reflect the committed changes. Closed by commit rL366566: [Clangd] Fixed SelectionTree bug for macros (authored by SureYeaah, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://review

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah updated this revision to Diff 210800. SureYeaah marked 5 inline comments as done. SureYeaah added a comment. Minor changes and disabled extraction from label statement Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64717/new/ https://revie

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah updated this revision to Diff 210803. SureYeaah marked an inline comment as done. SureYeaah added a comment. Added test for label Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64717/new/ https://reviews.llvm.org/D64717 Files: clang-too

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/unittests/TweakTests.cpp:427 R"cpp(void f(int a) { auto dummy = 1; label: [ [gsl::suppress("type")] ] for (;;) a = dummy; })cpp"}, kadircet w

[PATCH] D64717: [Clangd] Fixed ExtractVariable for MemberExprs and Assignment Exprs

This revision was automatically updated to reflect the committed changes. Closed by commit rGb0ed5bea8812: [Clangd] Fixed ExtractVariable for certain types of Exprs (authored by SureYeaah). Changed prior to commit: https://reviews.llvm.org/D64717?vs=210803&id=211423#toc Repository: rG LLVM G

[PATCH] D65139: [clangd] Support extraction of binary "subexpressions" like a + [[b + c]].

SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:210 +// - we don't look inside macro expansions in the subexpressions +// - we only adjust the extracted range, so references in the unselected parts +//of the AST expr

[PATCH] D65139: [clangd] Support extraction of binary "subexpressions" like a + [[b + c]].

SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:50 // Generate Replacement for declaring the selected Expr as a new variable - tooling::Replacement insertDeclaration(llvm::StringRef VarName) const; + tooling::Replacem

[PATCH] D65139: [clangd] Support extraction of binary "subexpressions" like a + [[b + c]].

SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:50 // Generate Replacement for declaring the selected Expr as a new variable - tooling::Replacement insertDeclaration(llvm::StringRef VarName) const; + tooling::Replacem

[PATCH] D65139: [clangd] Support extraction of binary "subexpressions" like a + [[b + c]].

SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/unittests/TweakTests.cpp:448 + + // Binary subexpressions + {R"cpp(void f() { sammccall wrote: > SureYeaah wrote: > > Can we have some tests with macros as well? > Added a sim

[PATCH] D65332: [Clangd] Disable ExtractVariable for all types of assignments

SureYeaah created this revision. SureYeaah added reviewers: sammccall, kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D65332 Files: clang-tools-extra/clan

[PATCH] D65332: [Clangd] Disable ExtractVariable for all types of assignments

This revision was automatically updated to reflect the committed changes. Closed by commit rL367113: [Clangd] Disable ExtractVariable for all types of assignments (authored by SureYeaah, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit

[PATCH] D65337: [clangd] Disallow extraction of expression-statements.

SureYeaah added a comment. What was the bug in getCallExpr() ? Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:391 +return false; + Outer->dumpColor(); + // Blacklist the most common places where an expr can appear but be unused.

[PATCH] D65337: [clangd] Disallow extraction of expression-statements.

SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/unittests/TweakTests.cpp:329 while(a < [[1]]) -[[a++]]; +a++; // do while Change to a=[[1]];? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D65486: [clangd] Ignore semicolons, whitespace, and comments in SelectionTree.

SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/Selection.cpp:50 + S.StartOffset = SM.getFileOffset(Tok.location()); + S.EndOffset = S.StartOffset + Tok.length(); + if (S.StartOffset >= SelBegin && S.EndOffset <= SelEnd) Would

[PATCH] D65526: [Clangd] Initial prototype version of ExtractFunction

SureYeaah created this revision. SureYeaah added reviewers: kadircet, sammccall. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ilya-biryukov, mgorny. Herald added a project: clang. - Prototype version; Only looking for a high level review. - Only works for extraction from fre

[PATCH] D65574: Added hack to prevent toHalfOpenFileRange assertion fail

SureYeaah created this revision. SureYeaah added reviewers: sammccall, kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, ilya-biryukov. Herald added a project: clang. - Still need to investigate why this is happening. - Fixes the crash described at https://github.com/clangd/clang

[PATCH] D65574: Added hack to prevent toHalfOpenFileRange assertion fail

SureYeaah marked an inline comment as done. SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/SourceCode.cpp:314 while (!FileRange.getBegin().isFileID()) { -assert(!FileRange.getEnd().isFileID() && - "Both Begin and End should be MacroIDs.");

[PATCH] D65526: [Clangd] Initial prototype version of ExtractFunction

SureYeaah marked 2 inline comments as done. SureYeaah added a comment. In D65526#1612117 , @sammccall wrote: > I guess you're looking for design review at this point. > > But the most important part of that is the documentation of the high-level > structu

[PATCH] D65754: Fix toHalfOpenFileRange assertion fail

SureYeaah created this revision. SureYeaah added reviewers: sammccall, kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, ilya-biryukov. Herald added a project: clang. - Removed assumption that begin and end need to be in same file id. - Fixes the crash at https://github.com/clang

[PATCH] D65754: Fix toHalfOpenFileRange assertion fail

SureYeaah updated this revision to Diff 213583. SureYeaah added a comment. Implemented function to get expansion range such that ends are both in the same file id. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65754/new/ https://reviews.llvm.org/D

[PATCH] D65804: Fixed toHalfOpenFileRange assertion fail

SureYeaah created this revision. Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, ilya-biryukov. Herald added a project: clang. - Added new function that gets Expansion range with both ends in the same file. - Fixes the crash at https://github.com/clangd/clangd/issues/113 R

[PATCH] D65754: Fix toHalfOpenFileRange assertion fail

SureYeaah updated this revision to Diff 213599. SureYeaah marked 8 inline comments as done. SureYeaah added a comment. Addressed review comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65754/new/ https://reviews.llvm.org/D65754 Files: clan

[PATCH] D65754: Fix toHalfOpenFileRange assertion fail

SureYeaah updated this revision to Diff 213614. SureYeaah added a comment. use fileID instead of hash in map Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65754/new/ https://reviews.llvm.org/D65754 Files: clang-tools-extra/clangd/SourceCode.cpp

[PATCH] D65754: Fix toHalfOpenFileRange assertion fail

This revision was automatically updated to reflect the committed changes. Closed by commit rL368058: Fixed toHalfOpenFileRange assertion fail (authored by SureYeaah, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.l

[PATCH] D65878: [Refactor] Moving SourceExtraction header from lib to include

SureYeaah created this revision. SureYeaah added a reviewer: arphaman. Herald added subscribers: cfe-commits, kadircet, dexonsmith, ilya-biryukov. Herald added a project: clang. - Moved the SourceExtraction header from lib to include so that it can be used in clangd. Repository: rG LLVM Githu

[PATCH] D65883: [Extract] Fixed SemicolonExtractionPolicy for Do and Switch Stmt

SureYeaah created this revision. SureYeaah added reviewers: arphaman, sammccall. Herald added subscribers: cfe-commits, dexonsmith. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D65883 Files: clang/lib/Tooling/Refactoring/Extract/SourceExtraction

[PATCH] D65883: [Extract] Fixed SemicolonExtractionPolicy for SwitchStmt and SwitchCase

SureYeaah updated this revision to Diff 213909. SureYeaah added a comment. Removed Do Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65883/new/ https://reviews.llvm.org/D65883 Files: clang/lib/Tooling/Refactoring/Extract/SourceExtraction.cpp In

[PATCH] D65883: [Extract] Fixed SemicolonExtractionPolicy for SwitchStmt and SwitchCase

SureYeaah updated this revision to Diff 213925. SureYeaah added a comment. Added tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65883/new/ https://reviews.llvm.org/D65883 Files: clang/lib/Tooling/Refactoring/Extract/SourceExtraction.cpp c

[PATCH] D65878: [Refactor] Moving SourceExtraction header from lib to include

SureYeaah updated this revision to Diff 213973. SureYeaah added a comment. Added endif comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65878/new/ https://reviews.llvm.org/D65878 Files: clang/include/clang/Tooling/Refactoring/Extract/Source

[PATCH] D65883: [Extract] Fixed SemicolonExtractionPolicy for SwitchStmt and SwitchCase

This revision was automatically updated to reflect the committed changes. Closed by commit rL368267: [Extract] Fixed SemicolonExtractionPolicy for SwitchStmt and SwitchCase (authored by SureYeaah, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior

[PATCH] D65878: [Refactor] Moving SourceExtraction header from lib to include

This revision was automatically updated to reflect the committed changes. Closed by commit rL368743: [Refactor] Moving SourceExtraction header from lib to include (authored by SureYeaah, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit

[PATCH] D65526: [Clangd] Initial prototype version of ExtractFunction

SureYeaah updated this revision to Diff 215077. SureYeaah marked 11 inline comments as done. SureYeaah added a comment. Refactored design and added unit tests. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65526/new/ https://reviews.llvm.org/D65526

[PATCH] D65526: [Clangd] Initial prototype version of ExtractFunction

SureYeaah updated this revision to Diff 215089. SureYeaah added a comment. Removed unrelated changes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65526/new/ https://reviews.llvm.org/D65526 Files: clang-tools-extra/clangd/refactor/tweaks/CMakeLi

[PATCH] D65526: [Clangd] Initial prototype version of ExtractFunction

SureYeaah updated this revision to Diff 215098. SureYeaah added a comment. Removed debug info Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65526/new/ https://reviews.llvm.org/D65526 Files: clang-tools-extra/clangd/refactor/tweaks/CMakeLists.txt

[PATCH] D65526: [Clangd] Initial prototype version of ExtractFunction

SureYeaah updated this revision to Diff 215131. SureYeaah added a comment. Better Naming Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65526/new/ https://reviews.llvm.org/D65526 Files: clang-tools-extra/clangd/refactor/tweaks/CMakeLists.txt cl

[PATCH] D65526: [Clangd] Initial prototype version of ExtractFunction

SureYeaah updated this revision to Diff 215574. SureYeaah added a comment. Fixed bug in getLocType Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65526/new/ https://reviews.llvm.org/D65526 Files: clang-tools-extra/clangd/refactor/tweaks/CMakeList

[PATCH] D65526: [Clangd] Initial prototype version of ExtractFunction

SureYeaah updated this revision to Diff 215615. SureYeaah added a comment. Fixed semicolon bug Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65526/new/ https://reviews.llvm.org/D65526 Files: clang-tools-extra/clangd/refactor/tweaks/CMakeLists.tx

[PATCH] D63773: [clangd] dummy variable extraction on a function scope

SureYeaah created this revision. SureYeaah added reviewers: sammccall, kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, mgorny. Herald added a project: clang. SureYeaah retitled this revision from "dummy variable extraction on a function scope" to "[clangd] dummy variable extrac

[PATCH] D63773: [clangd] dummy variable extraction on a function scope

SureYeaah marked 12 inline comments as done. SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:54 + +// RAV subclass to find all DeclRefs in a given Stmt +class FindDeclRefsVisitor kadircet wrote: > I believ

[PATCH] D63773: [clangd] dummy variable extraction on a function scope

SureYeaah updated this revision to Diff 206663. SureYeaah marked 4 inline comments as done. SureYeaah added a comment. Refactored code - Refactored code as pointed by kadircet - Fixed crash for if statements without an else clause Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D63773: [clangd] dummy variable extraction on a function scope

SureYeaah marked 14 inline comments as done. SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:173 + // give up if extraction will take a variable out of scope + if (!extractionAllowed(ParStmt, N, M)) +brea

[PATCH] D63773: [clangd] dummy variable extraction on a function scope

SureYeaah updated this revision to Diff 207084. SureYeaah added a comment. [Clangd] Refactored code - Created new class Extract to store information about the expression being extracted. - Doesn't fix all of previous comments Looking for comments on the new class stucture Repository: rG LLV

[PATCH] D63773: [clangd] dummy variable extraction on a function scope

SureYeaah marked 13 inline comments as done. SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:71 +} +std::vector Extract::getReferencedDecls() { + // RAV subclass to find all DeclRefs in a given Stmt kadir

[PATCH] D63989: Summary: [Clangd] Added hidden flag --disable-tweaks to disable tweaks- Only for testing purposes

SureYeaah created this revision. SureYeaah added reviewers: sammccall, kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D63989 Files: clang-tools-extra/clan

[PATCH] D63989: Summary: [Clangd] Added hidden flag --disable-tweaks to disable tweaks

SureYeaah updated this revision to Diff 207313. SureYeaah added a comment. Replaced the boolean flag with a list of tweaks flag Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63989/new/ https://reviews.llvm.org/D63989 Files: clang-tools-extra/cla

[PATCH] D63989: [Clangd] Added hidden command line option -tweaks to specify which tweaks to enable

SureYeaah updated this revision to Diff 207321. SureYeaah added a comment. Removed extra comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63989/new/ https://reviews.llvm.org/D63989 Files: clang-tools-extra/clangd/ClangdServer.cpp clang-to

[PATCH] D63989: [Clangd] Added hidden command line option -tweaks to specify which tweaks to enable

SureYeaah updated this revision to Diff 207323. SureYeaah added a comment. Whitespace formatting Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63989/new/ https://reviews.llvm.org/D63989 Files: clang-tools-extra/clangd/ClangdServer.cpp clang-to

[PATCH] D63989: [Clangd] Added hidden command line option -tweaks to specify which tweaks to enable

SureYeaah updated this revision to Diff 207358. SureYeaah marked 3 inline comments as done. SureYeaah added a comment. Refactored code Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63989/new/ https://reviews.llvm.org/D63989 Files: clang-tools-ex

[PATCH] D63989: [Clangd] Added hidden command line option -tweaks to specify which tweaks to enable

This revision was automatically updated to reflect the committed changes. Closed by commit rL364809: Summary: [Clangd] Added hidden command line option -tweaks to specify which… (authored by SureYeaah, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed

[PATCH] D63773: [clangd] dummy variable extraction on a function scope

SureYeaah updated this revision to Diff 207522. SureYeaah added a comment. Removed check for braces and fixed code for finding insertionpoint Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63773/new/ https://reviews.llvm.org/D63773 Files: clang-

[PATCH] D63773: [clangd] dummy variable extraction on a function scope

SureYeaah updated this revision to Diff 208174. SureYeaah marked 21 inline comments as done. SureYeaah added a comment. Added whitelist for computeInsertionPoint Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63773/new/ https://reviews.llvm.org/D637

[PATCH] D63773: [clangd] dummy variable extraction on a function scope

SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:90 +bool isAFunctionRef(const clang::Expr *Expr) { + const clang::DeclRefExpr *DeclRef = dyn_cast_or_null(Expr); + if (DeclRef && isa(DeclRef->getDecl()))

[PATCH] D64329: [Clangd] Fixed SelectionTree bug for macros

SureYeaah created this revision. SureYeaah added reviewers: sammccall, kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Fixed SelectionTree bug for macros - Fixed SelectionTree claimRange for macros and template instantiat

[PATCH] D64329: [Clangd] Fixed SelectionTree bug for macros

SureYeaah updated this revision to Diff 208387. SureYeaah added a comment. Removed debugging code Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64329/new/ https://reviews.llvm.org/D64329 Files: clang-tools-extra/clangd/Selection.cpp clang-tool

[PATCH] D64329: [Clangd] Fixed SelectionTree bug for macros

SureYeaah updated this revision to Diff 208389. SureYeaah added a comment. Removed extra includes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64329/new/ https://reviews.llvm.org/D64329 Files: clang-tools-extra/clangd/Selection.cpp clang-tool

[PATCH] D63773: [clangd] dummy variable extraction on a function scope

This revision was automatically updated to reflect the committed changes. SureYeaah marked 8 inline comments as done. Closed by commit rL365453: dummy variable extraction on a function scope (authored by SureYeaah, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commi

[PATCH] D63773: [clangd] dummy variable extraction on a function scope

SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:187 + std::string ExtractedVarDecl = std::string("auto ") + VarName.str() + " = " + + ExtractionCode.str() + "; "; + return tooling::Replace

[PATCH] D64562: [clangd] Fixed toHalfOpenFileRange

SureYeaah created this revision. SureYeaah added reviewers: sammccall, kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. - Fixed toHalfOpenFileRange to work for macros as well as template instantiations - Added unit tests

[PATCH] D64562: [clangd] Fixed toHalfOpenFileRange

SureYeaah updated this revision to Diff 209193. SureYeaah added a comment. Minor fixes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64562/new/ https://reviews.llvm.org/D64562 Files: clang-tools-extra/clangd/SourceCode.cpp clang-tools-extra/cl

[PATCH] D64329: [Clangd] Fixed SelectionTree bug for macros

SureYeaah updated this revision to Diff 209199. SureYeaah marked 6 inline comments as done. SureYeaah added a comment. Changed claimRange to use toHalfOpenFileRange Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64329/new/ https://reviews.llvm.org/D

[PATCH] D64329: [Clangd] Fixed SelectionTree bug for macros

SureYeaah updated this revision to Diff 209229. SureYeaah added a comment. Changed a breaking test in ExtractVariable Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64329/new/ https://reviews.llvm.org/D64329 Files: clang-tools-extra/clangd/Select

[PATCH] D66872: [clangd] SelectionTree should mark a node as fully-selected if the only claimed tokens were early-claimed.

SureYeaah accepted this revision. SureYeaah added a comment. This revision is now accepted and ready to land. Herald added a subscriber: usaxena95. Sorry about the delay. LGTM. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66872/new/ https:

[PATCH] D68182: [Clangd] Ensure children are always RootStmt in ExtractFunction (Fixes #153)

SureYeaah created this revision. SureYeaah added reviewers: sammccall, kadircet. Herald added subscribers: cfe-commits, usaxena95, arphaman, jkorous, MaskRay, ilya-biryukov, kristof.beyls. Herald added a project: clang. SureYeaah edited the summary of this revision. We weren't always checking if

[PATCH] D68182: [Clangd] Ensure children are always RootStmt in ExtractFunction (Fixes #153)

SureYeaah updated this revision to Diff 93. SureYeaah added a comment. Added llmv_unreachable Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68182/new/ https://reviews.llvm.org/D68182 Files: clang-tools-extra/clangd/refactor/tweaks/ExtractFun

[PATCH] D68182: [Clangd] Ensure children are always RootStmt in ExtractFunction (Fixes #153)

SureYeaah updated this revision to Diff 222499. SureYeaah added a comment. Moved Unittest Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68182/new/ https://reviews.llvm.org/D68182 Files: clang-tools-extra/clangd/refactor/tweaks/ExtractFunction.cp

  1   2   >