Author: abataev
Date: Mon Feb 15 02:07:17 2016
New Revision: 260877
URL: http://llvm.org/viewvc/llvm-project?rev=260877&view=rev
Log:
[OPENMP] Remove extra sync barriers for 'firstprivate' clause.
Sync barrier will be emitted after generation of firstprivate variables
only if one of the firstpriva
Am Tuesday 09 February 2016, 17:06:58 schrieb Richard Smith:
> On Thu, Feb 4, 2016 at 3:17 PM, Niels Ole Salscheider via cfe-commits
>
> wrote:
> > Author: olesalscheider
> > Date: Thu Feb 4 17:17:20 2016
> > New Revision: 259842
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=259842&view=
ABataev added inline comments.
Comment at: lib/CodeGen/CGExpr.cpp:2937-2943
@@ -2936,2 +2936,9 @@
QualType BaseTy = Base.getType();
+ // If the base is a pointer, we actually need to compute the offset in the
+ // pointee.
+ if (BaseTy->isAnyPointerType()) {
+auto RBase
ABataev added inline comments.
Comment at: lib/CodeGen/CGOpenMPRuntime.h:1009-1015
@@ +1008,9 @@
+
+ /// \brief Emits call to void __kmpc_push_num_teamss(ident_t *loc, kmp_int32
+ /// global_tid, kmp_int32 num_teams, kmp_int32 thread_limit) to generate code
+ /// for num_teams
hokein updated this revision to Diff 47959.
hokein marked an inline comment as done.
hokein added a comment.
Minimize test case.
http://reviews.llvm.org/D17134
Files:
clang-tidy/readability/BracesAroundStatementsCheck.cpp
test/clang-tidy/readability-braces-around-statements-assert-failure.c
hokein marked an inline comment as done.
Comment at:
test/clang-tidy/readability-braces-around-statements-assert-failure.cpp:1
@@ -1,2 +1,2 @@
-// RUN: %check_clang_tidy %s readability-braces-around-statements %t
+// RUN: clang-tidy -checks='-*,readability-braces-around-statement
AndyG added a comment.
Richard, are you happy with this latest version? Can I proceed to commit it?
Thanks.
http://reviews.llvm.org/D15636
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
danielmarjamaki updated this revision to Diff 47962.
danielmarjamaki added a comment.
Run this check on C++ code also. I have rerun the add_new_check python script.
Minor code cleanups.
I have run this on the debian packages again. In 1806 projects there were 9691
warnings. I have so far triage
Author: asbokhan
Date: Mon Feb 15 04:39:04 2016
New Revision: 260881
URL: http://llvm.org/viewvc/llvm-project?rev=260881&view=rev
Log:
PR26449: Fixes for bugs in __builtin_classify_type implementation
This patch fixes the following bugs in __builtin_classify_type implementation:
1) Support for me
This revision was automatically updated to reflect the committed changes.
Closed by commit rL260881: PR26449: Fixes for bugs in __builtin_classify_type
implementation (authored by asbokhan).
Changed prior to commit:
http://reviews.llvm.org/D16846?vs=47475&id=47966#toc
Repository:
rL LLVM
ht
andreybokhanko added a comment.
John, thank you for all these tireless re-reviews -- very much appreciated!
Yours,
Andrey
Repository:
rL LLVM
http://reviews.llvm.org/D16846
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llv
Author: asbokhan
Date: Mon Feb 15 04:44:27 2016
New Revision: 260882
URL: http://llvm.org/viewvc/llvm-project?rev=260882&view=rev
Log:
PR26449: Tests for builtin_classify_type fix
In my previous commit (rL260881) I forget to svn add tests. This commit adds
them.
Differential Revision: http://rev
Hi Hans,
Could we please add the description of OpenCL support into the release notes?
It covers a larger time frame than just the current release but since it hasn't
been in the release notes before, I hope it's fine. It would be good to inform
what we were working on in the past.
I propose t
On Sun, Feb 14, 2016 at 11:00 PM, Felix Berger via cfe-commits
wrote:
> Author: flx
> Date: Sun Feb 14 22:00:39 2016
> New Revision: 260872
>
> URL: http://llvm.org/viewvc/llvm-project?rev=260872&view=rev
> Log:
> Add isAnyPointer() matchers. Register missing matchers.
>
> Summary:
> The isAnyPoin
aaboud updated this revision to Diff 47988.
aaboud added a comment.
Simplified LIT tests according to David comment.
http://reviews.llvm.org/D15977
Files:
lib/CodeGen/CGDebugInfo.cpp
lib/CodeGen/CGDebugInfo.h
lib/CodeGen/CGDecl.cpp
test/CodeGenCXX/debug-info-lb.cpp
Index: lib/CodeGen/C
jmolloy resigned from this revision.
jmolloy removed a reviewer: jmolloy.
jmolloy added a comment.
I'm not the right person to review this.
http://reviews.llvm.org/D16586
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/
jmolloy added a comment.
Hi Pawel,
Thanks for this - it generally looks good to me. Just one comment.
James
Comment at: lib/Target/AArch64/InstPrinter/AArch64InstPrinter.h:107
@@ -106,3 @@
- template
- void printAMIndexedWB(const MCInst *MI, unsigned OpNum,
-
Hal Finkel wrote on 09.02.2016 18:16:21:
> > I guess any application that uses C++11 threads should show this
> > problem immediately.
>
> I'm not sure we have anything like that in our test suite. Could you
> add a simple test?
Hmm, it looks like Nemanja already added this:
SingleSource/UnitTe
- Original Message -
> From: "Ulrich Weigand"
> To: "Hal Finkel"
> Cc: cfe-commits@lists.llvm.org
> Sent: Monday, February 15, 2016 9:06:42 AM
> Subject: Re: r259931 - [SystemZ] Define __GCC_HAVE_SYNC_COMPARE_AND_SWAP
> macros
>
>
>
> Hal Finkel wrote on 09.02.2016 18:16:21:
>
> > >
danielmarjamaki added a comment.
In http://reviews.llvm.org/D15332#352556, @danielmarjamaki wrote:
>
> I see no warning when running on Clang source code (files in
> llvm/tools/clang/lib/...).
For information I rerun with --header-filter=* and got some results.. I will
triage those..
ht
Hal Finkel wrote on 15.02.2016 16:30:15:
> > Hmm, it looks like Nemanja already added this:
> >
> > SingleSource/UnitTests/C++11/ stdthreadbug.cpp
> >
> > But the test is restricted to run only on ppc64le:
> >
> > #if defined(_LITTLE_ENDIAN) && defined(__PPC64__)
> > ...
> >
> > I think there's n
Author: tnorthover
Date: Mon Feb 15 10:38:10 2016
New Revision: 260898
URL: http://llvm.org/viewvc/llvm-project?rev=260898&view=rev
Log:
Darwin: fix stdlib handling when CLANG_DEFAULT_STDLIB is set
I'd accidentally skipped the CMake check in a premature optimisation. I'd also
put the original tes
Hi Jonas,
Thanks for looking at the patch!
> On 13 Feb 2016, at 01:26, Hahnfeld, Jonas wrote:
> as I did the change: Sorry that it broke for you on Darwin! I could not test
> myself and there doesn't seem to be a build-bot…
No worries. I think it only triggered in the stage 2 Green Dragon bots
carlo.bertolli marked an inline comment as done.
carlo.bertolli added a comment.
I updated the patch to reflect the comments. I will wait for an answer to the
comment asking to split the emit function for push_num_teams. Thanks!
Comment at: lib/CodeGen/CGOpenMPRuntime.h:1009-10
carlo.bertolli set the repository for this revision to rL LLVM.
carlo.bertolli updated this revision to Diff 47995.
carlo.bertolli added a comment.
Added comment for boolean parameter.
Repository:
rL LLVM
http://reviews.llvm.org/D17148
Files:
lib/CodeGen/CGOpenMPRuntime.cpp
lib/CodeGen/C
uweigand added a subscriber: uweigand.
uweigand added a comment.
I'm not sure this is the right approach. In any case, it differs from how the
option is handled in GCC.
-mzvector enables the language *syntax* extension, i.e. allows use of the
"vector unsigned int" etc. data types. Note that t
rjmccall added inline comments.
Comment at: lib/Sema/SemaTemplate.cpp:7829
@@ -7828,2 +7828,3 @@
UnresolvedSet<8> Matches;
+ FunctionDecl *Specialization = nullptr;
TemplateSpecCandidateSet FailedCandidates(D.getIdentifierLoc());
Please name this variable N
aaboud updated this revision to Diff 47997.
aaboud marked an inline comment as done.
aaboud added a comment.
Applied Adrian comment.
http://reviews.llvm.org/D16135
Files:
include/clang/AST/ASTConsumer.h
lib/AST/ASTConsumer.cpp
lib/CodeGen/CGDebugInfo.cpp
lib/CodeGen/CGDebugInfo.h
lib/
hintonda updated this revision to Diff 47999.
hintonda added a comment.
- Renamed variables to make code self-documenting
http://reviews.llvm.org/D17215
Files:
lib/Sema/SemaTemplate.cpp
test/SemaCXX/pr14211-crash-on-invalid.cpp
Index: test/SemaCXX/pr14211-crash-on-invalid.cpp
=
hintonda added a comment.
Btw, I don't have commit access, so if accepted, could you please commit for me.
http://reviews.llvm.org/D17215
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm
aaron.ballman added a comment.
A few minor nits, but one question (to me) remains: should this be in misc, or
is this a cppcoreguideline? I think the check, as is, is acceptable (and can
possibly be extended to be a rule-of-five check aliased under
cppcoreguidelines), but wanted to understand o
joker.eph created this revision.
joker.eph added a reviewer: tejohnson.
joker.eph added a subscriber: cfe-commits.
Herald added a subscriber: joker.eph.
Use the new pipeline implemented in D17115
http://reviews.llvm.org/D17272
Files:
lib/CodeGen/BackendUtil.cpp
Index: lib/CodeGen/BackendUtil.
aaron.ballman added inline comments.
Comment at: include/clang/ASTMatchers/ASTMatchers.h:4821
@@ +4820,3 @@
+/// initializer for i.
+AST_MATCHER(Expr, nullPointerConstant) {
+ return Matcher(
sbenza wrote:
> aaron.ballman wrote:
> > aaron.ballman wrote:
> > > s
aaron.ballman marked 2 inline comments as done.
aaron.ballman added a comment.
Ping
http://reviews.llvm.org/D15935
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
LegalizeAdulthood added inline comments.
Comment at: docs/clang-tidy/checks/readability-ternary-operator.rst:10
@@ +9,3 @@
+And it gets transformed into::
+ (condition ? expression0 : expression1);
+
Why add the parentheses around the entire statement? They seem
LegalizeAdulthood added a subscriber: LegalizeAdulthood.
Comment at: docs/clang-tidy/checks/readability-non-const-parameter.rst:15
@@ +14,3 @@
+
+ // warning here; p should be const
+ char f1(char *p) {
With pointers, there are always two layers of constness:
LegalizeAdulthood added inline comments.
Comment at: test/clang-tidy/readability-non-const-parameter.cpp:3
@@ +2,3 @@
+
+// Currently the checker only warns about pointer arguments.
+//
How hard is it to extend it to references?
Certainly the confusion about what
manmanren added a subscriber: manmanren.
manmanren added a comment.
Hi Akira,
Can you give a high-level explanation of how this patch fixes the problem? This
patch enforces that the capacity is 8-byte aligned, is it required to solve the
alignment issue here?
Thanks,
Manman
http://reviews.ll
manmanren added a comment.
Thanks for working on this!
Manman
Comment at: lib/Sema/TypeLocBuilder.cpp:99
@@ +98,3 @@
+// element was pushed.
+RemoveOrInsertPadding = NumBytesAtAlign4 != 0;
+ }
This patch seems to do two things:
1> simplify the logic of
zaks.anna requested changes to this revision.
zaks.anna added a comment.
This revision now requires changes to proceed.
Not sure how you got these changes, but some of them seem wrong and some seem
inconsistently applied.
Has this been tested?
Comment at: C:/LLVM/llvm/tools/cl
Author: sylvestre
Date: Mon Feb 15 14:38:54 2016
New Revision: 260912
URL: http://llvm.org/viewvc/llvm-project?rev=260912&view=rev
Log:
fix an indent issue in the doc which causes sphinx to fail with some versions
Modified:
clang-tools-extra/trunk/docs/clang-tidy/checks/readability-simplify-
timonvo added a comment.
Could someone please take a look at this patch as well as
http://reviews.llvm.org/D15781? Thanks!
http://reviews.llvm.org/D15883
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/
rjmccall added inline comments.
Comment at: lib/Sema/SemaTemplate.cpp:7845
@@ -7842,1 +7844,3 @@
+ } else
+NonTemplateMatch = Method;
}
Could you add an assertion here that NonTemplateMatch is still null? That
should definitely neve
compnerd added a subscriber: compnerd.
compnerd requested changes to this revision.
compnerd added a reviewer: compnerd.
compnerd added a comment.
This revision now requires changes to proceed.
These should be guarded by a check to ensure that they are not defined when
EHABI is not in effect at @
hintonda added inline comments.
Comment at: lib/Sema/SemaTemplate.cpp:7845
@@ -7842,1 +7844,3 @@
+ } else
+NonTemplateMatch = Method;
}
rjmccall wrote:
> Could you add an assertion here that NonTemplateMatch is still null? That
> sh
ariccio added a comment.
Whoops, forgot to submit my earlier comments. Responding to comments right
now...
Comment at: C:/LLVM/llvm/tools/clang/utils/analyzer/CmpRuns.py:31
@@ -30,3 +30,3 @@
import plistlib
-import CmpRuns
+import CmpRuns # ?
This file impo
ldionne updated this revision to Diff 48012.
ldionne added a comment.
Address some remaining issues:
- Rename __nth_element to __type_pack_element, as suggested by Richard Smith
- Fix template parameter position issue in createTypePackElementParameterList
AFAICT, the only remaining issue is whet
ariccio added a comment.
In http://reviews.llvm.org/D17253#353000, @zaks.anna wrote:
> Not sure how you got these changes, but some of them seem wrong and some seem
> inconsistently applied.
>
> Has this been tested?
Not yet, I was toying with running the first SARD test under it, and read
th
ariccio added inline comments.
Comment at: C:/LLVM/llvm/tools/clang/utils/analyzer/CmpRuns.py:194
@@ -182,3 +193,3 @@
# Backward compatibility API.
-def loadResults(path, opts, root = "", deleteEmpty=True):
+def loadResults(path, opts, root="", deleteEmpty=True):
return load
ariccio added a comment.
I've added a few comments where I think the changes are not quite clear.
Comment at: C:/LLVM/llvm/tools/clang/utils/analyzer/CmpRuns.py:159
@@ -147,3 +158,3 @@
if 'clang_version' in data:
-if self.clang_version == None:
+
+ Mike
If you like this… Mike has been setting up a bot running it. Which is super
awesome!
-Chris
> On Feb 14, 2016, at 1:40 PM, Duncan P. N. Exon Smith
> wrote:
>
>>
>> On 2016-Feb-08, at 22:01, Chris Bieneman via cfe-commits
>> wrote:
>>
>> Author: cbieneman
>> Date: Tue Feb 9 00:01:
zaks.anna added a comment.
Please, test these changes.
Thanks!
Anna.
Comment at: C:/LLVM/llvm/tools/clang/utils/analyzer/CmpRuns.py:31
@@ -30,3 +30,3 @@
import plistlib
-import CmpRuns
+import CmpRuns # ?
ariccio wrote:
> This file imports itself?
>
> I wa
compnerd added a comment.
It was on the mailing list, which won't show up on phabricator (email is still
the defacto review system).
Why not create a local macro and use that to make this easier to read?
#if defined(__arm__) && !defined(__USING_SJLJ_EXCEPTIONS__) &&
!defined(__ARM_DWARF_EH__
timonvo added a comment.
Hi compnerd, thanks for getting back to me. I've updated the code.
I was wondering though, where did you see @nbjoerg's comment? I couldn't find
any reference to it here or on cfe-commits.
http://reviews.llvm.org/D15883
__
timonvo updated this revision to Diff 48015.
timonvo added a comment.
Guard ARM EHABI enums using \#ifs (compnerd/nbjoerg's advice).
http://reviews.llvm.org/D15883
Files:
lib/Headers/unwind.h
Index: lib/Headers/unwind.h
===
---
rjmccall added inline comments.
Comment at: lib/Sema/SemaTemplate.cpp:7845
@@ -7842,1 +7844,3 @@
+ } else
+NonTemplateMatch = Method;
}
hintonda wrote:
> rjmccall wrote:
> > Could you add an assertion here that NonTemplateMatch is sti
timonvo updated this revision to Diff 48026.
timonvo added a comment.
Added local macro for enhanced readability.
http://reviews.llvm.org/D15883
Files:
lib/Headers/unwind.h
Index: lib/Headers/unwind.h
===
--- lib/Headers/unwind.
ayartsev added a comment.
Ping.
http://reviews.llvm.org/D16317
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
timonvo added a comment.
Note that I avoided using a local macro at first because I didn't want to
export any additional symbols/macros from this file. But I've moved to use one
now. Note that I changed the name slightly to be more in line with libunwind's
naming (it uses _LIBUNWIND_ARM_EHABI a
jbcoe removed rL LLVM as the repository for this revision.
jbcoe updated this revision to Diff 48028.
jbcoe added a comment.
Minor fixes from review.
http://reviews.llvm.org/D16376
Files:
clang-tidy/misc/CMakeLists.txt
clang-tidy/misc/DeprecatedSpecialMemberGenerationCheck.cpp
clang-tidy/
jbcoe marked 2 inline comments as done.
jbcoe added a comment.
http://reviews.llvm.org/D16376
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hintonda updated this revision to Diff 48031.
hintonda added a comment.
- Added assert and additional test
http://reviews.llvm.org/D17215
Files:
lib/Sema/SemaTemplate.cpp
test/SemaCXX/pr14211-crash-on-invalid.cpp
Index: test/SemaCXX/pr14211-crash-on-invalid.cpp
hintonda added inline comments.
Comment at: lib/Sema/SemaTemplate.cpp:7848
@@ -7842,2 +7847,3 @@
+ }
}
}
Got it...
http://reviews.llvm.org/D17215
___
cfe-commits mailing list
cfe-commits@lis
ahatanak updated this revision to Diff 48033.
ahatanak added a comment.
The bug is in the handling of elements whose LocalAlignment is 8, so I've fixed
just that part.
I'll resend a patch to simplify the logic of TypeLocBuilder::pushImpl later as
it's not urgent.
http://reviews.llvm.org/D1684
> On 2016-Feb-10, at 19:10, Marshall Clow wrote:
>
> On Wed, Feb 10, 2016 at 2:46 PM, Duncan P. N. Exon Smith via cfe-commits
> wrote:
> I'm hoping only a year or two (or three...)?
>
> As I pointed out to Eric on IRC, those files have had a "deprecation warning"
> every time they are includ
benlangmuir added a comment.
Okay, let's go with the two-path solution. Thanks!
http://reviews.llvm.org/D17104
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: ahatanak
Date: Mon Feb 15 18:46:03 2016
New Revision: 260921
URL: http://llvm.org/viewvc/llvm-project?rev=260921&view=rev
Log:
Remove unnecessary regex in the test cases I fixed in r260496.
Modified:
cfe/trunk/test/CodeGenObjC/exceptions-asm-attribute.m
cfe/trunk/test/CodeGenObjC/
Thanks for the catch! Those checks aren’t necessary at all. I’ve removed them
in r260921.
> On Feb 14, 2016, at 1:08 PM, Duncan P. N. Exon Smith
> wrote:
>
>
>> On 2016-Feb-10, at 22:36, Akira Hatanaka via cfe-commits
>> wrote:
>>
>> Author: ahatanak
>> Date: Thu Feb 11 00:36:35 2016
>> Ne
manmanren added inline comments.
Comment at: lib/Sema/TypeLocBuilder.cpp:130
@@ +129,3 @@
+ else if (CurrentlyHasPadding) {
+// Remove 4-byte padding.
+memmove(&Buffer[Index + 4], &Buffer[Index], NumBytesAtAlign4);
Expand this comment a little
steven_watanabe created this revision.
steven_watanabe added reviewers: rsmith, eli.friedman.
steven_watanabe added a subscriber: cfe-commits.
Currently clang -cc1 -ast-print puts the struct
definition in the wrong place, like this:
struct {} typedef S;
The reason that this happens is th
Author: vedantk
Date: Mon Feb 15 20:14:44 2016
New Revision: 260927
URL: http://llvm.org/viewvc/llvm-project?rev=260927&view=rev
Log:
Simplify users of StringRef::{l,r}trim (clang) (NFC)
r260925 introduced a version of the *trim methods which is preferable
when trimming a single kind of character
Author: chapuni
Date: Mon Feb 15 21:17:13 2016
New Revision: 260929
URL: http://llvm.org/viewvc/llvm-project?rev=260929&view=rev
Log:
clang/test/Driver/ps4-linker-win.c: Tweak substitutions if %PATH% ends with
'\\', or lit complains with "parser error".
Modified:
cfe/trunk/test/Driver/ps4-li
Author: akirtzidis
Date: Mon Feb 15 23:39:33 2016
New Revision: 260937
URL: http://llvm.org/viewvc/llvm-project?rev=260937&view=rev
Log:
[Frontend] Make sure WrapperFrontendAction updates CurrentInput after calling
BeginSourceFileAction.
I don't have a test case to add unfortunately.
Modified:
nikola added a comment.
Ping.
http://reviews.llvm.org/D15588
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
nikola added a comment.
Ping.
http://reviews.llvm.org/D15509
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Hi Tim,
> -Original Message-
> From: tnortho...@apple.com [mailto:tnortho...@apple.com]
> Sent: Monday, February 15, 2016 5:43 PM
> To: Hahnfeld, Jonas
> Cc: cfe-commits@lists.llvm.org
> Subject: Re: r260755 - Darwin: pass -stdlib=libc++ down to cc1 whenever
> we're targeting libc++
>
> H
pxli168 updated this revision to Diff 48046.
pxli168 marked 3 inline comments as done.
pxli168 added a comment.
Update doc for OpenCL access qualifier, still find where to handle access
qualifier for pipe type.
http://reviews.llvm.org/D16040
Files:
include/clang/Basic/Attr.td
include/clang
pxli168 requested a review of this revision.
pxli168 added a comment.
Need to refine access qualifier with pipe type.
Comment at: lib/Sema/SemaDeclAttr.cpp:5052
@@ +5051,3 @@
+ if (D->hasAttr()) {
+S.Diag(D->getLocation(), diag::err_opencl_multiple_access_qualifiers)
+
78 matches
Mail list logo
