FailedChanges

Summary

  1. [COFF, ARM64] Remove definitions for _byteswap library functions _byteswap_* functions are are implemented in below file as normal function from libucrt.lib and declared in stdlib.h. Define them in intrin.h triggers lld error "conflicting comdat type" and "duplicate symbols" which was just added to LLD (https://reviews.llvm.org/D57324). C:\Program Files (x86)\Windows Kits\10\Source\10.0.17763.0\ucrt\stdlib\byteswap.cpp Differential Revision: https://reviews.llvm.org/D57915
  2. [MemorySSA] Remove verifyClobberSanity. Summary: This verification may fail after certain transformations due to BasicAA's fragility. Added a small explanation and a testcase that triggers the assert in checkClobberSanity (before its removal). Addresses PR40509. Reviewers: george.burgess.iv Subscribers: sanjoy, jlebar, llvm-commits, Prazek Tags: #llvm Differential Revision: https://reviews.llvm.org/D57973
  3. Refactor setAlreadyUnrolled() and setAlreadyVectorized(). Loop::setAlreadyUnrolled() and LoopVectorizeHints::setLoopAlreadyUnrolled() both add loop metadata that stops the same loop from being transformed multiple times. This patch merges both implementations. In doing so we fix 3 potential issues: * setLoopAlreadyUnrolled() kept the llvm.loop.vectorize/interleave.* metadata even though it will not be used anymore. This already caused problems such as http://llvm.org/PR40546. Change the behavior to the one of setAlreadyUnrolled which deletes this loop metadata. * setAlreadyUnrolled() used to create a new LoopID by calling MDNode::get with nullptr as the first operand, then replacing it by the returned references using replaceOperandWith. It is possible that MDNode::get would instead return an existing node (due to de-duplication) that then gets modified. To avoid, use a fresh TempMDNode that does not get uniqued with anything else before replacing it with replaceOperandWith. * LoopVectorizeHints::matchesHintMetadataName() only compares the suffix of the attribute to set the new value for. That is, when called with "enable", would erase attributes such as "llvm.loop.unroll.enable", "llvm.loop.vectorize.enable" and "llvm.loop.distribute.enable" instead of the one to replace. Fortunately, function was only called with "isvectorized". Differential Revision: https://reviews.llvm.org/D57566
  4. [NFC] Re-enable XFAILed fuzzer test on iOS
  5. [InstCombine] Fix matchRotate bug when one operand is a ConstantExpr shift This bug seems to be harmless in release builds, but will cause an error in UBSAN builds or an assertion failure in debug builds. When it gets to this opcode comparison, it assumes both of the operands are BinaryOperators, but the prior m_LogicalShift will also match a ConstantExpr. The cast<BinaryOperator> will assert in a debug build, or reading an invalid value for BinaryOp from memory with ((BinaryOperator*)constantExpr)->getOpcode() will cause an error in a UBSAN build. The test I added will fail without this change in debug/UBSAN builds, but not in release. Patch by: @AndrewScheidecker (Andrew Scheidecker) Differential Revision: https://reviews.llvm.org/D58049
  6. [SelectionDAGBuilder] Add restrictions to EmitFuncArgumentDbgValue Summary: This patch fixes PR40587. When a dbg.value instrinsic is emitted to the DAG by using EmitFuncArgumentDbgValue the resulting DBG_VALUE is hoisted to the beginning of the entry block. I think the idea is to be able to locate a formal argument already from the start of the function. However, EmitFuncArgumentDbgValue only checked that the value that was used to describe a variable was originating from a function parameter, not that the variable itself actually was an argument to the function. So when for example assigning a local variable "local" the value from an argument "a", the assocated DBG_VALUE instruction would be hoisted to the beginning of the function, even if the scope for "local" started somewhere else (or if "local" was mapped to other values earlier in the function). This patch adds some logic to EmitFuncArgumentDbgValue to check that the variable being described actually is an argument to the function. And that the dbg.value being lowered already is in the entry block. Otherwise we bail out, and the dbg.value will be handled as an ordinary dbg.value (not as a "FuncArgumentDbgValue"). A tricky situation is when both the variable and the value is related to function arguments, but not neccessarily the same argument. We make sure that we do not describe the same argument more than once as a "FuncArgumentDbgValue". This solution works as long as opt has injected a "first" dbg.value that corresponds to the formal argument at the function entry. Reviewers: jmorse, aprantl Subscribers: jyknight, hiraditya, fedor.sergeev, dstenb, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D57702
  7. [LICM&MSSA] Limit store hoisting. Summary: If there is no clobbering access for a store inside the loop, that store can only be hoisted if there are no interfearing loads. A more general verification introduced here: there are no loads that are not optimized to an access outside the loop. Addresses PR40586. Reviewers: george.burgess.iv Subscribers: sanjoy, jlebar, Prazek, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D57967
  8. [TargetLibraryInfo] Update run time support for Windows It seems that the run time for Windows has changed and supports more math functions than it used to, especially on AArch64, ARM, and AMD64. Fixes PR40541. Differential revision: https://reviews.llvm.org/D57625
  9. [AArch64][GlobalISel] Add isel support for a couple vector exts/truncs Add support for - v4s16 <-> v4s32 - v2s64 <-> v2s32 And update tests that use them to show that we generate the correct instructions. Differential Revision: https://reviews.llvm.org/D57832
  10. [GlobalISel][AArch64] NFC: Remove unnecessary IR from select-fp-casts.mir The IR section in this test doesn't do anything, so there's no point in it being there. Since it's redundant, just remove it.
  11. [DebugInfo] Fix /usr/lib/debug llvm-symbolizer lookup with relative paths Summary: rL189250 added a realpath call, and rL352916 because realpath breaks assumptions with some build systems. However, the /usr/lib/debug case has been clarified, falling back to /usr/lib/debug is currently broken if the obj passed in is a relative path. Adding a call to use absolute paths when falling back to /usr/lib/debug fixes that while still not making any realpath assumptions. This also adds a --fallback-debug-path command line flag for testing (since we probably can't write to /usr/lib/debug from buildbot environments), but was also verified manually: ``` $ rm -f path/to/dwarfdump-test.elf-x86-64 $ strace llvm-symbolizer --obj=relative/path/to/dwarfdump-test.elf-x86-64.debuglink 0x40113f |& grep dwarfdump ``` Lookups went to relative/path/to/dwarfdump-test.elf-x86-64, relative/path/to/.debug/dwarfdump-test.elf-x86-64, and then finally /usr/lib/debug/absolute/path/to/dwarfdump-test.elf-x86-64. Reviewers: dblaikie, samsonov Reviewed By: dblaikie Subscribers: krytarowski, aprantl, hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D57916
  12. Attempt to pacify bots more after r353718 and r353725
  13. [MCA][Scheduler] Track resources that were found busy when issuing an instruction. This is a follow up of r353706. When the scheduler fails to issue a ready instruction to the underlying pipelines, it now updates a mask of 'busy resource units'. That information will be used in future to obtain the set of "problematic" resources in the case of bottlenecks caused by resource pressure. No functional change intended.
  14. Attempt to pacify bots after r353718
  15. [PowerPC] Avoid scalarization of vector truncate The PowerPC code generator currently scalarizes vector truncates that would fit in a vector register, resulting in vector extracts, scalar operations, and vector merges. This patch custom lowers a vector truncate that would fit in a register to a vector shuffle instead. Differential Revision: https://reviews.llvm.org/D56507
  16. [GlobalISel][AArch64] Select G_FFLOOR This teaches the legalizer about G_FFLOOR, and lets us select G_FFLOOR in AArch64. It updates the existing floating point tests, and adds a select-floor.mir test. Differential Revision: https://reviews.llvm.org/D57486
  17. Recommit "[GlobalISel] Add IRTranslator support for G_FFLOOR" After the changes introduced in r353586, this instruction doesn't cause any issues for any backend. Original review: https://reviews.llvm.org/D57485
  18. GlobalISel: Add G_FCANONICALIZE instruction
  19. Make test actually test something (colons were missing)
  20. [AMDGPU] fix atomic_optimizations_buffer.ll test after DPP combiner was enabled by default. Related commits: rL353691, rL353703.
  21. [X86] Regenerate insertelement tests Add common X86/X64 prefixes (and use X86 instead of X32)
  22. Add recipes for migrating downstream branches of git mirrors Add some common recipes for downstream users developing on top of the existing git mirrors. These instructions show how to migrate local branches to the monorepo. Differential Revision: https://reviews.llvm.org/D56550
  23. Revamp the "[clangd] Format tweak's replacements" Summary: This patch contains two parts: 1) reverts commit r353306. 2) move the format logic out from tweaks, keep tweaks API unchanged. Reviewers: sammccall, ilya-biryukov Reviewed By: ilya-biryukov Subscribers: ilya-biryukov, ioeric, MaskRay, jkorous, arphaman, kadircet, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D58051
  24. Fixed header underline in docs. + Removed trailing whitespace.
  25. Move some classes into anonymous namespaces. NFC.
  26. [clangd] Prefer location from codegen files when merging symbols. Summary: For example, if an index symbol has location in a .proto file and an AST symbol has location in a generated .proto.h file, then we prefer location in .proto which is more meaningful to users. Also use `mergeSymbols` to get the preferred location between AST location and index location in go-to-def. Reviewers: sammccall Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, kadircet, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D58037
  27. [tools] Fix python DeprecationWarning: invalid escape sequence The python documentation says "it’s highly recommended that you use raw strings for all but the simplest expressions." (https://docs.python.org/3/library/re.html) So do that with the attached patch generated by sed -i -e "s/re.search('/re.search(r'/g" $(git grep -l 're.search(') The warning can be seen in e.g. python3.7: $ python3.7 -Wd >>> import re; re.search('\s', '') <stdin>:1: DeprecationWarning: invalid escape sequence \s Commited on behalf of Marco Falke. Differential Revision: https://reviews.llvm.org/D57528
  28. [MCA] Return a mask of busy resources from method ResourceManager::checkAvailability(). NFCI In case of bottlenecks caused by pipeline pressure, we want to be able to correctly report the set of problematic pipelines. This is a first step towards adding support for bottleneck hints in llvm-mca (see PR37494). No functional change intended.
  29. Make some helper functions static. NFC.
  30. [AMDGPU] Remove unused variable
  31. [AMDGPU] Fix DPP sequence in atomic optimizer. This commit fixes the DPP sequence in the atomic optimizer (which was previously missing the row_shr:3 step), and works around a read_register exec bug by using a ballot instead. Differential Revision: https://reviews.llvm.org/D57737
  32. Revert "[X86][SSE] Generalize X86ISD::BLENDI support to more value types" This reverts commit r353610. It causes a miscompile visible in macro expansion in a bootstrapped clang. http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20190211/626590.html
  33. [analyzer] New checker for detecting usages of unsafe I/O functions There are certain unsafe or deprecated (since C11) buffer handling functions which should be avoided in safety critical code. They could cause buffer overflows. A new checker, 'security.insecureAPI.DeprecatedOrUnsafeBufferHandling' warns for every occurrence of such functions (unsafe or deprecated printf, scanf family, and other buffer handling functions, which now have a secure variant). Patch by Dániel Kolozsvári! Differential Revision: https://reviews.llvm.org/D35068
  34. Format isInSystemMacro after D55782
  35. [clangd] Index parameters in function decls Reviewers: hokein Subscribers: ilya-biryukov, ioeric, MaskRay, jkorous, arphaman, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D57950
  36. [clang][Index] Add a knob to index function parameters in declarations Summary: Parameters in declarations are useful for clangd, so that we can provide symbol information for them as well. It also helps clangd to be consistent whether a function's definition is accessible or not. Reviewers: hokein, akyrtzi Subscribers: ilya-biryukov, ioeric, arphaman, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D57949
  37. [clangd] Fix broken windows build bots.
  38. [TEST] Add missing opportunity test for PR39673
  39. [ARM] Add v8m.base pattern for add negative imm The v8m.base ISA contains movw, which can operate on an unsigned 16-bit value. Add the pattern that converts an add with a negative value, that could fit into 16-bits when negated, into a sub with that positive value. Differential Revision: https://reviews.llvm.org/D57942
  40. [AMDGPU] Enable DPP combiner pass by default. Related revisions: https://reviews.llvm.org/D55444, https://reviews.llvm.org/D55314
  41. [libclang] Add attribute support for 'convergent'. This bumps CINDEX_VERSION_MINOR up (to 51). Patch by Hsin-Hsiao Lin. Differential Revision: https://reviews.llvm.org/D57946
  42. [NFC][ARM] Simplify loop-indexing codegen test Remove unnecessary offset checks, CHECK-BASE checks and add some extra -NOT checks and TODO comments.
Revision 353740 by tomtan:
[COFF, ARM64] Remove definitions for _byteswap library functions

_byteswap_* functions are are implemented in below file as normal function
from libucrt.lib and declared in stdlib.h. Define them in intrin.h triggers
lld error "conflicting comdat type" and "duplicate symbols" which was just
added to LLD (https://reviews.llvm.org/D57324).

C:\Program Files (x86)\Windows Kits\10\Source\10.0.17763.0\ucrt\stdlib\byteswap.cpp

Differential Revision: https://reviews.llvm.org/D57915
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/Headers/intrin.h (diff)clang.src/lib/Headers/intrin.h
The file was modified/cfe/trunk/test/Headers/ms-arm64-intrin.cpp (diff)clang.src/test/Headers/ms-arm64-intrin.cpp
Revision 353739 by asbirlea:
[MemorySSA] Remove verifyClobberSanity.

Summary:
This verification may fail after certain transformations due to
BasicAA's fragility. Added a small explanation and a testcase that
triggers the assert in checkClobberSanity (before its removal).
Addresses PR40509.

Reviewers: george.burgess.iv

Subscribers: sanjoy, jlebar, llvm-commits, Prazek

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D57973
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/Analysis/MemorySSA.h (diff)llvm.src/include/llvm/Analysis/MemorySSA.h
The file was modified/llvm/trunk/lib/Analysis/MemorySSA.cpp (diff)llvm.src/lib/Analysis/MemorySSA.cpp
The file was added/llvm/trunk/test/Analysis/MemorySSA/pr40509.llllvm.src/test/Analysis/MemorySSA/pr40509.ll
Revision 353738 by meinersbur:
Refactor setAlreadyUnrolled() and setAlreadyVectorized().

Loop::setAlreadyUnrolled() and
LoopVectorizeHints::setLoopAlreadyUnrolled() both add loop metadata that
stops the same loop from being transformed multiple times. This patch
merges both implementations.

In doing so we fix 3 potential issues:

* setLoopAlreadyUnrolled() kept the llvm.loop.vectorize/interleave.*
   metadata even though it will not be used anymore. This already caused
   problems such as http://llvm.org/PR40546. Change the behavior to the
   one of setAlreadyUnrolled which deletes this loop metadata.

* setAlreadyUnrolled() used to create a new LoopID by calling
   MDNode::get with nullptr as the first operand, then replacing it by
   the returned references using replaceOperandWith. It is possible
   that MDNode::get would instead return an existing node (due to
   de-duplication) that then gets modified. To avoid, use a fresh
   TempMDNode that does not get uniqued with anything else before
   replacing it with replaceOperandWith.

* LoopVectorizeHints::matchesHintMetadataName() only compares the
   suffix of the attribute to set the new value for. That is, when
   called with "enable", would erase attributes such as
   "llvm.loop.unroll.enable", "llvm.loop.vectorize.enable" and
   "llvm.loop.distribute.enable" instead of the one to replace.
   Fortunately, function was only called with "isvectorized".

Differential Revision: https://reviews.llvm.org/D57566
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/Analysis/LoopInfo.h (diff)llvm.src/include/llvm/Analysis/LoopInfo.h
The file was modified/llvm/trunk/include/llvm/Transforms/Vectorize/LoopVectorizationLegality.h (diff)llvm.src/include/llvm/Transforms/Vectorize/LoopVectorizationLegality.h
The file was modified/llvm/trunk/lib/Analysis/LoopInfo.cpp (diff)llvm.src/lib/Analysis/LoopInfo.cpp
The file was modified/llvm/trunk/lib/Transforms/Vectorize/LoopVectorizationLegality.cpp (diff)llvm.src/lib/Transforms/Vectorize/LoopVectorizationLegality.cpp
The file was modified/llvm/trunk/test/Transforms/LoopVectorize/X86/vect.omp.force.small-tc.ll (diff)llvm.src/test/Transforms/LoopVectorize/X86/vect.omp.force.small-tc.ll
The file was added/llvm/trunk/test/Transforms/LoopVectorize/remove_metadata.llllvm.src/test/Transforms/LoopVectorize/remove_metadata.ll
Revision 353737 by yln:
[NFC] Re-enable XFAILed fuzzer test on iOS
Change TypePath in RepositoryPath in Workspace
The file was modified/compiler-rt/trunk/test/fuzzer/merge.test (diff)compiler-rt.src/test/fuzzer/merge.test
Revision 353736 by spatel:
[InstCombine] Fix matchRotate bug when one operand is a ConstantExpr shift

This bug seems to be harmless in release builds, but will cause an error in UBSAN
builds or an assertion failure in debug builds.

When it gets to this opcode comparison, it assumes both of the operands are BinaryOperators,
but the prior m_LogicalShift will also match a ConstantExpr. The cast<BinaryOperator> will
assert in a debug build, or reading an invalid value for BinaryOp from memory with
((BinaryOperator*)constantExpr)->getOpcode() will cause an error in a UBSAN build.

The test I added will fail without this change in debug/UBSAN builds, but not in release.

Patch by: @AndrewScheidecker (Andrew Scheidecker)

Differential Revision: https://reviews.llvm.org/D58049
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp (diff)llvm.src/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
The file was modified/llvm/trunk/test/Transforms/InstCombine/rotate.ll (diff)llvm.src/test/Transforms/InstCombine/rotate.ll
Revision 353735 by bjope:
[SelectionDAGBuilder] Add restrictions to EmitFuncArgumentDbgValue

Summary:
This patch fixes PR40587.

When a dbg.value instrinsic is emitted to the DAG
by using EmitFuncArgumentDbgValue the resulting
DBG_VALUE is hoisted to the beginning of the entry
block. I think the idea is to be able to locate
a formal argument already from the start of the
function.
However, EmitFuncArgumentDbgValue only checked that
the value that was used to describe a variable was
originating from a function parameter, not that the
variable itself actually was an argument to the
function. So when for example assigning a local
variable "local" the value from an argument "a",
the assocated DBG_VALUE instruction would be hoisted
to the beginning of the function, even if the scope
for "local" started somewhere else (or if "local"
was mapped to other values earlier in the function).

This patch adds some logic to EmitFuncArgumentDbgValue
to check that the variable being described actually
is an argument to the function. And that the dbg.value
being lowered already is in the entry block. Otherwise
we bail out, and the dbg.value will be handled as an
ordinary dbg.value (not as a "FuncArgumentDbgValue").

A tricky situation is when both the variable and
the value is related to function arguments, but not
neccessarily the same argument. We make sure that we
do not describe the same argument more than once as
a "FuncArgumentDbgValue". This solution works as long
as opt has injected a "first" dbg.value that corresponds
to the formal argument at the function entry.

Reviewers: jmorse, aprantl

Subscribers: jyknight, hiraditya, fedor.sergeev, dstenb, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D57702
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/CodeGen/FunctionLoweringInfo.h (diff)llvm.src/include/llvm/CodeGen/FunctionLoweringInfo.h
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/FunctionLoweringInfo.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/FunctionLoweringInfo.cpp
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
The file was added/llvm/trunk/test/DebugInfo/X86/dbg-value-funcarg.llllvm.src/test/DebugInfo/X86/dbg-value-funcarg.ll
The file was added/llvm/trunk/test/DebugInfo/X86/dbg-value-funcarg2.llllvm.src/test/DebugInfo/X86/dbg-value-funcarg2.ll
Revision 353734 by asbirlea:
[LICM&MSSA] Limit store hoisting.

Summary:
If there is no clobbering access for a store inside the loop, that store
can only be hoisted if there are no interfearing loads.
A more general verification introduced here: there are no loads that are
not optimized to an access outside the loop.
Addresses PR40586.

Reviewers: george.burgess.iv

Subscribers: sanjoy, jlebar, Prazek, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D57967
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/Transforms/Utils/LoopUtils.h (diff)llvm.src/include/llvm/Transforms/Utils/LoopUtils.h
The file was modified/llvm/trunk/lib/Transforms/Scalar/LICM.cpp (diff)llvm.src/lib/Transforms/Scalar/LICM.cpp
The file was modified/llvm/trunk/lib/Transforms/Scalar/LoopSink.cpp (diff)llvm.src/lib/Transforms/Scalar/LoopSink.cpp
The file was modified/llvm/trunk/test/Transforms/LICM/store-hoisting.ll (diff)llvm.src/test/Transforms/LICM/store-hoisting.ll
Revision 353733 by evandro:
[TargetLibraryInfo] Update run time support for Windows

It seems that the run time for Windows has changed and supports more math
functions than it used to, especially on AArch64, ARM, and AMD64.

Fixes PR40541.

Differential revision: https://reviews.llvm.org/D57625
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Analysis/TargetLibraryInfo.cpp (diff)llvm.src/lib/Analysis/TargetLibraryInfo.cpp
The file was modified/llvm/trunk/test/Transforms/InstCombine/double-float-shrink-1.ll (diff)llvm.src/test/Transforms/InstCombine/double-float-shrink-1.ll
The file was modified/llvm/trunk/test/Transforms/InstCombine/double-float-shrink-2.ll (diff)llvm.src/test/Transforms/InstCombine/double-float-shrink-2.ll
The file was modified/llvm/trunk/test/Transforms/InstCombine/pow-1.ll (diff)llvm.src/test/Transforms/InstCombine/pow-1.ll
The file was modified/llvm/trunk/test/Transforms/InstCombine/win-math.ll (diff)llvm.src/test/Transforms/InstCombine/win-math.ll
Revision 353732 by paquette:
[AArch64][GlobalISel] Add isel support for a couple vector exts/truncs

Add support for

- v4s16 <-> v4s32
- v2s64 <-> v2s32

And update tests that use them to show that we generate the correct
instructions.

Differential Revision: https://reviews.llvm.org/D57832
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/AArch64/AArch64LegalizerInfo.cpp (diff)llvm.src/lib/Target/AArch64/AArch64LegalizerInfo.cpp
The file was modified/llvm/trunk/test/CodeGen/AArch64/GlobalISel/select-fp-casts.mir (diff)llvm.src/test/CodeGen/AArch64/GlobalISel/select-fp-casts.mir
The file was modified/llvm/trunk/test/CodeGen/AArch64/arm64-vcvt_f.ll (diff)llvm.src/test/CodeGen/AArch64/arm64-vcvt_f.ll
Revision 353731 by paquette:
[GlobalISel][AArch64] NFC: Remove unnecessary IR from select-fp-casts.mir

The IR section in this test doesn't do anything, so there's no point in it
being there. Since it's redundant, just remove it.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/CodeGen/AArch64/GlobalISel/select-fp-casts.mir (diff)llvm.src/test/CodeGen/AArch64/GlobalISel/select-fp-casts.mir
Revision 353730 by rupprecht:
[DebugInfo] Fix /usr/lib/debug llvm-symbolizer lookup with relative paths

Summary:
rL189250 added a realpath call, and rL352916 because realpath breaks assumptions with some build systems. However, the /usr/lib/debug case has been clarified, falling back to /usr/lib/debug is currently broken if the obj passed in is a relative path. Adding a call to use absolute paths when falling back to /usr/lib/debug fixes that while still not making any realpath assumptions.

This also adds a --fallback-debug-path command line flag for testing (since we probably can't write to /usr/lib/debug from buildbot environments), but was also verified manually:

```
$ rm -f path/to/dwarfdump-test.elf-x86-64
$ strace llvm-symbolizer --obj=relative/path/to/dwarfdump-test.elf-x86-64.debuglink 0x40113f |& grep dwarfdump
```

Lookups went to relative/path/to/dwarfdump-test.elf-x86-64, relative/path/to/.debug/dwarfdump-test.elf-x86-64, and then finally /usr/lib/debug/absolute/path/to/dwarfdump-test.elf-x86-64.

Reviewers: dblaikie, samsonov

Reviewed By: dblaikie

Subscribers: krytarowski, aprantl, hiraditya, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D57916
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/DebugInfo/Symbolize/Symbolize.h (diff)llvm.src/include/llvm/DebugInfo/Symbolize/Symbolize.h
The file was modified/llvm/trunk/lib/DebugInfo/Symbolize/Symbolize.cpp (diff)llvm.src/lib/DebugInfo/Symbolize/Symbolize.cpp
The file was added/llvm/trunk/test/DebugInfo/symbolize-gnu-debuglink-fallback.testllvm.src/test/DebugInfo/symbolize-gnu-debuglink-fallback.test
The file was modified/llvm/trunk/tools/llvm-symbolizer/llvm-symbolizer.cpp (diff)llvm.src/tools/llvm-symbolizer/llvm-symbolizer.cpp
Revision 353729 by nico:
Attempt to pacify bots more after r353718 and r353725
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/test/CodeGen/ms-x86-intrinsics.c (diff)clang.src/test/CodeGen/ms-x86-intrinsics.c
Revision 353728 by adibiagio:
[MCA][Scheduler] Track resources that were found busy when issuing an instruction.

This is a follow up of r353706. When the scheduler fails to issue a ready
instruction to the underlying pipelines, it now updates a mask of 'busy resource
units'. That information will be used in future to obtain the set of
"problematic" resources in the case of bottlenecks caused by resource pressure.
No functional change intended.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/MCA/HardwareUnits/Scheduler.h (diff)llvm.src/include/llvm/MCA/HardwareUnits/Scheduler.h
The file was modified/llvm/trunk/lib/MCA/HardwareUnits/Scheduler.cpp (diff)llvm.src/lib/MCA/HardwareUnits/Scheduler.cpp
Revision 353725 by nico:
Attempt to pacify bots after r353718
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/test/CodeGen/ms-x86-intrinsics.c (diff)clang.src/test/CodeGen/ms-x86-intrinsics.c
Revision 353724 by froese:
[PowerPC] Avoid scalarization of vector truncate

The PowerPC code generator currently scalarizes vector truncates that would fit in a vector register, resulting in vector extracts, scalar operations, and vector merges. This patch custom lowers a vector truncate that would fit in a register to a vector shuffle instead.

Differential Revision: https://reviews.llvm.org/D56507
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/PowerPC/PPCISelLowering.cpp (diff)llvm.src/lib/Target/PowerPC/PPCISelLowering.cpp
The file was modified/llvm/trunk/lib/Target/PowerPC/PPCISelLowering.h (diff)llvm.src/lib/Target/PowerPC/PPCISelLowering.h
The file was modified/llvm/trunk/test/CodeGen/PowerPC/vec-trunc.ll (diff)llvm.src/test/CodeGen/PowerPC/vec-trunc.ll
Revision 353722 by paquette:
[GlobalISel][AArch64] Select G_FFLOOR

This teaches the legalizer about G_FFLOOR, and lets us select G_FFLOOR in
AArch64.

It updates the existing floating point tests, and adds a select-floor.mir test.

Differential Revision: https://reviews.llvm.org/D57486
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/GlobalISel/LegalizerHelper.cpp (diff)llvm.src/lib/CodeGen/GlobalISel/LegalizerHelper.cpp
The file was modified/llvm/trunk/lib/Target/AArch64/AArch64LegalizerInfo.cpp (diff)llvm.src/lib/Target/AArch64/AArch64LegalizerInfo.cpp
The file was modified/llvm/trunk/lib/Target/AArch64/AArch64RegisterBankInfo.cpp (diff)llvm.src/lib/Target/AArch64/AArch64RegisterBankInfo.cpp
The file was added/llvm/trunk/test/CodeGen/AArch64/GlobalISel/select-floor.mirllvm.src/test/CodeGen/AArch64/GlobalISel/select-floor.mir
The file was modified/llvm/trunk/test/CodeGen/AArch64/arm64-vfloatintrinsics.ll (diff)llvm.src/test/CodeGen/AArch64/arm64-vfloatintrinsics.ll
The file was modified/llvm/trunk/test/CodeGen/AArch64/f16-instructions.ll (diff)llvm.src/test/CodeGen/AArch64/f16-instructions.ll
Revision 353720 by paquette:
Recommit "[GlobalISel] Add IRTranslator support for G_FFLOOR"

After the changes introduced in r353586, this instruction doesn't cause any
issues for any backend.

Original review: https://reviews.llvm.org/D57485
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/GlobalISel/IRTranslator.cpp (diff)llvm.src/lib/CodeGen/GlobalISel/IRTranslator.cpp
The file was modified/llvm/trunk/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll (diff)llvm.src/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll
Revision 353719 by arsenm:
GlobalISel: Add G_FCANONICALIZE instruction
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/Support/TargetOpcodes.def (diff)llvm.src/include/llvm/Support/TargetOpcodes.def
The file was modified/llvm/trunk/include/llvm/Target/GenericOpcodes.td (diff)llvm.src/include/llvm/Target/GenericOpcodes.td
The file was modified/llvm/trunk/lib/CodeGen/GlobalISel/IRTranslator.cpp (diff)llvm.src/lib/CodeGen/GlobalISel/IRTranslator.cpp
The file was modified/llvm/trunk/lib/CodeGen/GlobalISel/LegalizerHelper.cpp (diff)llvm.src/lib/CodeGen/GlobalISel/LegalizerHelper.cpp
The file was modified/llvm/trunk/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp (diff)llvm.src/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
The file was modified/llvm/trunk/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll (diff)llvm.src/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll
The file was modified/llvm/trunk/test/CodeGen/AArch64/GlobalISel/legalizer-info-validation.mir (diff)llvm.src/test/CodeGen/AArch64/GlobalISel/legalizer-info-validation.mir
The file was added/llvm/trunk/test/CodeGen/AMDGPU/GlobalISel/legalize-fcanonicalize.mirllvm.src/test/CodeGen/AMDGPU/GlobalISel/legalize-fcanonicalize.mir
Revision 353718 by nico:
Make test actually test something (colons were missing)
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/test/CodeGen/ms-x86-intrinsics.c (diff)clang.src/test/CodeGen/ms-x86-intrinsics.c
Revision 353717 by vpykhtin:
[AMDGPU] fix atomic_optimizations_buffer.ll test after DPP combiner was enabled by default.

Related commits: rL353691, rL353703.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/CodeGen/AMDGPU/atomic_optimizations_buffer.ll (diff)llvm.src/test/CodeGen/AMDGPU/atomic_optimizations_buffer.ll
Revision 353716 by rksimon:
[X86] Regenerate insertelement tests

Add common X86/X64 prefixes (and use X86 instead of X32)
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/CodeGen/X86/insertelement-shuffle.ll (diff)llvm.src/test/CodeGen/X86/insertelement-shuffle.ll
Revision 353713 by greened:
Add recipes for migrating downstream branches of git mirrors

Add some common recipes for downstream users developing on top of the
existing git mirrors. These instructions show how to migrate local
branches to the monorepo.

Differential Revision: https://reviews.llvm.org/D56550
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/docs/Proposals/GitHubMove.rst (diff)llvm.src/docs/Proposals/GitHubMove.rst
Revision 353712 by hokein:
Revamp the "[clangd] Format tweak's replacements"

Summary:
This patch contains two parts:

1) reverts commit r353306.
2) move the format logic out from tweaks, keep tweaks API unchanged.

Reviewers: sammccall, ilya-biryukov

Reviewed By: ilya-biryukov

Subscribers: ilya-biryukov, ioeric, MaskRay, jkorous, arphaman, kadircet, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D58051
Change TypePath in RepositoryPath in Workspace
The file was modified/clang-tools-extra/trunk/clangd/ClangdServer.cpp (diff)clang-tools-extra.src/clangd/ClangdServer.cpp
The file was modified/clang-tools-extra/trunk/clangd/ClangdUnit.cpp (diff)clang-tools-extra.src/clangd/ClangdUnit.cpp
The file was modified/clang-tools-extra/trunk/clangd/Compiler.h (diff)clang-tools-extra.src/clangd/Compiler.h
The file was modified/clang-tools-extra/trunk/clangd/refactor/Tweak.cpp (diff)clang-tools-extra.src/clangd/refactor/Tweak.cpp
The file was modified/clang-tools-extra/trunk/clangd/refactor/Tweak.h (diff)clang-tools-extra.src/clangd/refactor/Tweak.h
The file was modified/clang-tools-extra/trunk/clangd/refactor/tweaks/SwapIfBranches.cpp (diff)clang-tools-extra.src/clangd/refactor/tweaks/SwapIfBranches.cpp
The file was added/clang-tools-extra/trunk/test/clangd/tweaks-format.testclang-tools-extra.src/test/clangd/tweaks-format.test
The file was modified/clang-tools-extra/trunk/unittests/clangd/TweakTests.cpp (diff)clang-tools-extra.src/unittests/clangd/TweakTests.cpp
Revision 353711 by alexfh:
Fixed header underline in docs.

+ Removed trailing whitespace.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/docs/analyzer/checkers.rst (diff)clang.src/docs/analyzer/checkers.rst
Revision 353710 by d0k:
Move some classes into anonymous namespaces. NFC.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
The file was modified/llvm/trunk/lib/DebugInfo/CodeView/CodeViewError.cpp (diff)llvm.src/lib/DebugInfo/CodeView/CodeViewError.cpp
The file was modified/llvm/trunk/lib/DebugInfo/MSF/MSFError.cpp (diff)llvm.src/lib/DebugInfo/MSF/MSFError.cpp
The file was modified/llvm/trunk/lib/DebugInfo/PDB/GenericError.cpp (diff)llvm.src/lib/DebugInfo/PDB/GenericError.cpp
The file was modified/llvm/trunk/lib/DebugInfo/PDB/Native/RawError.cpp (diff)llvm.src/lib/DebugInfo/PDB/Native/RawError.cpp
The file was modified/llvm/trunk/lib/Target/AArch64/AArch64InstructionSelector.cpp (diff)llvm.src/lib/Target/AArch64/AArch64InstructionSelector.cpp
The file was modified/llvm/trunk/lib/Target/WebAssembly/WebAssemblyFixFunctionBitcasts.cpp (diff)llvm.src/lib/Target/WebAssembly/WebAssemblyFixFunctionBitcasts.cpp
Revision 353708 by ioeric:
[clangd] Prefer location from codegen files when merging symbols.

Summary:
For example, if an index symbol has location in a .proto file and an AST symbol
has location in a generated .proto.h file, then we prefer location in .proto
which is more meaningful to users.

Also use `mergeSymbols` to get the preferred location between AST location and index location in go-to-def.

Reviewers: sammccall

Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, kadircet, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D58037
Change TypePath in RepositoryPath in Workspace
The file was modified/clang-tools-extra/trunk/clangd/XRefs.cpp (diff)clang-tools-extra.src/clangd/XRefs.cpp
The file was modified/clang-tools-extra/trunk/clangd/index/Merge.cpp (diff)clang-tools-extra.src/clangd/index/Merge.cpp
The file was modified/clang-tools-extra/trunk/unittests/clangd/IndexTests.cpp (diff)clang-tools-extra.src/unittests/clangd/IndexTests.cpp
The file was modified/clang-tools-extra/trunk/unittests/clangd/XRefsTests.cpp (diff)clang-tools-extra.src/unittests/clangd/XRefsTests.cpp
Revision 353707 by serge_sans_paille:
[tools] Fix python DeprecationWarning: invalid escape sequence

The python documentation says "it’s highly recommended that you use raw strings for all but the simplest expressions." (https://docs.python.org/3/library/re.html)

So do that with the attached patch generated by

    sed -i -e "s/re.search('/re.search(r'/g" $(git grep -l 're.search(')

The warning can be seen in e.g. python3.7:

    $ python3.7 -Wd
    >>> import re; re.search('\s', '')
    <stdin>:1: DeprecationWarning: invalid escape sequence \s

Commited on behalf of Marco Falke.
Differential Revision: https://reviews.llvm.org/D57528
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/tools/clang-format/clang-format-diff.py (diff)clang.src/tools/clang-format/clang-format-diff.py
The file was modified/cfe/trunk/tools/scan-build-py/libscanbuild/analyze.py (diff)clang.src/tools/scan-build-py/libscanbuild/analyze.py
The file was modified/cfe/trunk/tools/scan-build/bin/set-xcode-analyzer (diff)clang.src/tools/scan-build/bin/set-xcode-analyzer
Revision 353706 by adibiagio:
[MCA] Return a mask of busy resources from method ResourceManager::checkAvailability(). NFCI

In case of bottlenecks caused by pipeline pressure, we want to be able to
correctly report the set of problematic pipelines. This is a first step towards
adding support for bottleneck hints in llvm-mca (see PR37494). No functional
change intended.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/MCA/HardwareUnits/ResourceManager.h (diff)llvm.src/include/llvm/MCA/HardwareUnits/ResourceManager.h
The file was modified/llvm/trunk/include/llvm/MCA/HardwareUnits/Scheduler.h (diff)llvm.src/include/llvm/MCA/HardwareUnits/Scheduler.h
The file was modified/llvm/trunk/lib/MCA/HardwareUnits/ResourceManager.cpp (diff)llvm.src/lib/MCA/HardwareUnits/ResourceManager.cpp
The file was modified/llvm/trunk/lib/MCA/HardwareUnits/Scheduler.cpp (diff)llvm.src/lib/MCA/HardwareUnits/Scheduler.cpp
Revision 353705 by d0k:
Make some helper functions static. NFC.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/Sema/SemaCodeComplete.cpp (diff)clang.src/lib/Sema/SemaCodeComplete.cpp
The file was modified/cfe/trunk/lib/StaticAnalyzer/Checkers/RetainCountChecker/RetainCountDiagnostics.cpp (diff)clang.src/lib/StaticAnalyzer/Checkers/RetainCountChecker/RetainCountDiagnostics.cpp
Revision 353704 by d0k:
[AMDGPU] Remove unused variable
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/AMDGPU/AMDGPUAtomicOptimizer.cpp (diff)llvm.src/lib/Target/AMDGPU/AMDGPUAtomicOptimizer.cpp
Revision 353703 by sheredom:
[AMDGPU] Fix DPP sequence in atomic optimizer.

This commit fixes the DPP sequence in the atomic optimizer (which was
previously missing the row_shr:3 step), and works around a read_register
exec bug by using a ballot instead.

Differential Revision: https://reviews.llvm.org/D57737
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/AMDGPU/AMDGPUAtomicOptimizer.cpp (diff)llvm.src/lib/Target/AMDGPU/AMDGPUAtomicOptimizer.cpp
The file was modified/llvm/trunk/test/CodeGen/AMDGPU/atomic_optimizations_buffer.ll (diff)llvm.src/test/CodeGen/AMDGPU/atomic_optimizations_buffer.ll
The file was modified/llvm/trunk/test/CodeGen/AMDGPU/atomic_optimizations_global_pointer.ll (diff)llvm.src/test/CodeGen/AMDGPU/atomic_optimizations_global_pointer.ll
The file was modified/llvm/trunk/test/CodeGen/AMDGPU/atomic_optimizations_local_pointer.ll (diff)llvm.src/test/CodeGen/AMDGPU/atomic_optimizations_local_pointer.ll
The file was modified/llvm/trunk/test/CodeGen/AMDGPU/atomic_optimizations_pixelshader.ll (diff)llvm.src/test/CodeGen/AMDGPU/atomic_optimizations_pixelshader.ll
The file was modified/llvm/trunk/test/CodeGen/AMDGPU/atomic_optimizations_raw_buffer.ll (diff)llvm.src/test/CodeGen/AMDGPU/atomic_optimizations_raw_buffer.ll
The file was modified/llvm/trunk/test/CodeGen/AMDGPU/atomic_optimizations_struct_buffer.ll (diff)llvm.src/test/CodeGen/AMDGPU/atomic_optimizations_struct_buffer.ll
Revision 353699 by sammccall:
Revert "[X86][SSE] Generalize X86ISD::BLENDI support to more value types"

This reverts commit r353610.
It causes a miscompile visible in macro expansion in a bootstrapped clang.

http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20190211/626590.html
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (diff)llvm.src/lib/Target/X86/X86ISelLowering.cpp
The file was modified/llvm/trunk/lib/Target/X86/X86InstrSSE.td (diff)llvm.src/lib/Target/X86/X86InstrSSE.td
The file was modified/llvm/trunk/test/CodeGen/X86/avx512-shuffles/partial_permute.ll (diff)llvm.src/test/CodeGen/X86/avx512-shuffles/partial_permute.ll
The file was modified/llvm/trunk/test/CodeGen/X86/combine-sdiv.ll (diff)llvm.src/test/CodeGen/X86/combine-sdiv.ll
The file was modified/llvm/trunk/test/CodeGen/X86/insertelement-ones.ll (diff)llvm.src/test/CodeGen/X86/insertelement-ones.ll
The file was modified/llvm/trunk/test/CodeGen/X86/known-signbits-vector.ll (diff)llvm.src/test/CodeGen/X86/known-signbits-vector.ll
The file was modified/llvm/trunk/test/CodeGen/X86/masked_load.ll (diff)llvm.src/test/CodeGen/X86/masked_load.ll
The file was modified/llvm/trunk/test/CodeGen/X86/masked_store.ll (diff)llvm.src/test/CodeGen/X86/masked_store.ll
The file was modified/llvm/trunk/test/CodeGen/X86/oddshuffles.ll (diff)llvm.src/test/CodeGen/X86/oddshuffles.ll
The file was modified/llvm/trunk/test/CodeGen/X86/packss.ll (diff)llvm.src/test/CodeGen/X86/packss.ll
The file was modified/llvm/trunk/test/CodeGen/X86/pr34592.ll (diff)llvm.src/test/CodeGen/X86/pr34592.ll
The file was modified/llvm/trunk/test/CodeGen/X86/prefer-avx256-mask-shuffle.ll (diff)llvm.src/test/CodeGen/X86/prefer-avx256-mask-shuffle.ll
The file was modified/llvm/trunk/test/CodeGen/X86/sse2.ll (diff)llvm.src/test/CodeGen/X86/sse2.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-reduce-smax.ll (diff)llvm.src/test/CodeGen/X86/vector-reduce-smax.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-reduce-smin.ll (diff)llvm.src/test/CodeGen/X86/vector-reduce-smin.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-shift-ashr-256.ll (diff)llvm.src/test/CodeGen/X86/vector-shift-ashr-256.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-shuffle-128-v8.ll (diff)llvm.src/test/CodeGen/X86/vector-shuffle-128-v8.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-shuffle-256-v16.ll (diff)llvm.src/test/CodeGen/X86/vector-shuffle-256-v16.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-shuffle-256-v32.ll (diff)llvm.src/test/CodeGen/X86/vector-shuffle-256-v32.ll
Revision 353698 by szelethus:
[analyzer] New checker for detecting usages of unsafe I/O functions

There are certain unsafe or deprecated (since C11) buffer handling
functions which should be avoided in safety critical code. They
could cause buffer overflows. A new checker,
'security.insecureAPI.DeprecatedOrUnsafeBufferHandling' warns for
every occurrence of such functions (unsafe or deprecated printf,
scanf family, and other buffer handling functions, which now have
a secure variant).

Patch by Dániel Kolozsvári!

Differential Revision: https://reviews.llvm.org/D35068
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/docs/analyzer/checkers.rst (diff)clang.src/docs/analyzer/checkers.rst
The file was modified/cfe/trunk/include/clang/StaticAnalyzer/Checkers/Checkers.td (diff)clang.src/include/clang/StaticAnalyzer/Checkers/Checkers.td
The file was modified/cfe/trunk/lib/StaticAnalyzer/Checkers/CheckSecuritySyntaxOnly.cpp (diff)clang.src/lib/StaticAnalyzer/Checkers/CheckSecuritySyntaxOnly.cpp
The file was modified/cfe/trunk/test/Analysis/security-syntax-checks.m (diff)clang.src/test/Analysis/security-syntax-checks.m
Revision 353697 by maskray:
Format isInSystemMacro after D55782
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/include/clang/Basic/SourceManager.h (diff)clang.src/include/clang/Basic/SourceManager.h
Revision 353696 by kadircet:
[clangd] Index parameters in function decls

Reviewers: hokein

Subscribers: ilya-biryukov, ioeric, MaskRay, jkorous, arphaman, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D57950
Change TypePath in RepositoryPath in Workspace
The file was modified/clang-tools-extra/trunk/clangd/XRefs.cpp (diff)clang-tools-extra.src/clangd/XRefs.cpp
The file was modified/clang-tools-extra/trunk/unittests/clangd/SymbolInfoTests.cpp (diff)clang-tools-extra.src/unittests/clangd/SymbolInfoTests.cpp
The file was modified/clang-tools-extra/trunk/unittests/clangd/XRefsTests.cpp (diff)clang-tools-extra.src/unittests/clangd/XRefsTests.cpp
Revision 353695 by kadircet:
[clang][Index] Add a knob to index function parameters in declarations

Summary:
Parameters in declarations are useful for clangd, so that we can
provide symbol information for them as well. It also helps clangd to be
consistent whether a function's definition is accessible or not.

Reviewers: hokein, akyrtzi

Subscribers: ilya-biryukov, ioeric, arphaman, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D57949
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/include/clang/Index/IndexingAction.h (diff)clang.src/include/clang/Index/IndexingAction.h
The file was modified/cfe/trunk/lib/Index/IndexDecl.cpp (diff)clang.src/lib/Index/IndexDecl.cpp
The file was modified/cfe/trunk/lib/Index/IndexingContext.cpp (diff)clang.src/lib/Index/IndexingContext.cpp
The file was modified/cfe/trunk/lib/Index/IndexingContext.h (diff)clang.src/lib/Index/IndexingContext.h
The file was modified/cfe/trunk/unittests/Index/IndexTests.cpp (diff)clang.src/unittests/Index/IndexTests.cpp
Revision 353694 by kadircet:
[clangd] Fix broken windows build bots.
Change TypePath in RepositoryPath in Workspace
The file was modified/clang-tools-extra/trunk/unittests/clangd/FileIndexTests.cpp (diff)clang-tools-extra.src/unittests/clangd/FileIndexTests.cpp
Revision 353693 by mkazantsev:
[TEST] Add missing opportunity test for PR39673
Change TypePath in RepositoryPath in Workspace
The file was added/llvm/trunk/test/Transforms/IndVarSimplify/pr39673.llllvm.src/test/Transforms/IndVarSimplify/pr39673.ll
Revision 353692 by sam_parker:
[ARM] Add v8m.base pattern for add negative imm

The v8m.base ISA contains movw, which can operate on an unsigned
16-bit value. Add the pattern that converts an add with a negative
value, that could fit into 16-bits when negated, into a sub with that
positive value.

Differential Revision: https://reviews.llvm.org/D57942
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/ARM/ARMInstrThumb2.td (diff)llvm.src/lib/Target/ARM/ARMInstrThumb2.td
The file was modified/llvm/trunk/test/CodeGen/ARM/sub.ll (diff)llvm.src/test/CodeGen/ARM/sub.ll
Revision 353691 by vpykhtin:
[AMDGPU] Enable DPP combiner pass by default.

Related revisions: https://reviews.llvm.org/D55444, https://reviews.llvm.org/D55314
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp (diff)llvm.src/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
Revision 353690 by svenvh:
[libclang] Add attribute support for 'convergent'.

This bumps CINDEX_VERSION_MINOR up (to 51).

Patch by Hsin-Hsiao Lin.

Differential Revision: https://reviews.llvm.org/D57946
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/bindings/python/clang/cindex.py (diff)clang.src/bindings/python/clang/cindex.py
The file was modified/cfe/trunk/include/clang-c/Index.h (diff)clang.src/include/clang-c/Index.h
The file was modified/cfe/trunk/test/Index/attributes.c (diff)clang.src/test/Index/attributes.c
The file was modified/cfe/trunk/tools/libclang/CIndex.cpp (diff)clang.src/tools/libclang/CIndex.cpp
The file was modified/cfe/trunk/tools/libclang/CXCursor.cpp (diff)clang.src/tools/libclang/CXCursor.cpp
Revision 353689 by sam_parker:
[NFC][ARM] Simplify loop-indexing codegen test

Remove unnecessary offset checks, CHECK-BASE checks and add some
extra -NOT checks and TODO comments.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/CodeGen/ARM/loop-indexing.ll (diff)llvm.src/test/CodeGen/ARM/loop-indexing.ll