Commit
221e418f0c56d6b4a8b188fb4d8e75e21f7b0c8d
by jdenny.ornl[lit] Make internal diff work in pipelines When using lit's internal shell, RUN lines like the following accidentally execute an external `diff` instead of lit's internal `diff`: ``` # RUN: program | diff file - # RUN: not diff file1 file2 | FileCheck %s ``` Such cases exist now, in `clang/test/Analysis` for example. We are preparing patches to ensure lit's internal `diff` is called in such cases, which will then fail because lit's internal `diff` cannot currently be used in pipelines and doesn't recognize `-` as a command-line option. To enable pipelines, this patch moves lit's `diff` implementation into an out-of-process script, similar to lit's `cat` implementation. A follow-up patch will implement `-` to mean stdin. Reviewed By: probinson, stella.stamenova Differential Revision: https://reviews.llvm.org/D66574 llvm-svn: 375114
|
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-pipes.txt |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-error-0.txt |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-unified-error-1.txt |
 | llvm/utils/lit/tests/shtest-shell.py |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-unified-error-0.txt |
 | llvm/utils/lit/tests/max-failures.py |
 | llvm/utils/lit/lit/builtin_commands/diff.py |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-unified.txt |
 | llvm/utils/lit/lit/TestRunner.py |
Commit
849d67a70080982cc1261759015a67b06b25eb16
by jdenny.ornl[lit] Extend internal diff to support `-` argument When using lit's internal shell, RUN lines like the following accidentally execute an external `diff` instead of lit's internal `diff`: ``` # RUN: program | diff file - ``` Such cases exist now, in `clang/test/Analysis` for example. We are preparing patches to ensure lit's internal `diff` is called in such cases, which will then fail because lit's internal `diff` doesn't recognize `-` as a command-line option. This patch adds support for `-` to mean stdin. Reviewed By: probinson, rnk Differential Revision: https://reviews.llvm.org/D67643 llvm-svn: 375116
|
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-pipes.txt |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-encodings.txt |
 | llvm/utils/lit/tests/shtest-shell.py |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-r-error-7.txt |
 | llvm/utils/lit/lit/builtin_commands/diff.py |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-r-error-8.txt |
 | llvm/utils/lit/tests/max-failures.py |
Commit
c8e3f43ab514bf527b2d2d01f66d4c921a2985c4
by hokein[clangd] Use our own relation kind. Summary: Move the RelationKind from Serialization.h to Relation.h. This patch doesn't introduce any breaking changes. Reviewers: kadircet Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, usaxena95, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D68981 llvm-svn: 375117
|
 | clang-tools-extra/clangd/index/Index.h |
 | clang-tools-extra/clangd/index/YAMLSerialization.cpp |
 | clang-tools-extra/clangd/index/Serialization.cpp |
 | clang-tools-extra/clangd/unittests/DexTests.cpp |
 | clang-tools-extra/clangd/unittests/SerializationTests.cpp |
 | clang-tools-extra/clangd/unittests/IndexTests.cpp |
 | clang-tools-extra/clangd/unittests/BackgroundIndexTests.cpp |
 | clang-tools-extra/clangd/index/dex/Dex.h |
 | clang-tools-extra/clangd/index/Relation.h |
 | llvm/include/llvm/ADT/DenseMapInfo.h |
 | clang-tools-extra/clangd/index/MemIndex.h |
 | clang-tools-extra/clangd/index/Relation.cpp |
 | clang-tools-extra/clangd/unittests/SymbolCollectorTests.cpp |
 | clang-tools-extra/clangd/index/Serialization.h |
 | clang-tools-extra/clangd/XRefs.cpp |
 | clang-tools-extra/clangd/unittests/TypeHierarchyTests.cpp |
 | clang-tools-extra/clangd/index/SymbolCollector.cpp |
 | clang-tools-extra/clangd/index/MemIndex.cpp |
 | clang-tools-extra/clangd/unittests/FileIndexTests.cpp |
 | clang-tools-extra/clangd/index/dex/Dex.cpp |
Commit
af6248cbb9e70122359a12ff16541fe510ada600
by sven.vanhaastregt[OpenCL] Preserve addrspace in CGClass (PR43145) PR43145 revealed two places where Clang was attempting to create a bitcast without considering the address space of class types during C++ class code generation. Differential Revision: https://reviews.llvm.org/D68403 llvm-svn: 375118
|
 | clang/test/CodeGenOpenCLCXX/addrspace-derived-base.cl |
 | clang/lib/CodeGen/CGClass.cpp |
Commit
3e0f4f8e928cabb3da9fc955321fb5142a0a0110
by a.bataev[OPENMP]Fix thread id passed to outlined region in sequential parallel regions. The real global thread id must be passed to the outlined region instead of the zero thread id. llvm-svn: 375119
|
 | clang/test/OpenMP/parallel_if_codegen.cpp |
 | clang/lib/CodeGen/CGOpenMPRuntime.cpp |
Commit
5e684e8d88ba60c867b12dbde274db0730d7b517
by jdenny.ornlRevert r375116: "[lit] Extend internal diff to support `-` argument" This series of patches still breaks a Windows bot. llvm-svn: 375120
|
 | llvm/utils/lit/tests/max-failures.py |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-encodings.txt |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-pipes.txt |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-r-error-8.txt |
 | llvm/utils/lit/lit/builtin_commands/diff.py |
 | llvm/utils/lit/tests/shtest-shell.py |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-r-error-7.txt |
Commit
e96e2d322723f45b052dc66e17d556e7346d4336
by jdenny.ornlRevert r375114: "[lit] Make internal diff work in pipelines" This series of patches still breaks a Windows bot. llvm-svn: 375121
|
 | llvm/utils/lit/lit/TestRunner.py |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-error-0.txt |
 | llvm/utils/lit/lit/builtin_commands/diff.py |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-pipes.txt |
 | llvm/utils/lit/tests/max-failures.py |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-unified-error-1.txt |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-unified.txt |
 | llvm/utils/lit/tests/Inputs/shtest-shell/diff-unified-error-0.txt |
 | llvm/utils/lit/tests/shtest-shell.py |
Commit
faf6b2543e4789061cbced2557f6bc7d1bd0d71d
by Tatyana Krasnukha[ARC] Basic support in gdb-remote process plugin Differential Revision: https://reviews.llvm.org/D55718 llvm-svn: 375122
|
 | lldb/source/Target/Thread.cpp |
 | lldb/include/lldb/Utility/ArchSpec.h |
 | lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp |
 | lldb/source/Target/Platform.cpp |
 | lldb/source/Utility/ArchSpec.cpp |
Commit
92e498d58cf47f4c1266cedba981a0c2cee171d0
by Tatyana Krasnukha[ARC] Add SystemV ABI Differential Revision: https://reviews.llvm.org/D55724 llvm-svn: 375123
|
 | lldb/source/Plugins/ABI/CMakeLists.txt |
 | lldb/source/Plugins/ABI/SysV-arc/CMakeLists.txt |
 | lldb/source/Plugins/ABI/SysV-arc/ABISysV_arc.h |
 | lldb/source/API/SystemInitializerFull.cpp |
 | lldb/source/Plugins/ABI/SysV-arc/ABISysV_arc.cpp |
Commit
1c982af0599781bdb049f898a2d512656c807485
by jyknight[ObjC] Add some additional test cases around pointer conversions. This is especially important for Objective-C++, which is entirely missing this testing at the moment. This annotates with "FIXME" the cases which I change in the next patch -- I primarily wanted to document the current state of things so that the effect of the code change is made clear. Differential Revision: https://reviews.llvm.org/D67982 llvm-svn: 375124
|
 | clang/test/SemaObjCXX/comptypes-7.mm |
 | clang/test/SemaObjCXX/class-method-self.mm |
 | clang/test/SemaObjCXX/comptypes-1.mm |
 | clang/test/SemaObjC/comptypes-7.m |
 | clang/test/SemaObjCXX/instancetype.mm |
 | clang/test/SemaObjC/class-method-self.m |
 | clang/test/SemaObjC/comptypes-1.m |
Commit
ccc4d83cda16bea1d9dfd0967dc7d2cfb24b8e75
by jyknight[ObjC] Diagnose implicit type coercion from ObjC 'Class' to object pointer types. For example, in Objective-C mode, the initialization of 'x' in: ``` @implementation MyType + (void)someClassMethod { MyType *x = self; } @end ``` is correctly diagnosed with an incompatible-pointer-types warning, but in Objective-C++ mode, it is not diagnosed at all -- even though incompatible pointer conversions generally become an error in C++. This patch fixes that oversight, allowing implicit conversions involving Class only to/from unqualified-id, and between qualified and unqualified Class, where the protocols are compatible. Note that this does change some behaviors in Objective-C, as well, as shown by the modified tests. Of particular note is that assignment from from 'Class<MyProtocol>' to 'id<MyProtocol>' now warns. (Despite appearances, those are not compatible types. 'Class<MyProtocol>' is not expected to have instance methods defined by 'MyProtocol', while 'id<MyProtocol>' is.) Differential Revision: https://reviews.llvm.org/D67983 llvm-svn: 375125
|
 | clang/lib/Sema/SemaExpr.cpp |
 | clang/test/SemaObjCXX/class-method-self.mm |
 | clang/test/SemaObjCXX/comptypes-7.mm |
 | clang/test/SemaObjCXX/comptypes-1.mm |
 | clang/test/SemaObjCXX/instancetype.mm |
 | clang/test/SemaObjC/comptypes-1.m |
 | clang/lib/AST/ASTContext.cpp |
Commit
1b6c3ca1267218b31c230815e1cc0b56aa89ddd2
by rupprecht[lld][test] Fix use of escape character in an lld test on Windows Summary: Glob support was improved to accept `\` as an escape character in r375051, but reverted as r375052 due to a failure in this test on Windows. The reason this failure seems Windows specific is because the path separator `\` is currently being relied on to be interpreted literally instead of as an escape character. Per documentation on linker input section wildcard patterns, this seems to be a bug in lld accepting `\` as a literal instead of an escape character. For example: ``` SECTIONS{ .foo :{ /path/to/foo.o(.foo) }} # OK: standard UNIX path SECTIONS{ .foo :{ C:/path/to/foo.o(.foo) }} # OK: windows accepts slashes in either direction SECTIONS{ .foo :{ C:\\path\\to\\foo.o(.foo) }} # OK: escape character used to match a literal \ SECTIONS{ .foo :{ C:\path\to\foo.o(.foo) }} # BAD: this actually matches the path C:pathtofoo.o(.foo) ``` This avoids the problem in the test by using `%/T` in place of `%T` to normalize the path separator to `/`, which windows should also accept. This patch just fixes the test, and glob support will be be relanded separately. For a sample buildbot error, see: http://lab.llvm.org:8011/builders/clang-x64-windows-msvc/builds/11578/steps/stage%201%20check/logs/stdio Reviewers: evgeny777, ruiu, MaskRay, espindola Reviewed By: ruiu, MaskRay Subscribers: emaste, arichardson, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D69074 llvm-svn: 375126
|
 | lld/test/ELF/linkerscript/filename-spec.s |
Commit
d06a2f3ad681fc9e2126b39f1110a82cd7870356
by Adrian PrantlFix an inverted condition in test. llvm-svn: 375127
|
 | lldb/packages/Python/lldbsuite/test/tools/lldb-server/gdbremote_testcase.py |
Commit
d25c766aa255662a91465662fef02ef8b5990bba
by jlettner[lit] Create derived classes for serial/parallel test runs The hope is that with a little OO we can nicely factor out the differences. llvm-svn: 375128
|
 | llvm/utils/lit/lit/run.py |
 | llvm/utils/lit/lit/main.py |
Commit
aa05e0e9729fe7236c8961c27092b780face8a85
by jlettner[lit] Synthesize artificial deadline We always want to use a deadline when calling `result.await`. Let's synthesize an artificial deadline (positive infinity) to simplify code and do less busy waiting. llvm-svn: 375129
|
 | llvm/utils/lit/lit/run.py |
Commit
a660dc590a5e8dafa1ba6ed56447ede151d17bd9
by jlettner[lit] Move computation of deadline up into base class llvm-svn: 375130
|
 | llvm/utils/lit/lit/run.py |