Author: wmi
Date: Fri Sep 8 11:10:13 2017
New Revision: 312805
URL: http://llvm.org/viewvc/llvm-project?rev=312805&view=rev
Log:
Revert rL312801 since it generated some calls from libatomic and broke some
tests.
Modified:
cfe/trunk/lib/CodeGen/CGAtomic.cpp
cfe/trunk/test/CodeGenCXX/atom
Author: wmi
Date: Fri Sep 8 11:31:21 2017
New Revision: 312810
URL: http://llvm.org/viewvc/llvm-project?rev=312810&view=rev
Log:
Delete empty file test/CodeGenCXX/atomic-align.cpp after the revert at rL312805.
Removed:
cfe/trunk/test/CodeGenCXX/atomic-align.cpp
Removed: cfe/trunk/test/CodeG
lebedev.ri added a comment.
In https://reviews.llvm.org/D37629#865068, @lebedev.ri wrote:
> And finish reducing the code by for-range-loop`ing over array + use
> `std::array`.
I will need to fix handling of the second edge-case (comparison with max
unsigned value or with min/max for signed va
vmiklos created this revision.
The use case is that renaming multiple symbols in a large enough codebase is
much faster if all of these can be done with a single invocation, but
there will be multiple translation units where one or more symbols are
not found.
Old behavior was to exit with
Author: aaronballman
Date: Fri Sep 8 11:40:39 2017
New Revision: 312811
URL: http://llvm.org/viewvc/llvm-project?rev=312811&view=rev
Log:
Add some documentation on how to generate the documentation.
Modified:
cfe/trunk/include/clang/Basic/AttrDocs.td
Modified: cfe/trunk/include/clang/Basic/
> On Sep 8, 2017, at 10:54 AM, Paul Robinson via Phabricator
> wrote:
>
> probinson added a comment.
>
> In https://reviews.llvm.org/D37604#864187, @aprantl wrote:
>
>> This seems reasonable to me, thanks!
>> When you commit this, could you please double-check that the tests are still
>> run
aaron.ballman added inline comments.
Comment at: include/clang/Basic/AttrDocs.td:3154
+
+def SelectAnyDocs : Documentation {
+ let Content = [{This attribute makes global symbol have a weak definition
aaron.ballman wrote:
> majnemer wrote:
> > aaron.ballman wro
Author: vedantk
Date: Fri Sep 8 11:44:56 2017
New Revision: 312818
URL: http://llvm.org/viewvc/llvm-project?rev=312818&view=rev
Log:
[Coverage] Precise region termination with deferred regions (reapply)
The current coverage implementation doesn't handle region termination
very precisely. Take fo
rsmith added a comment.
Accepting this under `-std=c2x` is premature. We don't even know whether there
will be such a standard yet, and this has not been voted into a working draft.
But the `-f` flag form is OK.
Comment at: include/clang/Driver/Options.td:607
+def fcattribu
> On Sep 8, 2017, at 11:43 AM, Adrian Prantl wrote:
>
>
>> On Sep 8, 2017, at 10:54 AM, Paul Robinson via Phabricator
>> wrote:
>>
>> probinson added a comment.
>>
>> In https://reviews.llvm.org/D37604#864187, @aprantl wrote:
>>
>>> This seems reasonable to me, thanks!
>>> When you commit
JonasToth added inline comments.
Comment at: test/clang-tidy/cppcoreguidelines-owning-memory.cpp:39
+ return new int(42);
+ // CHECK-MESSAGES: [[@LINE-1]]:3: warning: returning a 'gsl::owner<>' from a
function but not declaring it; return type is 'int *'
+}
aa
cameron314 updated this revision to Diff 114414.
cameron314 added a comment.
The latest patch. I think this one should do the trick :-)
https://reviews.llvm.org/D37474
Files:
include/clang/Frontend/PrecompiledPreamble.h
lib/Frontend/ASTUnit.cpp
unittests/Frontend/CMakeLists.txt
unittest
jbcoe requested changes to this revision.
jbcoe added inline comments.
This revision now requires changes to proceed.
Comment at: bindings/python/tests/cindex/test_cursor.py:399
+else:
+assert False, "Could not find cursor for deleted constructor"
+
C
rwols updated this revision to Diff 114415.
rwols added a comment.
Update the description for the "-enable-snippets" option.
https://reviews.llvm.org/D37101
Files:
clangd/ClangdLSPServer.cpp
clangd/ClangdLSPServer.h
clangd/ClangdServer.cpp
clangd/ClangdServer.h
clangd/ClangdUnit.cpp
cameron314 added a comment.
It seems there's other users of `PrecompiledPreamble` that would have to be
fixed, yes. If we go with my original fix of taking into account the BOM in the
preamble bounds, there's no way of reusing the PCH when the BOM
appears/disappears. I still maintain this is a
Nebiroth updated this revision to Diff 114424.
Nebiroth marked 7 inline comments as done.
Nebiroth added a comment.
Ran clang-format on modified files.
Minor refactoring.
https://reviews.llvm.org/D36150
Files:
clangd/ClangdLSPServer.cpp
clangd/ClangdServer.cpp
clangd/ClangdServer.h
clan
aaron.ballman added inline comments.
Comment at: test/clang-tidy/misc-suspicious-enum-usage.cpp:122
+struct a> {
+ enum { ah = ad::m,
+ ai = ae::m,
alexfh wrote:
> alexfh wrote:
> > aaron.ballman wrote:
> > > alexfh wrote:
> > > > aaron.ballman wrote:
>
aaron.ballman added inline comments.
Comment at: test/clang-tidy/cppcoreguidelines-owning-memory.cpp:39
+ return new int(42);
+ // CHECK-MESSAGES: [[@LINE-1]]:3: warning: returning a 'gsl::owner<>' from a
function but not declaring it; return type is 'int *'
+}
---
vsk added inline comments.
Comment at: test/Index/skipped-ranges.c:23
// RUN: env CINDEXTEST_SHOW_SKIPPED_RANGES=1 c-index-test
-test-annotate-tokens=%s:1:1:16:1 %s | FileCheck %s
-// CHECK: Skipping: [5:2 - 6:7]
-// CHECK: Skipping: [8:2 - 12:7]
-// CHECK: Skipping: [14:2 - 20
On Fri, Sep 8, 2017 at 4:48 AM, Sam McCall via cfe-commits
wrote:
> Nice fix! It catches a lot of new cases on our codebase, all technically
> correct so far.
>
> A couple of issues though:
> A) Rollout - until we've completely cleaned up, we need to disable
> -Wtautological-compare entirely, whic
zahiraam updated this revision to Diff 114428.
zahiraam added a comment.
Responding to Erich 's and Aaron's reviews. Thanks.
https://reviews.llvm.org/D37308
Files:
lib/Sema/SemaDeclCXX.cpp
test/SemaCXX/ms-uuid.cpp
Index: test/SemaCXX/ms-uuid.cpp
===
erichkeane requested changes to this revision.
erichkeane added inline comments.
This revision now requires changes to proceed.
Comment at: lib/Sema/SemaDeclCXX.cpp:2377
+/// \brief Tests if the __interface base is public.
+static bool IsRecordPublicInterface(const CXXRecordDecl
erichkeane added a comment.
1 more thing I missed.
Comment at: lib/Sema/SemaDeclCXX.cpp:2389
+ return RD->isStruct() && RD->getName() == "IUnknown" && RD->isEmpty() &&
+ Uuid && Uuid->getGuid() =="---C000-0046";
+}
This also has
On Fri, Sep 8, 2017 at 11:45 PM, Hans Wennborg wrote:
> On Fri, Sep 8, 2017 at 4:48 AM, Sam McCall via cfe-commits
> wrote:
>> Nice fix! It catches a lot of new cases on our codebase, all technically
>> correct so far.
>>
>> A couple of issues though:
>> A) Rollout - until we've completely cleane
mstorsjo updated this revision to Diff 114429.
mstorsjo retitled this revision from "[MinGW] Allow overriding which version of
msvcrt to link to" to "[MinGW] Don't link -lmsvcrt if a different msvcrt
version is to be linked".
mstorsjo edited the summary of this revision.
mstorsjo added a comment.
efriedma added a comment.
Oh, you don't want to emit them by default. :)
I'm not sure what you're trying to do here... there are three possibilities:
1. The function could have an exception thrown through it, so we need an unwind
table.
2. The function can't have an exception thrown through it,
On Fri, Sep 8, 2017 at 2:09 PM, Roman Lebedev wrote:
> On Fri, Sep 8, 2017 at 11:45 PM, Hans Wennborg wrote:
>> On Fri, Sep 8, 2017 at 4:48 AM, Sam McCall via cfe-commits
>> wrote:
>>> Nice fix! It catches a lot of new cases on our codebase, all technically
>>> correct so far.
>>>
>>> A couple o
On 9/8/2017 2:18 PM, Hans Wennborg via cfe-commits wrote:
On Fri, Sep 8, 2017 at 2:09 PM, Roman Lebedev wrote:
Interesting. My first thought was to explicitly specify enum as signed:
enum MediaDeviceType : signed int {
MEDIA_DEVICE_TYPE_AUDIO_INPUT = 0,
MEDIA_DEVICE_TYPE_VIDEO_INPUT,
MEDIA_DE
vsk updated this revision to Diff 114433.
vsk added a comment.
Herald added subscribers: kbarton, nemanjai.
- Add an 'EndifLoc' parameter to the SourceRangeSkipped callback so that
indexing clients can preserve their existing behavior.
- I'll submit a follow-up patch which updates the pp-trace te
vsk created this revision.
Herald added subscribers: kbarton, nemanjai.
Depends on https://reviews.llvm.org/D36642
https://reviews.llvm.org/D37642
Files:
pp-trace/PPCallbacksTracker.cpp
pp-trace/PPCallbacksTracker.h
test/pp-trace/pp-trace-conditional.cpp
test/pp-trace/pp-trace-macro.cpp
On Fri, Sep 8, 2017 at 2:26 PM, Friedman, Eli wrote:
> On 9/8/2017 2:18 PM, Hans Wennborg via cfe-commits wrote:
>>
>> On Fri, Sep 8, 2017 at 2:09 PM, Roman Lebedev
>> wrote:
>>>
>>>
>>> Interesting. My first thought was to explicitly specify enum as signed:
>>>
>>> enum MediaDeviceType : signed
On Sat, Sep 9, 2017 at 12:18 AM, Hans Wennborg wrote:
> On Fri, Sep 8, 2017 at 2:09 PM, Roman Lebedev wrote:
>> On Fri, Sep 8, 2017 at 11:45 PM, Hans Wennborg wrote:
>>> On Fri, Sep 8, 2017 at 4:48 AM, Sam McCall via cfe-commits
>>> wrote:
Nice fix! It catches a lot of new cases on our cod
On Fri, Sep 8, 2017 at 5:49 PM, Hans Wennborg via cfe-commits
wrote:
> On Fri, Sep 8, 2017 at 2:26 PM, Friedman, Eli wrote:
>> On 9/8/2017 2:18 PM, Hans Wennborg via cfe-commits wrote:
>>>
>>> On Fri, Sep 8, 2017 at 2:09 PM, Roman Lebedev
>>> wrote:
Interesting. My first thought w
Author: wmi
Date: Fri Sep 8 14:58:18 2017
New Revision: 312830
URL: http://llvm.org/viewvc/llvm-project?rev=312830&view=rev
Log:
Reinstall the patch "Use EmitPointerWithAlignment to get alignment information
of the pointer used in atomic expr".
This is to fix PR34347. EmitAtomicExpr now only us
fjricci added inline comments.
Comment at: lib/Driver/ToolChain.cpp:851
+ XOpenMPTargetArg->setBaseArg(A);
+ A = XOpenMPTargetArg.release();
+ DAL->append(A);
Hahnfeld wrote:
> This is a memory leak that is currently triggered in
> `tests/Driver/o
kastiglione created this revision.
Herald added a subscriber: klimek.
Add the `objcImplementationDecl` matcher. See related:
https://reviews.llvm.org/D30854
Tested with:
./tools/clang/unittests/ASTMatchers/ASTMatchersTests
https://reviews.llvm.org/D37643
Files:
docs/LibASTMatchersReferen
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM, thanks!
https://reviews.llvm.org/D37643
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin
aaron.ballman updated this revision to Diff 114447.
aaron.ballman marked 2 inline comments as done.
aaron.ballman added a comment.
Updated based on Richard's comments and some further discussion on IRC.
https://reviews.llvm.org/D37436
Files:
include/clang/Basic/Attr.td
include/clang/Basic/A
vsk created this revision.
https://reviews.llvm.org/D37647
Files:
docs/ReleaseNotes.rst
docs/UndefinedBehaviorSanitizer.rst
Index: docs/UndefinedBehaviorSanitizer.rst
===
--- docs/UndefinedBehaviorSanitizer.rst
+++ docs/Undefin
Hello everyone,
Below are some buildbot numbers for the week of 07/16/2017 - 07/22/2017.
Please see the same data in attached csv files:
The longest time each builder was red during the last week;
"Status change ratio" by active builder (percent of builds that changed the
builder status from gre
Hello everyone,
Below are some buildbot numbers for the week of 07/23/2017 - 07/29/2017.
Please see the same data in attached csv files:
The longest time each builder was red during the last week;
"Status change ratio" by active builder (percent of builds that changed the
builder status from gre
Hello everyone,
Below are some buildbot numbers for the week of 07/30/2017 - 08/05/2017.
Please see the same data in attached csv files:
The longest time each builder was red during the last week;
"Status change ratio" by active builder (percent of builds that changed the
builder status from gre
Hello everyone,
Below are some buildbot numbers for the week of 08/06/2017 - 08/12/2017.
Please see the same data in attached csv files:
The longest time each builder was red during the last week;
"Status change ratio" by active builder (percent of builds that changed the
builder status from gre
Hello everyone,
Below are some buildbot numbers for the week of 08/13/2017 - 08/19/2017.
Please see the same data in attached csv files:
The longest time each builder was red during the last week;
"Status change ratio" by active builder (percent of builds that changed the
builder status from gre
Hello everyone,
Below are some buildbot numbers for last the week of 08/20/2017 -
08/26/2017.
Please see the same data in attached csv files:
The longest time each builder was red during the last week;
"Status change ratio" by active builder (percent of builds that changed the
builder status fro
vsk created this revision.
Make it possible to use the minimal ubsan runtime on Darwin.
https://reviews.llvm.org/D37649
Files:
lib/Driver/ToolChains/Darwin.cpp
test/Driver/sanitizer-ld.c
Index: test/Driver/sanitizer-ld.c
===
Hello everyone,
Below are some buildbot numbers for the last week of 08/27/2017 -
09/02/2017.
Please see the same data in attached csv files:
The longest time each builder was red during the last week;
"Status change ratio" by active builder (percent of builds that changed the
builder status fro
vsk added a comment.
Can you add a test?
Repository:
rL LLVM
https://reviews.llvm.org/D37470
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: compnerd
Date: Fri Sep 8 16:41:17 2017
New Revision: 312835
URL: http://llvm.org/viewvc/llvm-project?rev=312835&view=rev
Log:
CodeGen: correct arguments for NSFastEnumeration
When performing a NSFastEnumeration, the compiler synthesizes a call to
`countByEnumeratingWithState:objects:coun
inglorion updated this revision to Diff 114463.
inglorion added a comment.
added examples suggested by @zturner, verified step over and step into specific
behavior matches MSVC, and added tests for them
https://reviews.llvm.org/D37529
Files:
clang/lib/CodeGen/CGDebugInfo.cpp
clang/lib/Code
Well, if they worked I wasn't going to say we needed to add tests for them,
i just wanted to make sure they work before we move onto something else.
In any case, lgtm
On Fri, Sep 8, 2017 at 4:43 PM Bob Haarman via Phabricator <
revi...@reviews.llvm.org> wrote:
> inglorion updated this revision to
vsk added a comment.
Looks good to me.
https://reviews.llvm.org/D37564
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: jvesely
Date: Fri Sep 8 16:58:53 2017
New Revision: 312837
URL: http://llvm.org/viewvc/llvm-project?rev=312837&view=rev
Log:
configure.py: Simplify compatibility sources
Just add the SOURCE_X.Y list to the list of sources if X.Y is the current llvm
version.
Signed-off-by: Jan Vesely
R
Author: jvesely
Date: Fri Sep 8 16:59:00 2017
New Revision: 312839
URL: http://llvm.org/viewvc/llvm-project?rev=312839&view=rev
Log:
Implement vload_half{,n} and vload(half)
v2: add vload(half) as well
make helpers amdgpu specific (NVPTX uses different private AS numbering)
use clang bui
Author: jvesely
Date: Fri Sep 8 16:58:57 2017
New Revision: 312838
URL: http://llvm.org/viewvc/llvm-project?rev=312838&view=rev
Log:
vstore: Cleanup and add vstore(half)
Add missing undefs
Make helpers amdgpu specific (NVPTX uses different numbering for private AS)
Use clang builtins on clang >=
inglorion updated this revision to Diff 114465.
inglorion added a comment.
Of course, ApplyDebugLocation is also a perfectly legitimate way to add a debug
location to nodes that are not nested inside nodes that already have a
location. I updated the diff so that we do end up applying the locatio
Author: rtrieu
Date: Fri Sep 8 17:25:05 2017
New Revision: 312842
URL: http://llvm.org/viewvc/llvm-project?rev=312842&view=rev
Log:
Catch more cases with -Wenum-compare
Treat typedef enum as named enums instead of anonymous enums. Anonymous enums
are ignored by the warning, so previously, typed
eugenis created this revision.
Herald added subscribers: hiraditya, kristof.beyls, srhines, aemerson.
Set target_cpu and target_features attributes on __cfi_check_fail and
__cfi_check. Make cfi_check use Thumb encoding on ARM target.
https://reviews.llvm.org/D37656
Files:
clang/lib/CodeGen/CG
Author: rsmith
Date: Fri Sep 8 18:11:04 2017
New Revision: 312850
URL: http://llvm.org/viewvc/llvm-project?rev=312850&view=rev
Log:
[cxx_status] Change "Clang 5" items from yellow to green now that Clang 5 has
been released.
Modified:
cfe/trunk/www/cxx_dr_status.html
cfe/trunk/www/cxx_s
Author: rsmith
Date: Fri Sep 8 18:14:04 2017
New Revision: 312851
URL: http://llvm.org/viewvc/llvm-project?rev=312851&view=rev
Log:
Fix ownership of the MemoryBuffer in a FrontendInputFile.
This fixes a possible crash on certain kinds of corrupted AST file, but
checking in an AST file corrupted
Author: awatry
Date: Fri Sep 8 19:23:54 2017
New Revision: 312854
URL: http://llvm.org/viewvc/llvm-project?rev=312854&view=rev
Log:
integer: Add popcount implementation using ctpop intrinsic
Also copy/modify the unary_intrin.inc from math/ to make the
intrinsic declaration somewhat reusable.
Pa
clang-format: improve layout of help message
Signed-off-by: Bas van den Berg
---
tools/clang-format/ClangFormat.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/clang-format/ClangFormat.cpp
b/tools/clang-format/ClangFormat.cpp
index 14bff19..6cfce07 100644
--- a/
101 - 162 of 162 matches
Mail list logo
