sepavloff added a comment.
The patch requires tests.
For instance, import of CXXBaseSpecifier can be tested along with static_cast:
if a class indeed is a base, static_cast will be processed silent, otherwise a
message must appear. ArraySubscript could be tested using constexpr function.
Materi
curdeius added a comment.
Ping?
http://reviews.llvm.org/D13811
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: vkalintiris
Date: Mon Nov 9 04:21:04 2015
New Revision: 252457
URL: http://llvm.org/viewvc/llvm-project?rev=252457&view=rev
Log:
Add initial support for the MUSL C library.
Summary:
This patch adds the LIBCXX_LIBC_IS_MUSL cmake option to allow the
building of libcxx with the Musl C libra
This revision was automatically updated to reflect the committed changes.
Closed by commit rL252457: Add initial support for the MUSL C library.
(authored by vkalintiris).
Changed prior to commit:
http://reviews.llvm.org/D13673?vs=38788&id=39673#toc
Repository:
rL LLVM
http://reviews.llvm.o
rengolin accepted this revision.
rengolin added a comment.
This revision is now accepted and ready to land.
LGTM, thanks!
Repository:
rL LLVM
http://reviews.llvm.org/D14184
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm
Author: chapuni
Date: Mon Nov 9 04:55:04 2015
New Revision: 252458
URL: http://llvm.org/viewvc/llvm-project?rev=252458&view=rev
Log:
Make check-libcxx Py3-compatible.
Modified:
libcxx/trunk/test/libcxx/test/tracing.py
Modified: libcxx/trunk/test/libcxx/test/tracing.py
URL:
http://llvm.org/
vkalintiris added a comment.
Thanks for the review!
Everything seems fine from the buildbots except for a single test that failed:
http://lab.llvm.org:8011/builders/libcxx-libcxxabi-x86_64-linux-debian/builds/730
The next build (731) of that buildbot was green. Is
libc++::try_lock_shared_fo
Author: rengolin
Date: Mon Nov 9 06:40:41 2015
New Revision: 252463
URL: http://llvm.org/viewvc/llvm-project?rev=252463&view=rev
Log:
[EABI] Add Clang support for -meabi flag
The -meabi flag to control LLVM EABI version.
Without '-meabi' or with '-meabi default' imply LLVM triple default.
With
klimek accepted this revision.
klimek added a comment.
This revision is now accepted and ready to land.
Ok, this patch LG. I'd really like to get rid of the current-file-dependent
stuff in ClangTidyOptions though :)
http://reviews.llvm.org/D14192
_
rengolin closed this revision.
rengolin added a comment.
r252463
Repository:
rL LLVM
http://reviews.llvm.org/D14184
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
danielmarjamaki updated this revision to Diff 39679.
danielmarjamaki marked 2 inline comments as done.
danielmarjamaki added a comment.
fixes of review comments
http://reviews.llvm.org/D12359
Files:
include/clang/AST/Decl.h
include/clang/Basic/DiagnosticGroups.td
include/clang/Basic/Diagn
hfinkel added auditors: rjmccall.
hfinkel added a comment.
John is the right auditor for this, not me.
Users:
hfinkel (Author, Auditor)
3.7-release (Auditor)
cfe-commits (Auditor)
tstellarAMD (Auditor)
rjmccall (Auditor)
http://reviews.llvm.org/rL246882
jroelofs accepted this revision.
jroelofs added a comment.
This revision is now accepted and ready to land.
LGTM
http://reviews.llvm.org/D14292
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cf
Author: aaronballman
Date: Mon Nov 9 09:24:09 2015
New Revision: 252470
URL: http://llvm.org/viewvc/llvm-project?rev=252470&view=rev
Log:
Fixing SPHINX warnings with incorrect indentations.
Modified:
cfe/trunk/include/clang/Basic/AttrDocs.td
Modified: cfe/trunk/include/clang/Basic/AttrDocs.
seaneveson added a comment.
Fix an assertion which occurs when getCXXThisVal() returns an Unknown SVal and
the Analyzer tries to get the corresponding memory region.
This assertion was reported in PR25392. The test case from this Bugzilla has
been added.
(Forgot to add cfe-commits when creatin
Author: alexfh
Date: Mon Nov 9 09:53:28 2015
New Revision: 252471
URL: http://llvm.org/viewvc/llvm-project?rev=252471&view=rev
Log:
[clang-tidy] Fix message style (capitalization, trailing period).
Modified:
clang-tools-extra/trunk/clang-tidy/readability/ContainerSizeEmptyCheck.cpp
clang
jroelofs updated this revision to Diff 39693.
jroelofs added a comment.
Add flags to turn the new targets off.
http://reviews.llvm.org/D14403
Files:
tools/CMakeLists.txt
tools/Makefile
tools/scan-build/CMakeLists.txt
tools/scan-build/Makefile
tools/scan-build/scan-build
tools/scan-v
Author: jroelofs
Date: Mon Nov 9 10:12:56 2015
New Revision: 252474
URL: http://llvm.org/viewvc/llvm-project?rev=252474&view=rev
Log:
Create install targets for scan-build and scan-view
http://reviews.llvm.org/D14403
Added:
cfe/trunk/tools/scan-build/CMakeLists.txt
cfe/trunk/tools/scan-
jroelofs accepted this revision.
jroelofs added a reviewer: jroelofs.
jroelofs added a comment.
This revision is now accepted and ready to land.
r252474
http://reviews.llvm.org/D14403
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://li
Author: alexfh
Date: Mon Nov 9 10:28:11 2015
New Revision: 252485
URL: http://llvm.org/viewvc/llvm-project?rev=252485&view=rev
Log:
Add ExtraArgs and ExtraArgsBefore options to enable clang warnings via
configuration files.
Summary: This patch depends on http://reviews.llvm.org/D14191
Reviewer
rengolin added a comment.
Hi Akira,
I'm uncomfortable with this change, since it introduces a dependency between
the two calls, and that's fragile. Also, the nullptr fiddling is not a good
design overall.
If there is a dependency, I suggest you encode it directly into
getAArch64TargetFeatures
tstellarAMD edited projects, added 3.7.1-merged; removed 3.7-release.
Users:
dim (Author)
3.7-release (Auditor)
cfe-commits (Auditor)
tstellarAMD (Auditor)
joerg (Auditor)
rsmith (Auditor)
http://reviews.llvm.org/rL250657
___
cfe-commits
ahatanak updated this revision to Diff 39702.
ahatanak added a comment.
I ended up making a few more changes based on the feedback on the
not_tail_called attribute:
- Changed wording in docs and added a code example.
- Made changes to detect both (naked,disable_tail_calls) and
(disable_tail_cal
rizsotto.mailinglist updated this revision to Diff 39703.
rizsotto.mailinglist added a comment.
findings from compare from older Perl implementation
http://reviews.llvm.org/D9600
Files:
tools/scan-build-py/README.md
tools/scan-build-py/libear/CMakeLists.txt
tools/scan-build-py/libear/__in
Author: alexfh
Date: Mon Nov 9 10:45:17 2015
New Revision: 252488
URL: http://llvm.org/viewvc/llvm-project?rev=252488&view=rev
Log:
Adjust printQualifiedName to handle unscoped enums in a way similar to
anonymous namespaces.
Patch by Sterling Augustine!
Differential revision: http://reviews.ll
Author: jroelofs
Date: Mon Nov 9 10:47:09 2015
New Revision: 252489
URL: http://llvm.org/viewvc/llvm-project?rev=252489&view=rev
Log:
Fix the cmake build after r252474 broke it
Modified:
cfe/trunk/tools/scan-build/CMakeLists.txt
cfe/trunk/tools/scan-view/CMakeLists.txt
Modified: cfe/tru
This revision was automatically updated to reflect the committed changes.
Closed by commit rL252488: Adjust printQualifiedName to handle unscoped enums
in a way similar to… (authored by alexfh).
Changed prior to commit:
http://reviews.llvm.org/D14459?vs=39605&id=39704#toc
Repository:
rL LLVM
On Mon, Nov 9, 2015 at 11:45 AM, Alexander Kornienko via cfe-commits
wrote:
> Author: alexfh
> Date: Mon Nov 9 10:45:17 2015
> New Revision: 252488
>
> URL: http://llvm.org/viewvc/llvm-project?rev=252488&view=rev
> Log:
> Adjust printQualifiedName to handle unscoped enums in a way similar to
> a
alexfh created this revision.
alexfh added a reviewer: aaron.ballman.
alexfh added a subscriber: cfe-commits.
Use "auto" when the type name is redundant
http://reviews.llvm.org/D14501
Files:
lib/AST/Decl.cpp
Index: lib/AST/Decl.cpp
=
On Mon, Nov 9, 2015 at 8:50 AM, Aaron Ballman
wrote:
> On Mon, Nov 9, 2015 at 11:45 AM, Alexander Kornienko via cfe-commits
> wrote:
> > Author: alexfh
> > Date: Mon Nov 9 10:45:17 2015
> > New Revision: 252488
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=252488&view=rev
> > Log:
> > Ad
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
Thank you for tackling this! LGTM!
~Aaron
http://reviews.llvm.org/D14501
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
h
Author: alexfh
Date: Mon Nov 9 11:53:06 2015
New Revision: 252494
URL: http://llvm.org/viewvc/llvm-project?rev=252494&view=rev
Log:
Use "auto" when the type name is redundant
Summary: Use "auto" when the type name is redundant
Reviewers: aaron.ballman
Subscribers: cfe-commits
Differential Rev
Author: aaronballman
Date: Mon Nov 9 12:04:34 2015
New Revision: 252496
URL: http://llvm.org/viewvc/llvm-project?rev=252496&view=rev
Log:
Rewording some of this documentation to describe the check instead of try to
rationalize the behavior of the check. The links already provide sufficient
rati
aaron.ballman created this revision.
aaron.ballman added a reviewer: beanz.
aaron.ballman added a subscriber: cfe-commits.
When generating an MSVC solution from CMake, it creates a clang-install
project. We do not have any *-install targets as part of MSVC, so this seems
out of place (not to men
dvadym updated this revision to Diff 39717.
dvadym marked 5 inline comments as done.
dvadym added a comment.
Addressed reviewers' comments
1.Change message generation on using diag() string substitution
2.Added copyright
3.Rebase
http://reviews.llvm.org/D12031
Files:
clang-tidy/misc/CMakeList
dvadym added a comment.
Thanks for review comments! Could you please have an another look and help me
with my questions?
Comment at: clang-tidy/misc/MoveConstantArgumentCheck.cpp:10
@@ +9,3 @@
+void MoveConstantArgumentCheck::registerMatchers(MatchFinder* Finder) {
+ Finder->a
bcraig added a subscriber: bcraig.
bcraig added a comment.
Does this affect the presence or functionality of the "INSTALL" project?
http://reviews.llvm.org/D14502
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/
lvoufo updated this revision to Diff 39724.
lvoufo added a comment.
Fix patch (wrong one submitted initially).
http://reviews.llvm.org/D14419
Files:
include/clang/AST/DeclCXX.h
lib/AST/DeclCXX.cpp
lib/AST/Type.cpp
test/CodeGenCXX/const-invariant.cpp
Index: test/CodeGenCXX/const-invaria
eugenis added a comment.
Richard, are you OK with this?
Repository:
rL LLVM
http://reviews.llvm.org/D13925
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aaron.ballman added a comment.
In http://reviews.llvm.org/D14502#285305, @bcraig wrote:
> Does this affect the presence or functionality of the "INSTALL" project?
There is no INSTALL project in MSVC from what I can see (before or after this
patch), so I don't think that this affects that funct
Author: aizatsky
Date: Mon Nov 9 13:12:18 2015
New Revision: 252501
URL: http://llvm.org/viewvc/llvm-project?rev=252501&view=rev
Log:
Moving FileManager::removeDotPaths to llvm::sys::path::remove_dots
Differential Revision: http://reviews.llvm.org/D14394
Modified:
cfe/trunk/include/clang/Ba
EricWF added a comment.
Yep. That's a flaky test. No need to worry.
Repository:
rL LLVM
http://reviews.llvm.org/D13673
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: dcoughlin
Date: Mon Nov 9 13:50:29 2015
New Revision: 252506
URL: http://llvm.org/viewvc/llvm-project?rev=252506&view=rev
Log:
[analyzer] Fix assertion failure invalidating on const member function calls
(PR25392).
We now return early when the 'this' value cannot be converted to a MemRe
dcoughlin added a comment.
Hi Sean, I just committed a slightly different fix that I already had prepared
(r252506). That fix also handles the case when the ThisVal is a loc that can't
be converted to a memory region.
Thanks for looking into this and my apologies for the duplication of effort!
t.p.northover closed this revision.
t.p.northover added a comment.
Thanks Saleem, committed in r252507.
Comment at: lib/CodeGen/CGAtomic.cpp:782
@@ +781,3 @@
+ LValue AtomicVal = MakeAddrLValue(Ptr, AtomicTy);
+ AtomicInfo Atomics(*this, AtomicVal);
+
compnerd
Author: tnorthover
Date: Mon Nov 9 13:56:35 2015
New Revision: 252507
URL: http://llvm.org/viewvc/llvm-project?rev=252507&view=rev
Log:
Atomics: support __c11_* calls on _Atomic struct types.
When a struct's size is not a power of 2, the corresponding _Atomic() type is
promoted to the nearest. W
aaron.ballman added a comment.
In http://reviews.llvm.org/D14502#285371, @aaron.ballman wrote:
> In http://reviews.llvm.org/D14502#285305, @bcraig wrote:
>
> > Does this affect the presence or functionality of the "INSTALL" project?
>
>
> There is no INSTALL project in MSVC from what I can see (b
bcraig created this revision.
bcraig added reviewers: zaks.anna, rsmith.
bcraig added a subscriber: cfe-commits.
Herald added a subscriber: klimek.
RecursiveASTVisitor allows visitors to specify whether they wish to visit
implicitly created code or not. I have ported that code over to
DataRecursi
davide added a comment.
Gentle Monday morning ping :)
Repository:
rL LLVM
http://reviews.llvm.org/D13854
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
bcraig added a comment.
In a text file, I have a big long cmake invocation. Part of that invocation is
the following:
-DCMAKE_INSTALL_PREFIX=c:/install/Tools
The VC "INSTALL" project copies files there, in a very unix-y hierarchy (i.e.
c:\install\Tools\bin, c:\install\Tools\include, c:\inst
Author: marshall
Date: Mon Nov 9 14:49:15 2015
New Revision: 252509
URL: http://llvm.org/viewvc/llvm-project?rev=252509&view=rev
Log:
Mark LWG#2486 as complete. Eric did this as part of r242959
Modified:
libcxx/trunk/www/cxx1z_status.html
Modified: libcxx/trunk/www/cxx1z_status.html
URL:
h
ahatanak added a comment.
Hi Renato
In http://reviews.llvm.org/D14471#285138, @rengolin wrote:
> I'm uncomfortable with this change, since it introduces a dependency between
> the two calls, and that's fragile. Also, the nullptr fiddling is not a good
> design overall.
>
> If there is a depend
rsmith added a comment.
Sorry for the long delay. This looks to be in good shape.
Comment at: include/clang/AST/Type.h:1210
@@ +1209,3 @@
+/// Which keyword(s) were used to create an AutoType.
+enum class AutoTypeKeyword : unsigned char {
+ /// \brief auto
The
rsmith added inline comments.
Comment at: lib/Sema/SemaType.cpp:2621
@@ -2620,3 +2620,3 @@
// C++14 In generic lambdas allow 'auto' in their parameters.
- if (ContainsPlaceholderType &&
+ if (D.getDeclSpec().containsPlaceholderType() &&
(!SemaRef.getLangOpts().CPlusPlu
Author: eugenis
Date: Mon Nov 9 15:10:54 2015
New Revision: 252514
URL: http://llvm.org/viewvc/llvm-project?rev=252514&view=rev
Log:
Extend linux header search to find libc++ headers in c++/vN for any N.
Added:
cfe/trunk/test/Driver/Inputs/basic_linux_libcxxv2_tree/
cfe/trunk/test/Driver
eugenis closed this revision.
eugenis added a comment.
r252514
Repository:
rL LLVM
http://reviews.llvm.org/D12382
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Or Tamas can write the tuning patch - it seems like it'd be relatively
straightforward. Perhaps you can give an idea of what you think it'd look
like, Paul (what the command line syntax would be, etc)
On Fri, Nov 6, 2015 at 11:16 AM, Paul Robinson via cfe-commits <
cfe-commits@lists.llvm.org> wrot
Any chance of a test case here?
(I don't know much about this code, but I am a bit confused about why we'd
ever need to visit a list of mangled names & look them up... )
On Thu, Nov 5, 2015 at 3:18 PM, Keno Fischer via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: kfischer
> Date: T
On Thu, Nov 5, 2015 at 11:05 AM, Robinson, Paul <
paul_robin...@playstation.sony.com> wrote:
> | What was your primary motivation?
>
> A similar concern to PR20455 from our own debugger. It much helps
> matching up the forward declaration and definition to have the parameters
> properly specified
beanz accepted this revision.
This revision is now accepted and ready to land.
Comment at: tools/driver/CMakeLists.txt:58
@@ -62,1 +57,3 @@
+if(NOT MSVC)
+ add_custom_target(install-clang
Can you change this to `if(NOT CMAKE_CONFIGURATION_TYPES)`?
With that it
eugenis updated this revision to Diff 39750.
eugenis added a comment.
Applied the same change to . No idea how I missed it.
Repository:
rL LLVM
http://reviews.llvm.org/D14409
Files:
include/istream
include/ostream
include/sstream
include/streambuf
Index: include/streambuf
==
eugenis updated this revision to Diff 39751.
eugenis added a comment.
Rebase.
Repository:
rL LLVM
http://reviews.llvm.org/D13925
Files:
include/clang/Basic/Attr.td
include/clang/Basic/AttrDocs.td
include/clang/Basic/DiagnosticSemaKinds.td
include/clang/Sema/Sema.h
lib/AST/Decl.cpp
aaron.ballman added inline comments.
Comment at: tools/driver/CMakeLists.txt:58
@@ -62,1 +57,3 @@
+if(NOT MSVC)
+ add_custom_target(install-clang
beanz wrote:
> Can you change this to `if(NOT CMAKE_CONFIGURATION_TYPES)`?
>
> With that it won't create the insta
EricWF added a comment.
No changes are needed in ``?
Repository:
rL LLVM
http://reviews.llvm.org/D14409
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
> On Nov 9, 2015, at 2:07 PM, Aaron Ballman wrote:
>
> aaron.ballman added inline comments.
>
>
> Comment at: tools/driver/CMakeLists.txt:58
> @@ -62,1 +57,3 @@
>
> +if(NOT MSVC)
> + add_custom_target(install-clang
>
> beanz wrote:
>> Can you change this to `
mgehre created this revision.
mgehre added reviewers: alexfh, sbenza, bkramer, aaron.ballman.
mgehre added a subscriber: cfe-commits.
The current matcher is
implicitCastExpr(unless(hasParent(explicitCastExpr(
but the AST in the bug is
`-CXXStaticCastExpr 0x2bb64f8 'void *const *'
static_c
echristo accepted this revision.
echristo added a comment.
This revision is now accepted and ready to land.
Sure. You should probably work on the verification side of things as well if
you expect to use this more.
-eric
http://reviews.llvm.org/D14307
Author: cbieneman
Date: Mon Nov 9 16:43:10 2015
New Revision: 252523
URL: http://llvm.org/viewvc/llvm-project?rev=252523&view=rev
Log:
[CMake] Support passing CMAKE_VERBOSE_MAKEFILE through to bootstrap builds.
This option enables full verbosity in recursive CMake builds.
Modified:
cfe/trun
probinson added a comment.
In http://reviews.llvm.org/D14354#285546, @dblaikie wrote:
> Or Tamas can write the tuning patch - it seems like it'd be relatively
> straightforward. Perhaps you can give an idea of what you think it'd look
> like, Paul (what the command line syntax would be, etc)
zaks.anna added a subscriber: zaks.anna.
zaks.anna added a comment.
Removed myself as a reviewer and added Argyrios instead.
http://reviews.llvm.org/D14506
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman
echristo added a comment.
I think the basic idea on how to do this is to make sure all of the information
is there from the front end and have the backend select how to emit based on
the code gen flag for this. libLTO is terrible for this, but the schemes
working for lld are showing progress in
eugenis added a comment.
is an interesting case.
There are no cases of visibility attribute present on an out-of-class
definition but missing on an in-class declaration, so nothing needs to be done
for a switch to internal_linkage.
Setting hidden visibility on an "extern template" method is pr
echristo added a comment.
My preference for this sort of thing would be an enum of failure reasons, but I
guess this is ok for now. One inline request.
-eric
Comment at: lib/Basic/Targets.cpp:3978-3981
@@ +3977,6 @@
+ bool &HasSizeMismatch)
rizsotto.mailinglist added a comment.
the latest update has the following improvements:
- makes it work without installation.
- pass the `SATestBuild.py` test harness against different projects with
`--strictness 2` arguments. (coreutils-8.23, gawk-4.1.1, make-4.0,
openssl-1.0.0s, patch-2.7.4,
Author: joerg
Date: Mon Nov 9 17:39:45 2015
New Revision: 252545
URL: http://llvm.org/viewvc/llvm-project?rev=252545&view=rev
Log:
Reorganise CPU handling for Sparc. When using -mcpu=v9 and co, __sparcv8
is not defined for 32bit mode, but __sparcv9 is. Pass down the correct
-target-cpu flags to t
| Why is matching by name insufficient/not correct?
I'm told we look at the mangled names in the ELF symbol table, demangle them,
and look in the DWARF for the corresponding types. Now, the mangled name (for
predefined types in particular) provides a type description, not the
name-as-emitted-by
vsk created this revision.
vsk added a reviewer: t.p.northover.
vsk added a subscriber: cfe-commits.
When adding profiling instrumentation, use `libclang_rt.profile_tvos.a` for
TVOS targets and `libclang_rt.profile_watchos.a` for WatchOS targets.
I've also fixed up a comment and added an assert(
On Mon, Nov 9, 2015 at 3:55 PM, Robinson, Paul <
paul_robin...@playstation.sony.com> wrote:
> | Why is matching by name insufficient/not correct?
>
> I'm told we look at the mangled names in the ELF symbol table, demangle
> them, and look in the DWARF for the corresponding types.
>
Not quite sure
t.p.northover accepted this revision.
t.p.northover added a comment.
This revision is now accepted and ready to land.
This looks fine to me. Thanks for working on it!
Tim.
http://reviews.llvm.org/D14521
___
cfe-commits mailing list
cfe-commits@list
Author: vedantk
Date: Mon Nov 9 18:20:34 2015
New Revision: 252558
URL: http://llvm.org/viewvc/llvm-project?rev=252558&view=rev
Log:
[Driver] Use platform-appropriate profiling libraries for WatchOS, TVOS
When adding profiling instrumentation, use libclang_rt.profile_tvos.a
for TVOS targets and
Author: joerg
Date: Mon Nov 9 18:23:12 2015
New Revision: 252562
URL: http://llvm.org/viewvc/llvm-project?rev=252562&view=rev
Log:
Use the normal switch over getArch() approach and not a long if chain.
Modified:
cfe/trunk/lib/Driver/Tools.cpp
Modified: cfe/trunk/lib/Driver/Tools.cpp
URL:
h
Author: joerg
Date: Mon Nov 9 18:40:29 2015
New Revision: 252571
URL: http://llvm.org/viewvc/llvm-project?rev=252571&view=rev
Log:
Use the generic Sparc CPU handling for Linux, FreeBSD and OpenBSD, too.
This currently changes the default toward the more historic -Av8/-Av9,
but as discussed with J
| when/where/why are types acquired from the mangled names of ELF symbols,
rather than from corresponding DWARF?
Pete, can you help me out here? David seems to want an ironclad case for not
being able to do something any other way, before he will let me put the
template type parameters on the
ahatanak updated this revision to Diff 39774.
ahatanak added a comment.
Added comments to clarify what the code is doing.
http://reviews.llvm.org/D13834
Files:
include/clang/Basic/DiagnosticSemaKinds.td
include/clang/Basic/TargetInfo.h
lib/Basic/Targets.cpp
lib/Sema/SemaDecl.cpp
test/
On Mon, Nov 9, 2015 at 5:08 PM, Robinson, Paul <
paul_robin...@playstation.sony.com> wrote:
> | when/where/why are types acquired from the mangled names of ELF
> symbols, rather than from corresponding DWARF?
>
>
>
> Pete, can you help me out here? David seems to want an ironclad case for
> not b
rsmith added inline comments.
Comment at: lib/Sema/SemaCXXScopeSpec.cpp:775-787
@@ +774,15 @@
+if (ClassTemplateDecl *CTD = Found.getAsSingle()) {
+ TemplateParameterList *TPL = CTD->getTemplateParameters();
+ assert(TPL && "NULL template parameter list");
+ std
Eugene.Zelenko updated the summary for this revision.
Eugene.Zelenko added a reviewer: aaron.ballman.
Eugene.Zelenko updated this revision to Diff 39778.
Eugene.Zelenko added a comment.
Synchronize with current code.
Repository:
rL LLVM
http://reviews.llvm.org/D13759
Files:
include/clang/A
Author: ctopper
Date: Mon Nov 9 23:08:00 2015
New Revision: 252585
URL: http://llvm.org/viewvc/llvm-project?rev=252585&view=rev
Log:
[X86] Fix bad intrinsic header comment. NFC.
Modified:
cfe/trunk/lib/Headers/__wmmintrin_pclmul.h
Modified: cfe/trunk/lib/Headers/__wmmintrin_pclmul.h
URL:
h
Author: ctopper
Date: Mon Nov 9 23:08:08 2015
New Revision: 252587
URL: http://llvm.org/viewvc/llvm-project?rev=252587&view=rev
Log:
[X86] Use setzero instead of set1(0) in a few places in intrinsic headers.
Modified:
cfe/trunk/lib/Headers/avx2intrin.h
cfe/trunk/lib/Headers/emmintrin.h
Author: ctopper
Date: Mon Nov 9 23:08:05 2015
New Revision: 252586
URL: http://llvm.org/viewvc/llvm-project?rev=252586&view=rev
Log:
[X86] Remove temporary variables from macros in x86 intrinsic headers. Prevents
duplicate names appearing from multiple macro expansions. NFC
Modified:
cfe/tr
90 matches
Mail list logo
