| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
llvm-svn: 296595
|
|
|
|
| |
llvm-svn: 296594
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Summary:
The test runner has code to autodetect this, but it's not very smart --
in particular, it fails in the case where we build the test executables
with the just-built clang. Since cmake already has the knowledge about
the right toolchain, we can just have it pass the appropriate flags to
the test runner.
This also removes the "temporary" cache-scrubbing hack added a couple
months ago.
Reviewers: zturner, beanz
Subscribers: mgorny, lldb-commits
Differential Revision: https://reviews.llvm.org/D30453
llvm-svn: 296593
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Summary:
Use StringRef and ArrayRef where possible. This adds an accessor to the
Args class to get a view of the arguments as ArrayRef<const char *>.
Reviewers: zturner
Subscribers: lldb-commits
Differential Revision: https://reviews.llvm.org/D30402
llvm-svn: 296592
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Windows does not treat `~` as a reference to home directory, so the call
to `llvm::sys::path::native` on, say, `~/somedir` produces `~\somedir`,
which has different meaning than the original path. With this change
tilde is expanded on Windows to user profile directory. Such behavior
keeps original meaning of the path and is consistent with the algorithm
of `llvm::sys::path::home_directory`.
Differential Revision: https://reviews.llvm.org/D27527
llvm-svn: 296590
|
|
|
|
| |
llvm-svn: 296586
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Summary:
Solves PR 31990.
The bad rewrite could replace a memcpy of one word with
store i4 -1
while it should actually be
store i8 -1
Hopefully opt and llc has improved enough so the original optimization
done by the code isn't needed anymore.
One already existing testcase is affected. It originally tested that
the memcpy was replaced with
load double
but since we now remove that rewrite it will be
load i64
instead.
Patch suggestion by Eli Friedman.
Reviewers: eli.friedman, majnemer, efriedma
Reviewed By: efriedma
Subscribers: efriedma, llvm-commits
Differential Revision: https://reviews.llvm.org/D30254
llvm-svn: 296585
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
potential capture list.
Fix Sema::getCurLambda() to return the innermost lambda scope when there
is a block enclosed in the lambda. Previously, the method would return a
nullptr in such cases, which would prevent a variable captured by the
enclosed block to be added to the lambda scope's potential capture list.
rdar://problem/28412462
Differential Revision: https://reviews.llvm.org/D25556
llvm-svn: 296584
|
|
|
|
|
|
|
|
|
|
|
|
| |
std::errc::no_buffer_space.
Unfortunately, mingw's libstdc++ doesn't provide winsock2 errors.
That said, we should avoid raising OS-oriented error code in our code.
For now, I suggest to define custom error from std::error_category.
See also; https://reviews.llvm.org/D20592
llvm-svn: 296583
|
|
|
|
|
|
|
|
| |
std::errc::no_buffer_space."
Wrong commit -- I have unstaged changes.
llvm-svn: 296582
|
|
|
|
|
|
|
|
|
|
| |
Unfortunately, mingw's libstdc++ doesn't provide winsock2 errors.
That said, we should avoid raising OS-oriented error code in our code.
For now, I suggest to define custom error from std::error_category.
See also; https://reviews.llvm.org/D20592
llvm-svn: 296581
|
|
|
|
|
|
|
| |
Thanks to Rafael for helping to dig down into what we're really trying
to communicate here.
llvm-svn: 296580
|
|
|
|
|
|
|
| |
We mark it as XFAIL to get buildbots back to green, until the upstream changes
have been addressed.
llvm-svn: 296579
|
|
|
|
|
|
|
|
| |
The practice in LV is that we emit analysis remarks and then finally report
either a missed or applied remark on the final decision whether vectorization
is taking place. On this code path, we were closing with an analysis remark.
llvm-svn: 296578
|
|
|
|
|
|
|
| |
Each input section knows its sh_entsize value, so we can set output
section's sh_entsize based on input sections values.
llvm-svn: 296577
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use the libc++abi visibility macros instead of pragmas or using
visibility attributes directly. Clean up redundant attributes on
definitions (where the declarations already have visibility attributes
applied, from either libc++ or libc++abi headers).
Introduce _LIBCXXABI_WEAK as a drive-by cleanup, which matches the
semantics of _LIBCPP_WEAK.
No functional change. Tested by building on Linux before and after this
change and verifying that the list of exported symbols is identical.
Differential Revision: https://reviews.llvm.org/D26949
llvm-svn: 296576
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
to be available
for VectorizeTree() API.This API uses it for proper mask computation to be used in shufflevector IR.
The fix is to compute the mask for out of order memory accesses while building the vectorizable tree
instead of actual vectorization of vectorizable tree.
Reviewers: mkuper
Differential Revision: https://reviews.llvm.org/D30159
Change-Id: Id1e287f073fa4959713ba545fa4254db5da8b40d
llvm-svn: 296575
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Modify the test so that it is still testing something
closer to what it was intended to originally.
I think the original intent was to test the situation where
there was a branch on execz and then unconditional branch
required relaxing.With the change in r296539,
there was no longer and execz branch.
Change the test so that there is now an execz branch inserted.
There is no longer an unconditional branch after the execz branch,
so this might need to be tricked in some other way to keep that
there.
llvm-svn: 296574
|
|
|
|
|
|
| |
Differential Revision: https://reviews.llvm.org/D30278
llvm-svn: 296573
|
|
|
|
|
|
|
|
|
| |
The exisiting warning for inconsistent overrides does not include the destructor
as it was noted in review that it was too noisy. Instead, add to a separate
warning group that is off by default for users who want consistent warnings
between methods and destructors.
llvm-svn: 296572
|
|
|
|
| |
llvm-svn: 296571
|
|
|
|
|
|
|
|
| |
This implementation is probably slightly inefficient than before,
but that should be negligible because this is not a performance-
critical pass.
llvm-svn: 296570
|
|
|
|
|
|
| |
makes it look like their is a bug in the loop indentation, and there is not
llvm-svn: 296569
|
|
|
|
| |
llvm-svn: 296568
|
|
|
|
|
|
| |
This patch adds the required leading underscore to those macros.
llvm-svn: 296567
|
|
|
|
|
|
|
|
|
| |
This patch adds a static assertion that the specified hash meets
the requirements of an enabled hash, and it ensures that the static
assertion is evaluated before __compressed_pair is instantiated.
That way the static assertion diagnostic is emitted first.
llvm-svn: 296565
|
|
|
|
| |
llvm-svn: 296564
|
|
|
|
|
|
|
|
|
|
|
|
| |
Summary: Fix issue reported where intrinsic calling convention is dropped after r295253.
Reviewers: sanjoy
Subscribers: materi, llvm-commits
Differential Revision: https://reviews.llvm.org/D30422
llvm-svn: 296563
|
|
|
|
|
|
|
|
|
|
|
| |
Fix a crash in the ObjCPropertyChecker when analyzing a 'copy' property of an
NSMutable* type in a protocol.
rdar://problem/30766684
Differential Revision: https://reviews.llvm.org/D30482
llvm-svn: 296562
|
|
|
|
|
|
|
|
|
|
| |
This patch fixes llvm.org/PR32097 by using the __is_abstract
builtin type-trait instead of the previous library-only implementation.
All supported compilers provide this trait. I've tested as far
back as Clang 3.2, GCC 4.6 and MSVC trunk.
llvm-svn: 296561
|
|
|
|
|
|
| |
GCC Linker doesn't seem to like this.
llvm-svn: 296560
|
|
|
|
| |
llvm-svn: 296559
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Summary:
`ConstexprTestTypes::NoCtors` is an aggregate type (and consequently a literal type) in C++17,
but not in C++14 since it has a base class. This patch updates the comment to accurately describe the reason for the XFAIL.
Reviewers: EricWF
Reviewed By: EricWF
Differential Revision: https://reviews.llvm.org/D30481
llvm-svn: 296558
|
|
|
|
| |
llvm-svn: 296557
|
|
|
|
|
|
|
|
|
|
|
| |
A bug was uncovered where if you have a StreamRef whose ViewOffset
is > 0, then when you call readLongestContiguousChunk it will
succeed even when it shouldn't, and it always return you a
buffer that was taken as if the ViewOffset was 0.
Fixed this bug and added a test for it.
llvm-svn: 296556
|
|
|
|
| |
llvm-svn: 296555
|
|
|
|
|
|
|
| |
Reapplies r296209 now that r296549 has fixed what really seems to be
the last problematic test.
llvm-svn: 296554
|
|
|
|
| |
llvm-svn: 296553
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When SDAGISel (top-down) selects a tail-call, it skips the remainder
of the block.
If, before that, FastISel (bottom-up) selected some of the (no-op) next
few instructions, we can end up with dead instructions following the
terminator (selected by SDAGISel).
We need to erase them, as we know they aren't necessary (in addition to
being incorrect).
We already do this when FastISel falls back on the tail-call itself.
Also remove the FastISel-emitted code if we fallback on the
instructions between the tail-call and the return.
llvm-svn: 296552
|
|
|
|
|
|
|
|
|
|
|
| |
Iterating on the use-list we're modifying doesn't work: after the first
iteration, the use-list iterator will point to a MachineOperand
referencing the new register. This caused us to skip the other uses to
replace.
Instead, use MRI.replaceRegWith(), which accounts for this behavior.
llvm-svn: 296551
|
|
|
|
| |
llvm-svn: 296550
|
|
|
|
|
|
| |
Differential Revision: https://reviews.llvm.org/D30430
llvm-svn: 296549
|
|
|
|
|
|
| |
review patches etc from his clayborg email address.
llvm-svn: 296548
|
|
|
|
|
|
|
| |
The asm-printer now emits remarks with function location which have
unspecified (0) source column number.
llvm-svn: 296547
|
|
|
|
|
|
|
|
|
|
| |
remarks"
This reverts commit r296544.
This got committed by accident.
llvm-svn: 296546
|
|
|
|
| |
llvm-svn: 296544
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Conflicting debug info for function arguments causes hard-to-debug
assertions in the DWARF backend, so the Verifier should reject it.
For performance reasons this only checks function arguments from
non-inlined debug intrinsics for now.
rdar://problem/30520286
This reapplies r295749 after fixing PR32042.
llvm-svn: 296543
|
|
|
|
|
|
| |
Differential Revision: https://reviews.llvm.org/D29445
llvm-svn: 296542
|
|
|
|
|
|
|
|
|
| |
This is for running the assembler with -g (to emit DWARF describing
the assembler source).
Differential Revision: http://reviews.llvm.org/D30475
llvm-svn: 296541
|
|
|
|
|
|
|
|
|
|
|
| |
target.
To facilitate this, add a new hidden command-line option to disable
the explicit-locals pass. That causes llc to emit invalid code that doesn't
have all locals converted to get_local/set_local, however it simplifies
testwriting in many cases.
llvm-svn: 296540
|