SuccessChanges

Summary

  1. OpaquePtr: add Type parameter to Loads analysis API. This makes the functions in Loads.h require a type to be specified independently of the pointer Value so that when pointers have no structure other than address-space, it can still do its job. Most callers had an obvious memory operation handy to provide this type, but a SROA and ArgumentPromotion were doing more complicated analysis. They get updated to merge the properties of the various instructions they were considering.
  2. [DwarfDebug] Dump call site debug info Dump the DWARF information about call sites and call site parameters into debug info sections. The patch also provides an interface for the interpretation of instructions that could load values of a call site parameters in order to generate DWARF about the call site parameters. ([13/13] Introduce the debug entry values.) Co-authored-by: Ananth Sowda <asowda@cisco.com> Co-authored-by: Nikola Prica <nikola.prica@rt-rk.com> Co-authored-by: Ivan Baev <ibaev@cisco.com> Differential Revision: https://reviews.llvm.org/D60716
  3. [RISCV] Fix RISCVTTIImpl::getIntImmCost for immediates where getMinSignedBits() > 64 APInt::getSExtValue will assert if getMinSignedBits() > 64. This can happen, for instance, if examining an i128. Avoid this assertion by checking Imm.getMinSignedBits() <= 64 before doing getTLI()->isLegalAddImmediate(Imm.getSExtValue()). We could directly check getMinSignedBits() <= 12 but it seems better to reuse the isLegalAddImmediate helper for this. Differential Revision: https://reviews.llvm.org/D64390
  4. [docs][llvm-nm] Improve some wording In particular, the --debug-syms switch really doesn't have anything to do with debuggers, so I've updated the document accordingly.
  5. [SelectionDAG] Simplify some calls to getSetCCResultType. NFC DAGTypeLegalizer and SelectionDAGLegalize has helper functions wrapping the call to TLI.getSetCCResultType(...). Use those helpers in more places.
  6. [LegalizeTypes] Fix saturation bug for smul.fix.sat Summary: Make sure we use SETGE instead of SETGT when checking if the sign bit is zero at SMULFIXSAT expansion. The faulty expansion occured when doing "expand" of SMULFIXSAT and the scale was exactly matching the size of the smaller type. For example doing i64 Z = SMULFIXSAT X, Y, 32 and expanding X/Y/Z into using two i32 values. The problem was that we sometimes did not saturate to min when overflowing. Here is an example using Q3.4 numbers: Consider that we are multiplying X and Y. X = 0x80 (-8.0 as Q3.4) Y = 0x20 (2.0 as Q3.4) To avoid loss of precision we do a widening multiplication, getting a 16 bit result Z = 0xF000 (-16.0 as Q7.8) To detect negative overflow we should check if the five most significant bits in Z are less than -1. Assume that we name the 4 most significant bits as HH and the next 4 bits as HL. Then we can do the check by examining if (HH < -1) or (HH == -1 && "sign bit in HL is zero"). The fault was that we have been doing the check as (HH < -1) or (HH == -1 && HL > 0) instead of (HH < -1) or (HH == -1 && HL >= 0). In our example HH is -1 and HL is 0, so the old code did not trigger saturation and simply truncated the result to 0x00 (0.0). With the bugfix we instead detect that we should saturate to min, and the result will be set to 0x80 (-8.0). Reviewers: leonardchan, bevinh Reviewed By: leonardchan Subscribers: hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D64331
  7. Retire VS2015 Support As proposed here: https://lists.llvm.org/pipermail/llvm-dev/2019-June/133147.html This patch raises the minimum supported version to build LLVM/Clang to Visual Studio 2017. Differential Revision: https://reviews.llvm.org/D64326
  8. [docs][llvm-dwarfdump] Make some option descriptions clearer and more precise Some of the wording in the doc (taken largely from the help text), was a little imprecise in some cases, so this patch makes it a little more precise. Reviewed by: JDevlieghere, probinson Differential Revision: https://reviews.llvm.org/D64332
  9. [llvm-profdata] Don't make the output overwrite the input file. Some file systems may not allow this behavior, the test fails on our internal system ("Permission denied").
Revision 365468 by Tim Northover:
OpaquePtr: add Type parameter to Loads analysis API.

This makes the functions in Loads.h require a type to be specified
independently of the pointer Value so that when pointers have no structure
other than address-space, it can still do its job.

Most callers had an obvious memory operation handy to provide this type, but a
SROA and ArgumentPromotion were doing more complicated analysis. They get
updated to merge the properties of the various instructions they were
considering.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/Analysis/Loads.htrunk/include/llvm/Analysis/Loads.h
The file was modified/llvm/trunk/lib/Analysis/Loads.cpptrunk/lib/Analysis/Loads.cpp
The file was modified/llvm/trunk/lib/Analysis/MemDerefPrinter.cpptrunk/lib/Analysis/MemDerefPrinter.cpp
The file was modified/llvm/trunk/lib/Analysis/ValueTracking.cpptrunk/lib/Analysis/ValueTracking.cpp
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpptrunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
The file was modified/llvm/trunk/lib/Transforms/IPO/ArgumentPromotion.cpptrunk/lib/Transforms/IPO/ArgumentPromotion.cpp
The file was modified/llvm/trunk/lib/Transforms/InstCombine/InstCombineCalls.cpptrunk/lib/Transforms/InstCombine/InstCombineCalls.cpp
The file was modified/llvm/trunk/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpptrunk/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp
The file was modified/llvm/trunk/lib/Transforms/Scalar/LICM.cpptrunk/lib/Transforms/Scalar/LICM.cpp
The file was modified/llvm/trunk/lib/Transforms/Scalar/MergeICmps.cpptrunk/lib/Transforms/Scalar/MergeICmps.cpp
The file was modified/llvm/trunk/lib/Transforms/Scalar/SROA.cpptrunk/lib/Transforms/Scalar/SROA.cpp
The file was modified/llvm/trunk/lib/Transforms/Scalar/TailRecursionElimination.cpptrunk/lib/Transforms/Scalar/TailRecursionElimination.cpp
Revision 365467 by djtodoro:
[DwarfDebug] Dump call site debug info

Dump the DWARF information about call sites and call site parameters into
debug info sections.

The patch also provides an interface for the interpretation of instructions
that could load values of a call site parameters in order to generate DWARF
about the call site parameters.

([13/13] Introduce the debug entry values.)

Co-authored-by: Ananth Sowda <asowda@cisco.com>
Co-authored-by: Nikola Prica <nikola.prica@rt-rk.com>
Co-authored-by: Ivan Baev <ibaev@cisco.com>

Differential Revision: https://reviews.llvm.org/D60716
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/docs/LangRef.rsttrunk/docs/LangRef.rst
The file was modified/llvm/trunk/include/llvm/CodeGen/TargetInstrInfo.htrunk/include/llvm/CodeGen/TargetInstrInfo.h
The file was modified/llvm/trunk/include/llvm/CodeGen/TargetRegisterInfo.htrunk/include/llvm/CodeGen/TargetRegisterInfo.h
The file was modified/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpptrunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp
The file was modified/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.htrunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.h
The file was modified/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpptrunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
The file was modified/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.htrunk/lib/CodeGen/AsmPrinter/DwarfDebug.h
The file was modified/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfExpression.cpptrunk/lib/CodeGen/AsmPrinter/DwarfExpression.cpp
The file was modified/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfExpression.htrunk/lib/CodeGen/AsmPrinter/DwarfExpression.h
The file was modified/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.cpptrunk/lib/CodeGen/AsmPrinter/DwarfUnit.cpp
The file was modified/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.htrunk/lib/CodeGen/AsmPrinter/DwarfUnit.h
The file was modified/llvm/trunk/lib/CodeGen/TargetInstrInfo.cpptrunk/lib/CodeGen/TargetInstrInfo.cpp
The file was modified/llvm/trunk/lib/CodeGen/TargetRegisterInfo.cpptrunk/lib/CodeGen/TargetRegisterInfo.cpp
The file was modified/llvm/trunk/lib/DebugInfo/DWARF/DWARFDie.cpptrunk/lib/DebugInfo/DWARF/DWARFDie.cpp
The file was modified/llvm/trunk/lib/IR/DebugInfoMetadata.cpptrunk/lib/IR/DebugInfoMetadata.cpp
The file was modified/llvm/trunk/lib/Target/X86/X86InstrInfo.cpptrunk/lib/Target/X86/X86InstrInfo.cpp
The file was modified/llvm/trunk/lib/Target/X86/X86InstrInfo.htrunk/lib/Target/X86/X86InstrInfo.h
The file was added/llvm/trunk/test/DebugInfo/MIR/X86/dbgcall-site-interpretation.mirtrunk/test/DebugInfo/MIR/X86/dbgcall-site-interpretation.mir
The file was added/llvm/trunk/test/DebugInfo/MIR/X86/dbgcall-site-lea-interpretation.mirtrunk/test/DebugInfo/MIR/X86/dbgcall-site-lea-interpretation.mir
The file was added/llvm/trunk/test/DebugInfo/MIR/X86/debug-call-site-param.mirtrunk/test/DebugInfo/MIR/X86/debug-call-site-param.mir
The file was modified/llvm/trunk/test/DebugInfo/X86/dwarf-callsite-related-attrs.lltrunk/test/DebugInfo/X86/dwarf-callsite-related-attrs.ll
The file was added/llvm/trunk/test/tools/llvm-dwarfdump/X86/stats-dbg-callsite-info.lltrunk/test/tools/llvm-dwarfdump/X86/stats-dbg-callsite-info.ll
The file was modified/llvm/trunk/tools/llvm-dwarfdump/Statistics.cpptrunk/tools/llvm-dwarfdump/Statistics.cpp
Revision 365462 by asb:
[RISCV] Fix RISCVTTIImpl::getIntImmCost for immediates where getMinSignedBits() > 64

APInt::getSExtValue will assert if getMinSignedBits() > 64. This can happen,
for instance, if examining an i128. Avoid this assertion by checking
Imm.getMinSignedBits() <= 64 before doing
getTLI()->isLegalAddImmediate(Imm.getSExtValue()). We could directly check
getMinSignedBits() <= 12 but it seems better to reuse the isLegalAddImmediate
helper for this.

Differential Revision: https://reviews.llvm.org/D64390
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/RISCV/RISCVTargetTransformInfo.cpptrunk/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
The file was modified/llvm/trunk/test/Transforms/ConstantHoisting/RISCV/immediates.lltrunk/test/Transforms/ConstantHoisting/RISCV/immediates.ll
Revision 365461 by jhenderson:
[docs][llvm-nm] Improve some wording

In particular, the --debug-syms switch really doesn't have anything to
do with debuggers, so I've updated the document accordingly.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/docs/CommandGuide/llvm-nm.rsttrunk/docs/CommandGuide/llvm-nm.rst
Revision 365456 by bjope:
[SelectionDAG] Simplify some calls to getSetCCResultType. NFC

DAGTypeLegalizer and SelectionDAGLegalize has helper
functions wrapping the call to TLI.getSetCCResultType(...).
Use those helpers in more places.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpptrunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpptrunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpptrunk/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp
Revision 365455 by bjope:
[LegalizeTypes] Fix saturation bug for smul.fix.sat

Summary:
Make sure we use SETGE instead of SETGT when checking
if the sign bit is zero at SMULFIXSAT expansion.

The faulty expansion occured when doing "expand" of
SMULFIXSAT and the scale was exactly matching the
size of the smaller type. For example doing
  i64 Z = SMULFIXSAT X, Y, 32
and expanding X/Y/Z into using two i32 values.

The problem was that we sometimes did not saturate
to min when overflowing.

Here is an example using Q3.4 numbers:

Consider that we are multiplying X and Y.
  X = 0x80 (-8.0 as Q3.4)
  Y = 0x20 (2.0 as Q3.4)
To avoid loss of precision we do a widening
multiplication, getting a 16 bit result
  Z = 0xF000 (-16.0 as Q7.8)

To detect negative overflow we should check if
the five most significant bits in Z are less than -1.
Assume that we name the 4 most significant bits
as HH and the next 4 bits as HL. Then we can do the
check by examining if
(HH < -1) or (HH == -1 && "sign bit in HL is zero").

The fault was that we have been doing the check as
(HH < -1) or (HH == -1 && HL > 0)
instead of
(HH < -1) or (HH == -1 && HL >= 0).

In our example HH is -1 and HL is 0, so the old
code did not trigger saturation and simply truncated
the result to 0x00 (0.0). With the bugfix we instead
detect that we should saturate to min, and the result
will be set to 0x80 (-8.0).

Reviewers: leonardchan, bevinh

Reviewed By: leonardchan

Subscribers: hiraditya, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D64331
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpptrunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp
The file was modified/llvm/trunk/test/CodeGen/X86/smul_fix_sat.lltrunk/test/CodeGen/X86/smul_fix_sat.ll
Revision 365452 by rksimon:
Retire VS2015 Support

As proposed here: https://lists.llvm.org/pipermail/llvm-dev/2019-June/133147.html

This patch raises the minimum supported version to build LLVM/Clang to Visual Studio 2017.

Differential Revision: https://reviews.llvm.org/D64326
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/cmake/modules/CheckCompilerVersion.cmaketrunk/cmake/modules/CheckCompilerVersion.cmake
The file was modified/llvm/trunk/docs/GettingStarted.rsttrunk/docs/GettingStarted.rst
The file was modified/llvm/trunk/docs/GettingStartedVS.rsttrunk/docs/GettingStartedVS.rst
The file was modified/llvm/trunk/docs/ReleaseNotes.rsttrunk/docs/ReleaseNotes.rst
Revision 365451 by jhenderson:
[docs][llvm-dwarfdump] Make some option descriptions clearer and more precise

Some of the wording in the doc (taken largely from the help text), was a
little imprecise in some cases, so this patch makes it a little more
precise.

Reviewed by: JDevlieghere, probinson

Differential Revision: https://reviews.llvm.org/D64332
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/docs/CommandGuide/llvm-dwarfdump.rsttrunk/docs/CommandGuide/llvm-dwarfdump.rst
Revision 365450 by hokein:
[llvm-profdata] Don't make the output overwrite the input file.

Some file systems may not allow this behavior, the test fails on our internal
system ("Permission denied").
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/tools/llvm-profdata/same-filename.testtrunk/test/tools/llvm-profdata/same-filename.test

Summary

  1. Reland r365355: [Syntax] Introduce syntax trees With a fix to a PS4 buildbot crash.
  2. Revert rL365355 : [Syntax] Introduce syntax trees Summary: A tooling-focused alternative to the AST. This commit focuses on the memory-management strategy and the structure of the AST. More to follow later: - Operations to mutate the syntax trees and corresponding textual replacements. - Mapping between clang AST nodes and syntax tree nodes. - More node types corresponding to the language constructs. Reviewers: sammccall Reviewed By: sammccall Subscribers: llvm-commits, mgorny, cfe-commits Tags: #clang, #llvm Differential Revision: https://reviews.llvm.org/D61637 ........ Fixes buildbots which were crashing on SyntaxTests.exe
  3. [ASTImporter] Added visibility context check for EnumDecl. Summary: ASTImporter makes now difference between enums with same name in different translation units if these are not visible outside. ("Scoped enums" are not handled yet.) Reviewers: martong, a.sidorin, shafik, a_sidorin Reviewed By: a_sidorin Subscribers: rnkovacs, dkrupp, Szelethus, gamesh411, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D62484
  4. Retire VS2015 Support As proposed here: https://lists.llvm.org/pipermail/llvm-dev/2019-June/133147.html This patch raises the minimum supported version to build LLVM/Clang to Visual Studio 2017. Differential Revision: https://reviews.llvm.org/D64326
Revision 365466 by ibiryukov:
Reland r365355: [Syntax] Introduce syntax trees

With a fix to a PS4 buildbot crash.
Change TypePath in RepositoryPath in Workspace
The file was added/cfe/trunk/include/clang/Tooling/Syntax/BuildTree.htrunk/include/clang/Tooling/Syntax/BuildTree.h
The file was added/cfe/trunk/include/clang/Tooling/Syntax/Nodes.htrunk/include/clang/Tooling/Syntax/Nodes.h
The file was added/cfe/trunk/include/clang/Tooling/Syntax/Tree.htrunk/include/clang/Tooling/Syntax/Tree.h
The file was added/cfe/trunk/lib/Tooling/Syntax/BuildTree.cpptrunk/lib/Tooling/Syntax/BuildTree.cpp
The file was modified/cfe/trunk/lib/Tooling/Syntax/CMakeLists.txttrunk/lib/Tooling/Syntax/CMakeLists.txt
The file was added/cfe/trunk/lib/Tooling/Syntax/Nodes.cpptrunk/lib/Tooling/Syntax/Nodes.cpp
The file was added/cfe/trunk/lib/Tooling/Syntax/Tree.cpptrunk/lib/Tooling/Syntax/Tree.cpp
The file was modified/cfe/trunk/unittests/Tooling/Syntax/CMakeLists.txttrunk/unittests/Tooling/Syntax/CMakeLists.txt
The file was added/cfe/trunk/unittests/Tooling/Syntax/TreeTest.cpptrunk/unittests/Tooling/Syntax/TreeTest.cpp
Revision 365465 by rksimon:
Revert rL365355 : [Syntax] Introduce syntax trees

Summary:
A tooling-focused alternative to the AST. This commit focuses on the
memory-management strategy and the structure of the AST.

More to follow later:
  - Operations to mutate the syntax trees and corresponding textual
    replacements.
  - Mapping between clang AST nodes and syntax tree nodes.
  - More node types corresponding to the language constructs.

Reviewers: sammccall

Reviewed By: sammccall

Subscribers: llvm-commits, mgorny, cfe-commits

Tags: #clang, #llvm

Differential Revision: https://reviews.llvm.org/D61637
........
Fixes buildbots which were crashing on SyntaxTests.exe
Change TypePath in RepositoryPath in Workspace
The file was removed/cfe/trunk/include/clang/Tooling/Syntax/BuildTree.htrunk/include/clang/Tooling/Syntax/BuildTree.h
The file was removed/cfe/trunk/include/clang/Tooling/Syntax/Nodes.htrunk/include/clang/Tooling/Syntax/Nodes.h
The file was removed/cfe/trunk/include/clang/Tooling/Syntax/Tree.htrunk/include/clang/Tooling/Syntax/Tree.h
The file was removed/cfe/trunk/lib/Tooling/Syntax/BuildTree.cpptrunk/lib/Tooling/Syntax/BuildTree.cpp
The file was modified/cfe/trunk/lib/Tooling/Syntax/CMakeLists.txttrunk/lib/Tooling/Syntax/CMakeLists.txt
The file was removed/cfe/trunk/lib/Tooling/Syntax/Nodes.cpptrunk/lib/Tooling/Syntax/Nodes.cpp
The file was removed/cfe/trunk/lib/Tooling/Syntax/Tree.cpptrunk/lib/Tooling/Syntax/Tree.cpp
The file was modified/cfe/trunk/unittests/Tooling/Syntax/CMakeLists.txttrunk/unittests/Tooling/Syntax/CMakeLists.txt
The file was removed/cfe/trunk/unittests/Tooling/Syntax/TreeTest.cpptrunk/unittests/Tooling/Syntax/TreeTest.cpp
Revision 365464 by balazske:
[ASTImporter] Added visibility context check for EnumDecl.

Summary:
ASTImporter makes now difference between enums with same name in different translation
units if these are not visible outside.
("Scoped enums" are not handled yet.)

Reviewers: martong, a.sidorin, shafik, a_sidorin

Reviewed By: a_sidorin

Subscribers: rnkovacs, dkrupp, Szelethus, gamesh411, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D62484
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/AST/ASTImporter.cpptrunk/lib/AST/ASTImporter.cpp
The file was modified/cfe/trunk/unittests/AST/ASTImporterVisibilityTest.cpptrunk/unittests/AST/ASTImporterVisibilityTest.cpp
Revision 365454 by rksimon:
Retire VS2015 Support

As proposed here: https://lists.llvm.org/pipermail/llvm-dev/2019-June/133147.html

This patch raises the minimum supported version to build LLVM/Clang to Visual Studio 2017.

Differential Revision: https://reviews.llvm.org/D64326
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/www/get_started.htmltrunk/www/get_started.html

Summary

  1. Enhance abseil-faster-strsplit-delimiter to handle other non-printable characters. Summary: Currently it fails on cases like '\001'. Note: Since `StringLiteral::outputString` dumps most nonprintable characters in octal value, the exact string literal format isn't preserved, e.g. `"\x01"` becomes `'\001'`. Reviewers: gribozavr Reviewed By: gribozavr Subscribers: lebedev.ri, Eugene.Zelenko, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D64151 Patch by Xiaoyi Zhang.
  2. Fixed assertion
  3. dummy variable extraction on a function scope Summary: - Added extraction to a dummy variable - using auto for the dummy variable type for now - Works on a function scope - Adding braces to create a compound statement not supported yet - added unit tests Reviewers: sammccall, kadircet Subscribers: mgorny, jkorous, arphaman, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D63773
Revision 365463 by gribozavr:
Enhance abseil-faster-strsplit-delimiter to handle other non-printable characters.

Summary:
Currently it fails on cases like '\001'.

Note: Since `StringLiteral::outputString` dumps most nonprintable
characters in octal value, the exact string literal format isn't preserved,
e.g. `"\x01"` becomes `'\001'`.

Reviewers: gribozavr

Reviewed By: gribozavr

Subscribers: lebedev.ri, Eugene.Zelenko, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D64151

Patch by Xiaoyi Zhang.
Change TypePath in RepositoryPath in Workspace
The file was modified/clang-tools-extra/trunk/clang-tidy/abseil/FasterStrsplitDelimiterCheck.cpptrunk/clang-tidy/abseil/FasterStrsplitDelimiterCheck.cpp
The file was modified/clang-tools-extra/trunk/test/clang-tidy/abseil-faster-strsplit-delimiter.cpptrunk/test/clang-tidy/abseil-faster-strsplit-delimiter.cpp
Revision 365460 by sureyeaah:
Fixed assertion
Change TypePath in RepositoryPath in Workspace
The file was modified/clang-tools-extra/trunk/clangd/refactor/tweaks/ExtractVariable.cpptrunk/clangd/refactor/tweaks/ExtractVariable.cpp
Revision 365453 by sureyeaah:
dummy variable extraction on a function scope

Summary:
- Added extraction to a dummy variable
- using auto for the dummy variable type for now
- Works on a function scope
- Adding braces to create a compound statement not supported yet
- added unit tests

Reviewers: sammccall, kadircet

Subscribers: mgorny, jkorous, arphaman, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D63773
Change TypePath in RepositoryPath in Workspace
The file was modified/clang-tools-extra/trunk/clangd/refactor/tweaks/CMakeLists.txttrunk/clangd/refactor/tweaks/CMakeLists.txt
The file was added/clang-tools-extra/trunk/clangd/refactor/tweaks/ExtractVariable.cpptrunk/clangd/refactor/tweaks/ExtractVariable.cpp
The file was modified/clang-tools-extra/trunk/clangd/unittests/TweakTests.cpptrunk/clangd/unittests/TweakTests.cpp