| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
|
| |
dereferenced the pointer at this point, and these routines are
exclusively called after the parser encounters a code completion token.
Other code completion routines called at that point do not check for
null either, so this is clearly the current invariant expected in the
code.
This fixes another PVS-Studio found issue.
llvm-svn: 285980
|
| |
|
|
|
|
|
| |
Output generated by option '-ast-print' must not contains enum
base type specifications if source language does not include C++11.
llvm-svn: 285979
|
| |
|
|
| |
llvm-svn: 285978
|
| |
|
|
|
|
| |
of clang.
llvm-svn: 285977
|
| |
|
|
| |
llvm-svn: 285976
|
| |
|
|
| |
llvm-svn: 285975
|
| |
|
|
|
|
|
|
|
|
| |
the process exists.
I also added some tests that crash before this fix, and work correctly after.
<rdar://problem/29083321>
llvm-svn: 285974
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Summary:
Instead of keeping track of TODOs for lit in a file checked into source
control, use LLVM's bug tracker. The TODOs have been migrated to the
following bugs:
* https://llvm.org/bugs/show_bug.cgi?id=30666
* https://llvm.org/bugs/show_bug.cgi?id=30667
* https://llvm.org/bugs/show_bug.cgi?id=30668
* https://llvm.org/bugs/show_bug.cgi?id=30669
* https://llvm.org/bugs/show_bug.cgi?id=30670
* https://llvm.org/bugs/show_bug.cgi?id=30671
Reviewers: ddunbar, beanz, echristo, delcypher
Subscribers: llvm-commits, mehdi_amini
Differential Revision: https://reviews.llvm.org/D25496
llvm-svn: 285973
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
behind the test that the MachineModuleInfo analysis was
actually available and can be used.
While the MachO bits may well be reasonable to assume in the darwin
assembly printer, the analysis isn't constructively guaranteed anywhere
I could find so it seems safest to avoid crashing here.
This issue was found with PVS-Studio. Pretty sure the Clang Static
Anaylzer flags similar issues but we've probably never pointed it at
this code effectively.
llvm-svn: 285972
|
| |
|
|
| |
llvm-svn: 285971
|
| |
|
|
|
|
|
|
|
| |
This fixes a mismatch between the declared error_type and the type used with
the placement new that initializes the field.
Patch by Yichao Yu.
llvm-svn: 285970
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Summary: ARMv6m supports dmb etc fench instructions but not ldrex/strex etc. So for some atomic load/store, LLVM should inline instructions instead of lowering to __sync_ calls.
Reviewers: rengolin, efriedma, t.p.northover, jmolloy
Subscribers: efriedma, aemerson, llvm-commits
Differential Revision: https://reviews.llvm.org/D26120
llvm-svn: 285969
|
| |
|
|
|
|
|
| |
If multi-threading is disabled, parallel_for_each will automatically
fall back to std::for_each, so we don't have to do that ourselves.
llvm-svn: 285968
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
in llvm-objdump for Mach-O files add the printing of the
ARM_THREAD_STATE64 in the same format as
otool-classic(1) on darwin.
To do this the 64-bit ARM general tread state
needed to be defined in include/llvm/Support/MachO.h .
rdar://28985800
llvm-svn: 285967
|
| |
|
|
| |
llvm-svn: 285966
|
| |
|
|
|
|
| |
This avoids duplicating the buffer in InputFile.
llvm-svn: 285965
|
| |
|
|
|
|
| |
Delete an empty line at the end of README.txt file.
llvm-svn: 285964
|
| |
|
|
| |
llvm-svn: 285962
|
| |
|
|
|
|
|
|
|
|
|
|
| |
All error checking now happens when the information is needed. The
only thing left is the minimum size of the buffer and that can be just
a precondition. I will add an ErrorOr create method in a followup
commit.
Also don't store a pointer to the Header, since it is just a trivial
cast.
llvm-svn: 285961
|
| |
|
|
|
|
| |
<rdar://problem/18616046>
llvm-svn: 285960
|
| |
|
|
|
|
|
|
|
| |
This implements the DWARF 5 DW_AT_export_symbols feature:
http://dwarfstd.org/ShowIssue.php?issue=141212.1
<rdar://problem/18616046>
llvm-svn: 285959
|
| |
|
|
| |
llvm-svn: 285958
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Summary:
These functions currently require that the new closed interval has a length of
at least 2. They also currently permit empty half-open intervals. This patch
defines nonEmpty in each traits structure and uses it to correct the
implementations of setStart and setStop.
Reviewers: stoklund, chandlerc
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D26064
llvm-svn: 285957
|
| |
|
|
| |
llvm-svn: 285956
|
| |
|
|
| |
llvm-svn: 285955
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
* if the base is produced by a series of derived-to-base conversions, check
the expression inside them when looking for an expression with a known
dynamic type
* step past MaterializeTemporaryExprs when checking for a known dynamic type
* when checking for a known dynamic type, treat all class prvalues as having
a known dynamic type after skipping all relevant rvalue subobject
adjustments
* treat callees formed by pointer-to-member access for a non-reference member
type like callees formed by member access.
llvm-svn: 285954
|
| |
|
|
|
|
|
| |
The buffer is already owned by the PDBFile for all of these APIs, so
don't pass it in separately.
llvm-svn: 285953
|
| |
|
|
| |
llvm-svn: 285952
|
| |
|
|
|
|
| |
Issue found by inspection.
llvm-svn: 285951
|
| |
|
|
|
|
|
|
|
|
|
|
| |
code, let's just assert that the DiagonsticEngine doesn't own the client
because our constructor took ownership of it and has a std::unique_ptr
that handles deleting it. This seems much more clear -- the release was
harmless but confusing as if there were some memory there it would have
leaked, and the reset was harmless but confusing as if there were some
memory there it would have been double-freed. But in both cases there
was nothing there.
llvm-svn: 285950
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Previously, it didn't support the character class, so we couldn't
eliminate the use fo llvm::Regex. Now that it is supported, we
can remove compileGlobPattern, which converts a glob pattern to
a regex.
This patch contains optimization for exact/prefix/suffix matches.
Differential Revision: https://reviews.llvm.org/D26284
llvm-svn: 285949
|
| |
|
|
|
|
|
|
| |
This file is unused as of r285939, but we need to keep it around
for bots that don't do full rebuilds. We should be able to delete this
again in a few days.
llvm-svn: 285948
|
| |
|
|
| |
llvm-svn: 285947
|
| |
|
|
|
|
|
|
| |
that its result is in fact used. Instead, use reset.
This was pointed out by PVS-Studio.
llvm-svn: 285946
|
| |
|
|
|
|
|
| |
Filed http://llvm.org/PR30897 to teach Clang to warn on this kind of
stuff.
llvm-svn: 285945
|
| |
|
|
| |
llvm-svn: 285944
|
| |
|
|
| |
llvm-svn: 285943
|
| |
|
|
| |
llvm-svn: 285942
|
| |
|
|
| |
llvm-svn: 285941
|
| |
|
|
| |
llvm-svn: 285940
|
| |
|
|
|
|
|
|
| |
Patch By: Wei Ding
Differential Revision: https://reviews.llvm.org/D18049
llvm-svn: 285939
|
| |
|
|
| |
llvm-svn: 285938
|
| |
|
|
|
|
|
|
|
|
|
| |
llvm-readobj.
Another bug caught by PVS-Studio.
It'd be nice to actually have a test for this, but I found it by
inspection from PVS-Studio.
llvm-svn: 285937
|
| |
|
|
|
|
|
|
| |
Quite sad we still aren't really using aggressive dead code warnings
from Clang that we could potentially use to catch this and so many other
things.
llvm-svn: 285936
|
| |
|
|
| |
llvm-svn: 285935
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Summary:
During clang-format source lexing >> and << operators are split and
treated as two less/greater operators but column position of following
tokens was not adjusted accordingly.
Fixes PR26887
Patch by Paweł Żukowski.
Reviewers: djasper
Subscribers: malcolm.parsons, mprobst, klimek, cfe-commits
Differential Revision: https://reviews.llvm.org/D25439
llvm-svn: 285934
|
| |
|
|
|
|
|
|
|
| |
Instead of remembering a raw Elf_Shdr, store the symbol table proper
and the index of the first non local.
This moves error handling upfront and simplifies it.
llvm-svn: 285933
|
| |
|
|
|
|
|
| |
This adds versions of getSectionIndex, getSection and getSymbol that
instead of a Elf_Shdr take the content of that section.
llvm-svn: 285932
|
| |
|
|
| |
llvm-svn: 285931
|
| |
|
|
|
|
|
|
|
|
|
| |
This condition is trivially always true prior to the change. The comment
at the call site makes it clear that we expect *all* of these to be '=',
'S', or 'I' so fix the code.
We have a bug I will update to track the fact that Clang doesn't warn on
this: http://llvm.org/PR13101
llvm-svn: 285930
|