sepavloff updated this revision to Diff 78652.
sepavloff added a comment.
Herald added a subscriber: aemerson.
Updated patch.
- Restrict the search for config files specified by mangled clang name to
binary directory only,
- Allow the search for config files specified by `--config` in binary dir
Author: kromanova
Date: Sun Nov 20 02:35:05 2016
New Revision: 287483
URL: http://llvm.org/viewvc/llvm-project?rev=287483&view=rev
Log:
Add doxygen comments to immintrin.h's intrinsics.
The doxygen comments are automatically generated based on Sony's intrinsics docu
ment.
I got an OK from Eric C
hfinkel added a comment.
What happens with unused arguments in the configuration files? This feature
looks potentially useful for me, but only if it suppresses unused-argument
warnings. For example, if I put
-L/usr/local/lib -stdlib=libc++
into a configuration file, I can't have these:
cl
AntonBikineev updated this revision to Diff 78654.
AntonBikineev added a comment.
Richard, thanks, I addressed your comments.
https://reviews.llvm.org/D26829
Files:
include/clang/Lex/LiteralSupport.h
lib/Lex/Lexer.cpp
lib/Lex/LiteralSupport.cpp
lib/Sema/SemaDeclCXX.cpp
test/SemaCXX/cx
ogoffart added a comment.
ping?
https://reviews.llvm.org/D26465
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ogoffart added a comment.
ping?
https://reviews.llvm.org/D26350
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sepavloff marked 5 inline comments as done.
sepavloff added a comment.
> Rather than inventing a new mechanism, could we extend our existing `@file`
> facility to support comments and nested inclusion of further `@file`s?
Reusing `@file` is an attractive idea, but it cannot be implemented due to
mgorny added a comment.
In https://reviews.llvm.org/D24933#600884, @sepavloff wrote:
> > Whoever makes the `blah-clang` symlink should get to control what the
> > default configuration for `blah-clang` is, I think.
>
> The patch is changed so that config file for `blah-clang` is searched for
>
coby abandoned this revision.
coby added a comment.
deprecated.
Repository:
rL LLVM
https://reviews.llvm.org/D26258
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
malcolm.parsons updated this revision to Diff 78663.
malcolm.parsons added a comment.
Use llvm_unreachable.
Ignore template instantiations.
https://reviews.llvm.org/D26750
Files:
clang-tidy/modernize/CMakeLists.txt
clang-tidy/modernize/ModernizeTidyModule.cpp
clang-tidy/modernize/UseDefau
Author: ericwf
Date: Sun Nov 20 15:33:12 2016
New Revision: 287507
URL: http://llvm.org/viewvc/llvm-project?rev=287507&view=rev
Log:
Merge r286774 - Fixes PR30979
Modified:
libcxx/branches/release_39/include/tuple
libcxx/branches/release_39/test/std/utilities/tuple/tuple.tuple/tuple.cnst
mgorny created this revision.
mgorny added reviewers: beanz, rnk, chapuni, chandlerc.
mgorny added a subscriber: cfe-commits.
Use the new --cmakedir option to obtain LLVM_CMAKE_PATH straight from
llvm-config instead of reconstructing it locally.
https://reviews.llvm.org/D26900
Files:
CMakeLis
mclow.lists added inline comments.
Comment at: test/std/containers/associative/map/map.access/max_size.pass.cpp:24
+
+template size_t alloc_max_size(Alloc const &a) {
+ typedef std::allocator_traits AT;
How about a newline here? (after the `template<>`)
And wh
Thanks for the great explanation, I agree with this advice. I'm going to
investigate writing a new patch to do this.
On Fri, Nov 18, 2016 at 1:59 PM, Arthur O'Dwyer
wrote:
> On Thu, Nov 17, 2016 at 2:14 PM, Sasha Bermeister
> wrote:
>
>> Although I agree with your philosophical discussion and s
Author: ericwf
Date: Sun Nov 20 18:41:32 2016
New Revision: 287512
URL: http://llvm.org/viewvc/llvm-project?rev=287512&view=rev
Log:
Adjust uses_alloc_types helpers for later changes
Modified:
libcxx/trunk/test/support/uses_alloc_types.hpp
Modified: libcxx/trunk/test/support/uses_alloc_types
hubert.reinterpretcast added inline comments.
Comment at: include/clang/AST/Decl.h:1915
+IsConstexprSpecified = IC;
+IsConstexpr = IC;
+ }
How is the `inline` property transmitted here? Why does the
`setImplicitlyConstexpr` function need to call `setImp
Author: ericwf
Date: Sun Nov 20 19:10:52 2016
New Revision: 287513
URL: http://llvm.org/viewvc/llvm-project?rev=287513&view=rev
Log:
Mark variadic lock guard tests as XFAIL with modules, since they have to define
macros to expose the new ABI
Modified:
libcxx/trunk/test/std/thread/thread.mut
coby updated this revision to Diff 78681.
Repository:
rL LLVM
https://reviews.llvm.org/D26587
Files:
test/CodeGen/ms-inline-asm-avx512-memory-adjustments.c
Index: test/CodeGen/ms-inline-asm-avx512-memory-adjustments.c
===
---
CaseyCarter added a comment.
I'll take a look at these tomorrow; I don't have easy access to my
implementation with the changes from P0510 and P0504 from home (it's waiting on
a compiler bugfix to checkin).
Comment at: test/std/utilities/variant/variant.get/get_type.pass.cpp:
This revision was automatically updated to reflect the committed changes.
Closed by commit rL287518: [XRay] Support AArch64 in Clang (authored by
dberris).
Changed prior to commit:
https://reviews.llvm.org/D26415?vs=78216&id=78685#toc
Repository:
rL LLVM
https://reviews.llvm.org/D26415
Fil
Author: dberris
Date: Sun Nov 20 21:24:59 2016
New Revision: 287518
URL: http://llvm.org/viewvc/llvm-project?rev=287518&view=rev
Log:
[XRay] Support AArch64 in Clang
This patch adds XRay support in Clang for AArch64 target.
This patch is one of a series:
LLVM: https://reviews.llvm.org/D26412
com
21 matches
Mail list logo
