Author: abataev
Date: Mon Oct 19 01:40:17 2015
New Revision: 250684
URL: http://llvm.org/viewvc/llvm-project?rev=250684&view=rev
Log:
[OPENMP] Fix for http://llvm.org/PR25221: Infinite loop while parsing OpenMP
directive
Clang skipped annot_pragma_openmp token, while it should be considered as a
mgrabovsky added a comment.
In http://reviews.llvm.org/D13643#266926, @aaron.ballman wrote:
> I would spend some time digging into how GCC handles those cases, and use
> that as a baseline that we can then improve upon. I like the fact that GCC
> basically says "use parens to clarify your inten
LegalizeAdulthood added a comment.
In http://reviews.llvm.org/D7639#266332, @Eugene.Zelenko wrote:
> What is preventing to add this check to Clang-tidy? Just found another piece
> of fresh C++ code in LLDB with (void) as argument list...
To be honest, I don't know. This review had taken S
LegalizeAdulthood updated this revision to Diff 37717.
http://reviews.llvm.org/D7639
Files:
clang-tidy/readability/CMakeLists.txt
clang-tidy/readability/ReadabilityTidyModule.cpp
clang-tidy/readability/RedundantVoidArgCheck.cpp
clang-tidy/readability/RedundantVoidArgCheck.h
test/clang-t
Author: ctopper
Date: Sun Oct 18 23:51:35 2015
New Revision: 250681
URL: http://llvm.org/viewvc/llvm-project?rev=250681&view=rev
Log:
Make getTargetBuiltins return an ArrayRef instead of having two out parameters
of a pointer and length. NFC
Modified:
cfe/trunk/include/clang/Basic/Builtins.h
Author: ctopper
Date: Sun Oct 18 22:52:27 2015
New Revision: 250678
URL: http://llvm.org/viewvc/llvm-project?rev=250678&view=rev
Log:
Recommit "Return an ArrayRef instead of having two out parameters of a pointer
and length. NFC". Hopefully this time the bots will be happy.
Modified:
cfe/tru
davide created this revision.
davide added a reviewer: rsmith.
davide added a subscriber: cfe-commits.
davide set the repository for this revision to rL LLVM.
Richard, this implements what you proposed in
https://llvm.org/bugs/show_bug.cgi?id=25223 , hopefully in the correct way.
This is what we
Author: ctopper
Date: Sun Oct 18 22:17:00 2015
New Revision: 250677
URL: http://llvm.org/viewvc/llvm-project?rev=250677&view=rev
Log:
Revert r250676 "Return an ArrayRef instead of having two out parameters of a
pointer and length. NFC"
Modified:
cfe/trunk/include/clang/Basic/TargetInfo.h
No worries! :)
On Sun, Oct 18, 2015, 5:42 PM Justin Bogner wrote:
> Eric Christopher writes:
> > I'm reasonably certain you forgot to rebuild or ran make test in the
> wrong
> > directory.
>
> You're right, I did something funny there. Sorry for the noise.
>
> > Two reasons:
> >
> > a) This is
Author: ctopper
Date: Sun Oct 18 22:05:12 2015
New Revision: 250676
URL: http://llvm.org/viewvc/llvm-project?rev=250676&view=rev
Log:
Return an ArrayRef instead of having two out parameters of a pointer and
length. NFC
Modified:
cfe/trunk/include/clang/Basic/TargetInfo.h
cfe/trunk/lib/Ba
thakis added inline comments.
Comment at: lib/Format/Format.cpp:1735
@@ -1734,3 +1734,3 @@
llvm::Regex IncludeRegex(
- R"(^[\t\ ]*#[\t\ ]*include[^"<]*(["<][^">]*[">]))");
+ R"(^[\t\ ]*#[\t\ ]*(import|include)[^"<]*(["<][^">]*[">]))");
SmallVector Matches;
thakis created this revision.
thakis added a reviewer: djasper.
thakis added a subscriber: cfe-commits.
Herald added a subscriber: klimek.
clang accepts both #include and #import for includes (the latter having an
implicit header guard). Let clang-format interleave both types if
--sort-includes
On Tue, Sep 29, 2015 at 12:53 AM, Daniel Jasper via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: djasper
> Date: Tue Sep 29 02:53:08 2015
> New Revision: 248782
>
> URL: http://llvm.org/viewvc/llvm-project?rev=248782&view=rev
> Log:
> clang-format: Extend #include sorting functionali
Author: nico
Date: Sun Oct 18 20:36:09 2015
New Revision: 250675
URL: http://llvm.org/viewvc/llvm-project?rev=250675&view=rev
Log:
clang-format: Extend main header include sorting heuristic to Objective-C files.
Modified:
cfe/trunk/lib/Format/Format.cpp
cfe/trunk/unittests/Format/SortIncl
Author: compnerd
Date: Sun Oct 18 20:24:08 2015
New Revision: 250674
URL: http://llvm.org/viewvc/llvm-project?rev=250674&view=rev
Log:
docs: remote stale refs
Since the attribute documentation is now auto-generated, the previous references
are no longer valid. This prevented the docs build from
I had committed this with
svn commit tools/clang-format -m "Update list of languages advertised in
`clang-format -h` output."
and since I used "" and not '' the shell executed the `clang-format -h` bit
and made the commit message look funny. Apologies.
On Sun, Oct 18, 2015 at 6:03 PM, Nico Web
Author: nico
Date: Sun Oct 18 20:08:30 2015
New Revision: 250672
URL: http://llvm.org/viewvc/llvm-project?rev=250672&view=rev
Log:
Update `clang-format -help` output in clang-format docs.
-assume-filename, -fallback-style, and -sort-includes are new. (They're also
longer than the previous option
Author: nico
Date: Sun Oct 18 20:03:19 2015
New Revision: 250671
URL: http://llvm.org/viewvc/llvm-project?rev=250671&view=rev
Log:
Update list of languages advertised in OVERVIEW: A tool to format
C/C++/Java/JavaScript/Objective-C/Protobuf code.
If no arguments are specified, it formats the code
thakis created this revision.
thakis added a reviewer: djasper.
thakis added a subscriber: cfe-commits.
Herald added a subscriber: klimek.
This makes the format tests look more like most other FileCheck tests in clang.
The multiple-inputs tests still use temp files, to make sure that the file
in
Eric Christopher writes:
> I'm reasonably certain you forgot to rebuild or ran make test in the wrong
> directory.
You're right, I did something funny there. Sorry for the noise.
> Two reasons:
>
> a) This is pretty much my first patch. I mean, identical and I tried that.
> b) I actually tried t
Author: nicholas
Date: Sun Oct 18 15:32:12 2015
New Revision: 250666
URL: http://llvm.org/viewvc/llvm-project?rev=250666&view=rev
Log:
No functionality change, just fix whitespace, a typo and remove an unnecessary
emacs mode marker. (Changes left behind from another patch that ended up not
working
Author: compnerd
Date: Sun Oct 18 15:24:53 2015
New Revision: 250665
URL: http://llvm.org/viewvc/llvm-project?rev=250665&view=rev
Log:
CodeGen: simplify TargetOptions setup
Do direct assignment of boolean values and regroup. Use StringSwitch instead of
custom cases. NFC.
Modified:
cfe/trun
dim added auditors: 3.7-release, cfe-commits, tstellarAMD, joerg.
dim added a comment.
Should be merged together with http://reviews.llvm.org/rL248379,
http://reviews.llvm.org/rL248424, and http://reviews.llvm.org/rL248426.
Together, these add support for linking against OpenMP on FreeBSD and N
dim added a subscriber: dim.
dim added auditors: 3.7-release, cfe-commits, tstellarAMD, joerg.
dim added a comment.
Should be merged together with http://reviews.llvm.org/rL248424,
http://reviews.llvm.org/rL248426, and http://reviews.llvm.org/rL250657.
Together, these add support for linking ag
dim added a subscriber: dim.
dim added auditors: 3.7-release, cfe-commits, tstellarAMD, joerg.
dim added a comment.
Should be merged together with http://reviews.llvm.org/rL248379,
http://reviews.llvm.org/rL248424, and http://reviews.llvm.org/rL250657.
Together, these add support for linking ag
dim added a subscriber: dim.
dim added auditors: 3.7-release, cfe-commits, tstellarAMD, joerg.
dim added a comment.
Should be merged together with http://reviews.llvm.org/rL248379,
http://reviews.llvm.org/rL248426, and http://reviews.llvm.org/rL250657.
Together, these add support for linking ag
Interestingly, gcc doesn't get this right either
*test.cpp:7:64:* *error: *‘*__builtin_ia32_xorpd256_mask*’ needs isa option
-m32 -mavx512dq -mavx512vl
I'm not even sure why it bothered to list -m32.
If you then give it -mavx512dq without -mavx512vl it proceeds to throw
internal compiler errors
Alexander_Droste added inline comments.
Comment at: tools/clang/lib/StaticAnalyzer/Checkers/MPI-Checker/MPITypes.h:67
@@ +66,3 @@
+ // Every time a request is 'set' a new 'RequestId' gets created.
+ // Therefore, the 'UserKind' does not need to be profiled.
+ const int RequestI
Author: dim
Date: Sun Oct 18 08:32:20 2015
New Revision: 250657
URL: http://llvm.org/viewvc/llvm-project?rev=250657&view=rev
Log:
Support linking against OpenMP runtime on FreeBSD.
Summary:
Similar to rL248426 (which was a followup to rL248379 and rL248424), add the
required libraries for OpenMP
djasper added inline comments.
Comment at: lib/Format/UnwrappedLineFormatter.cpp:193
@@ -192,3 +192,3 @@
(Style.AllowShortFunctionsOnASingleLine >= FormatStyle::SFS_Empty &&
- I[1]->First->is(tok::r_brace)) ||
+ I[1]->First->isOneOf(tok::l_brace, tok::r_br
djasper closed this revision.
djasper added a comment.
Submitted as r250648.
http://reviews.llvm.org/D13765
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: djasper
Date: Sun Oct 18 02:02:28 2015
New Revision: 250648
URL: http://llvm.org/viewvc/llvm-project?rev=250648&view=rev
Log:
clang-format: [JS] Handle string literals spanning character classes.
If a RegExp contains a character group with a quote (/["]/), the
trailing end of it is first
djasper accepted this revision.
djasper added a comment.
This revision is now accepted and ready to land.
Looks good.
http://reviews.llvm.org/D13765
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listin
33 matches
Mail list logo
