Author: rsmith
Date: Tue Feb 9 18:59:02 2016
New Revision: 260337
URL: http://llvm.org/viewvc/llvm-project?rev=260337&view=rev
Log:
Fix overload sets of strchr, strpbrk, strrchr, memchr and strstr from
and wcschr, wcspbrk, wcsrchr, wmemchr, and wcsstr from to
provide a const-correct overload se
On Wed, Jan 20, 2016 at 12:48 AM, Eric Fiselier via cfe-commits
wrote:
> The direction and change LGTM. Feel free to commit once the change has some
> tests.
Thanks, I updated the existing tests to no longer XFAIL Apple systems
with recent Clang and committed as r260337.
> On Fri, Jan 8, 2016 at
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=rev
> Log:
> Fix the search path for CMake files
>
> This allows to find the LL
On Tue, Feb 9, 2016 at 4:15 PM, Cong Liu wrote:
> Hi Richard,
>
> Thank you for your reply. Yes, the case I need to deal with is like what you
> said:
>>
>> If you want to make the assumption that the primary template will be
>> used for an unknown specialization, you'll need something like that
>
rsmith added a subscriber: rsmith.
rsmith added a comment.
The direction here generally seems fine to me.
Comment at: lib/Driver/ToolChain.cpp:538
@@ -536,7 +537,3 @@
ToolChain::CXXStdlibType ToolChain::GetCXXStdlibType(const ArgList &Args)
const{
- if (Arg *A = Args.getLastA
On Wed, Feb 10, 2016 at 10:32 AM, Tim Northover wrote:
> Hi Richard,
>
> On 9 February 2016 at 16:59, Richard Smith via cfe-commits
> wrote:
>> Fix overload sets of strchr, strpbrk, strrchr, memchr and strstr from
>> and wcschr, wcspbrk, wcsrchr, wmemchr, and wcsstr f
OK, got the build working. This should be fixed in r260425. Please let
me know if you're still seeing failures.
On Wed, Feb 10, 2016 at 11:59 AM, Richard Smith wrote:
> On Wed, Feb 10, 2016 at 10:32 AM, Tim Northover
> wrote:
>> Hi Richard,
>>
>> On 9 February 201
rsmith added a comment.
There seem to be two principled approaches here:
1. `DiagnosticsEngine` has a `Group` named `"everything"` (all of its interface
supports that group, and from the point of view of someone using
`DiagnosticsEngine` it behaves like any other group), or
2. There is no such
On Wed, Feb 10, 2016 at 5:05 PM, Joerg Sonnenberger via cfe-commits
wrote:
> On Wed, Feb 10, 2016 at 04:21:02PM +, Alexander Kornienko via cfe-commits
> wrote:
>> alexfh added a comment.
>>
>> Thank you for working on this check!
>>
>> I'd like to note that there is a library-side way to miti
Argh, the glibc folks broke their string.h and wchar.h prototypes in
this commit:
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=8e2e833ac4d6509b152d6b8d74d388725717c56f
On Thu, Feb 11, 2016 at 11:02 AM, Ismail Donmez wrote:
> Hi,
>
> On Wed, Feb 10, 2016 at 2:59 AM, Richard Smit
Author: rsmith
Date: Thu Feb 11 13:40:06 2016
New Revision: 260570
URL: http://llvm.org/viewvc/llvm-project?rev=260570&view=rev
Log:
Work around regression in glibc 2.22: request that glibc provides the correct
prototypes for functions that are converted into overload sets in
C++. This matches th
Feb 11, 2016 at 11:02 AM, Ismail Donmez wrote:
>> Hi,
>>
>> On Wed, Feb 10, 2016 at 2:59 AM, Richard Smith via cfe-commits
>> wrote:
>>> Author: rsmith
>>> Date: Tue Feb 9 18:59:02 2016
>>> New Revision: 260337
>>>
>>> URL: ht
Author: rsmith
Date: Thu Feb 11 13:57:37 2016
New Revision: 260577
URL: http://llvm.org/viewvc/llvm-project?rev=260577&view=rev
Log:
In C11, provide macros FLT_DECIMAL_DIG, DBL_DECIMAL_DIG, and LDBL_DECIMAL_DIG
in .
Patch by Jorge Teixeira!
Added:
cfe/trunk/test/Headers/float.c
Modified:
Thanks, I modified the test to also test C89 and C99 modes and
committed this as r260577.
On Thu, Feb 11, 2016 at 11:29 AM, Jorge Teixeira
wrote:
> Here is a revised test, which I renamed to c11-5_2_4_2_2p11.c instead
> of float.c because I am only checking a subset of what the standard
> mandate
On Thu, Feb 11, 2016 at 11:04 AM, Ben Langmuir wrote:
> FYI I changed this to early-exit again in r260563 to fix implicit builds.
> It’s not safe to keep reading if we haven’t told ReadOptionsBlock to expect
> failures, since it will itself early-exit, leaving the stream in the middle
> of a block
Author: rsmith
Date: Thu Feb 11 14:36:47 2016
New Revision: 260583
URL: http://llvm.org/viewvc/llvm-project?rev=260583&view=rev
Log:
XFAIL this test for PPC64 for now. It looks like the *_MIN_EXP macros are in
fact defined incorrectly for that target.
Modified:
cfe/trunk/test/Headers/float.c
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
LGTM
Please also add a test that a `-Weverything` on the command line can be
overridden by a `#pragma clang diagnostic ignored "-Weverything"`, even for a
diagnostic with no warning flag. (Th
Author: rsmith
Date: Thu Feb 11 16:18:10 2016
New Revision: 260610
URL: http://llvm.org/viewvc/llvm-project?rev=260610&view=rev
Log:
Remove bogus check that larger floating point types have smaller minimum
normalized exponents. That's not true for double versus double double.
Modified:
cfe/tr
/usr/include/string.h:81:1: error: unknown type name
> '__extern_always_inline'
> __extern_always_inline void *
> ^
> /usr/include/string.h:81:24: error: expected unqualified-id
> __extern_always_inline void *
>^
>
> On Thu, Feb 11, 2016 at
Author: rsmith
Date: Thu Feb 11 17:51:02 2016
New Revision: 260622
URL: http://llvm.org/viewvc/llvm-project?rev=260622&view=rev
Log:
Instead of asking glibc to provide correct C++ signatures for
functions, ask it whether it did provide them after the fact. Some versions of
glibc fail to compile i
;> /usr/include/string.h:81:1: error: unknown type name
>> '__extern_always_inline'
>> __extern_always_inline void *
>> ^
>> /usr/include/string.h:81:24: error: expected unqualified-id
>> __extern_always_inline void *
>>^
>>
On Thu, Feb 11, 2016 at 3:42 PM, Jorge Teixeira
wrote:
> Richard,
>
> Thanks and got it re: test filename and hosted mode.
>
> 1) AFAIK, C89/C90 does not define DECIMAL_DIG, so here is a revised
> patch, with minor comment modifications to explicitly list the (draft)
> normative references that I
Author: rsmith
Date: Thu Feb 11 19:15:33 2016
New Revision: 260639
URL: http://llvm.org/viewvc/llvm-project?rev=260639&view=rev
Log:
: do not define DECIMAL_DIG in -std=c89 mode; this macro was added in
C99.
Patch by Jorge Teixeira!
Modified:
cfe/trunk/lib/Headers/float.h
cfe/trunk/test
On Thu, Feb 11, 2016 at 4:53 PM, Jorge Teixeira
wrote:
>> You'll also need to change to only provide DECIMAL_DIG in C99
>> onwards.
> Done!
>
>> All of our -std versions are that standard plus applicable Defect
>> Reports. So -std=c89 includes TC1 and TC2, but not Amendment 1 (we
>> have -std=c9
rsmith added a subscriber: rsmith.
Comment at: lib/Lex/PPMacroExpansion.cpp:1438-1440
@@ -1432,3 +1437,5 @@
- // Get the module name.
- PP.LexNonComment(Tok);
+ // It is possible that the macro invocation has a multiple-nested argument,
+ // for example: __has_featureX))
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
LGTM, thanks!
http://reviews.llvm.org/D16973
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listin
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
LGTM
Comment at: lib/AST/StmtPrinter.cpp:1253-1256
@@ -1252,2 +1252,6 @@
default:
+// A character literal might be sign-extended, which
+// would result in an inval
rsmith added a comment.
In http://reviews.llvm.org/D16012#351821, @aaron.ballman wrote:
> Ping. Richard, I think we've provided justification that warrants this
> functionality (mostly for clang-tidy checks).
I am not convinced. This flag does not seem to make sense for `StringLiteral`,
becau
rsmith added a comment.
If a patch is LGTM'd and some small change is requested at the same time, that
typically means "LGTM once you make the following changes, which I trust you to
make with no further pre-commit review". If you'd prefer more pre-commit
review, of course, that's fine too.
In
u have in mind calling 'clang::Sema::MergeVarDeclTypes'
>>> or we to just "duplicate" the logic in
>>> clang::ASTDeclReader::mergeRedeclarable?
>>>
>>
>> It's not safe to call into Sema while we're in a partially-deserialized
On Tue, Feb 16, 2016 at 10:24 AM, H.J. Lu wrote:
>
> On Fri, Feb 12, 2016 at 11:39 AM, H.J. Lu wrote:
> > On Fri, Feb 12, 2016 at 6:58 AM, Matthijs van Duin
> > wrote:
> >> On 11 February 2016 at 16:31, H.J. Lu wrote:
> >>> struct A {
> >>> static void foo (void) ();
> >>> static int xxx;
> >>>
On Tue, Feb 16, 2016 at 12:25 PM, H.J. Lu wrote:
> On Tue, Feb 16, 2016 at 12:22 PM, Richard Smith wrote:
>> On Tue, Feb 16, 2016 at 10:24 AM, H.J. Lu wrote:
>>>
>>> On Fri, Feb 12, 2016 at 11:39 AM, H.J. Lu wrote:
>>> > On Fri, Feb 12, 2016 at 6:58 AM, Matthijs van Duin
>>> > wrote:
>>> >> On
On Tue, Feb 16, 2016 at 1:10 PM, H.J. Lu wrote:
> On Tue, Feb 16, 2016 at 1:02 PM, Richard Smith wrote:
>> On Tue, Feb 16, 2016 at 12:25 PM, H.J. Lu wrote:
>>> On Tue, Feb 16, 2016 at 12:22 PM, Richard Smith
>>> wrote:
On Tue, Feb 16, 2016 at 10:24 AM, H.J. Lu wrote:
>
> On Fri,
On Tue, Feb 16, 2016 at 1:21 PM, H.J. Lu wrote:
> On Tue, Feb 16, 2016 at 1:15 PM, Richard Smith wrote:
>> On Tue, Feb 16, 2016 at 1:10 PM, H.J. Lu wrote:
>>> On Tue, Feb 16, 2016 at 1:02 PM, Richard Smith
>>> wrote:
On Tue, Feb 16, 2016 at 12:25 PM, H.J. Lu wrote:
> On Tue, Feb 16,
rsmith added a comment.
The direction here looks fine to me.
Comment at: lib/Sema/SemaInit.cpp:369
@@ +368,3 @@
+ ND = dyn_cast(ND->getParent())) {
+if (S.getStdNamespace()->InEnclosingNamespaceSetOf(ND))
+ return true;
I wonder if this recursion
On Tue, Feb 16, 2016 at 1:48 PM, H.J. Lu wrote:
> On Tue, Feb 16, 2016 at 1:45 PM, Richard Smith wrote:
>> On Tue, Feb 16, 2016 at 1:21 PM, H.J. Lu wrote:
>>> On Tue, Feb 16, 2016 at 1:15 PM, Richard Smith
>>> wrote:
On Tue, Feb 16, 2016 at 1:10 PM, H.J. Lu wrote:
> On Tue, Feb 16, 2
Author: rsmith
Date: Tue Feb 16 18:04:04 2016
New Revision: 261034
URL: http://llvm.org/viewvc/llvm-project?rev=261034&view=rev
Log:
Improve diagnostics for ill-formed literal operator declarations.
Patch by Erik Pilkington!
Modified:
cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td
rsmith closed this revision.
rsmith added a comment.
Committed as r261034.
http://reviews.llvm.org/D16930
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: rsmith
Date: Wed Feb 17 15:52:44 2016
New Revision: 261161
URL: http://llvm.org/viewvc/llvm-project?rev=261161&view=rev
Log:
[modules] Cache 'acceptable decl' lookups for namespaces. In projects with
thousands of modules, each of which declares the same namespace, linearly
scanning the red
;
>>> That example actually reveals another problem: we really don't want the
>>> completed type to be visible unless there is a visible declaration with the
>>> completed type. That suggests that propagating the type across the
>>> redeclaration chain may be
>> I am not sure where else to put this logic if not in isSameEntity...
>>> Could you point me to a better place?
>>> --Vassil
>>>
>>> On 05/02/16 19:56, Richard Smith wrote:
>>>
>>> On Fri, Feb 5, 2016 at 7:04 AM, Vassil Vassilev
>>> w
rsmith added inline comments.
Comment at: include/clang/AST/DeclCXX.h:405-416
@@ -404,1 +404,14 @@
+enum AllowConstDefInitKind {
+ ACDI_Unknown,
+ ACDI_Yes,
+ ACDI_No,
+};
+unsigned AllowConstDefaultInit : 3;
+AllowConstDefInitKind allowConstDef
rsmith added a subscriber: rsmith.
rsmith accepted this revision.
rsmith added a reviewer: rsmith.
rsmith added a comment.
This revision is now accepted and ready to land.
LGTM
http://reviews.llvm.org/D17405
___
cfe-commits mailing list
cfe-commits@
On Thu, Feb 18, 2016 at 6:35 AM, Michael Matz wrote:
> Hi,
>
> On Tue, 16 Feb 2016, H.J. Lu wrote:
>
>> Here is the new definition:
>>
>> An empty type is a type where it and all of its subobjects (recursively)
>> are of class, structure, union, or array type. No memory slot nor
>> register shoul
rsmith added inline comments.
Comment at: lib/AST/DeclCXX.cpp:396-425
@@ -394,1 +395,32 @@
+bool CXXRecordDecl::allowConstDefaultInitSlow() const {
+ assert(getDefinition() && "only call this on completed records");
+ if (hasUserProvidedDefaultConstructor()) {
+data().setA
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
Thanks, LGTM!
If it's convenient, it'd be nice to check in a new cxx_dr_status.html without
your change first, as a separate commit, so it's more obvious which parts of
the file are changed b
This was already fixed in r261285.
On Thu, Feb 18, 2016 at 4:31 PM, Oleksiy Vyalov via cfe-commits
wrote:
> ovyalov added a subscriber: ovyalov.
> ovyalov added a comment.
>
> It seems this CL broke LLDB CMake build bot -
> http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/
On Fri, Feb 19, 2016 at 5:35 AM, Michael Matz wrote:
> Hi,
>
> On Thu, 18 Feb 2016, Richard Smith wrote:
>
>> >> An empty type is a type where it and all of its subobjects
>> >> (recursively) are of class, structure, union, or array type. No
>> >> memory slot nor register should be used to pass o
rsmith added a subscriber: rsmith.
rsmith added a comment.
Do they treat this as a keyword or as a predefined macro? (Does `#ifdef` think
`static_assert` is defined?)
http://reviews.llvm.org/D17444
___
cfe-commits mailing list
cfe-commits@lists.llv
On Thu, Feb 18, 2016 at 5:52 PM, Nico Weber via cfe-commits
wrote:
> Author: nico
> Date: Thu Feb 18 19:52:46 2016
> New Revision: 261297
>
> URL: http://llvm.org/viewvc/llvm-project?rev=261297&view=rev
> Log:
> Implement the likely resolution of core issue 253.
>
> C++11 requires const objects to
Author: rsmith
Date: Fri Feb 19 16:25:36 2016
New Revision: 261372
URL: http://llvm.org/viewvc/llvm-project?rev=261372&view=rev
Log:
[modules] Flatten -fmodule-name= and -fmodule-implementation-of= into a single
option. Previously these options could both be used to specify that you were
compiling
Author: rsmith
Date: Fri Feb 19 16:43:58 2016
New Revision: 261373
URL: http://llvm.org/viewvc/llvm-project?rev=261373&view=rev
Log:
[modules] Do less scanning of macro definition chains when computing the set of
exported module macros outside local submodule visibility mode. Related to
PR24667.
rsmith added inline comments.
Comment at: lib/CodeGen/CGVTables.cpp:904-919
@@ -900,5 +903,18 @@
+
+bool CodeGenModule::IsBitSetBlacklistedRecord(const CXXRecordDecl *RD) {
+ std::string TypeName = RD->getQualifiedNameAsString();
+ auto isInBlacklist = [&](const SanitizerBlackli
On Fri, Feb 19, 2016 at 4:16 PM, Eric Fiselier via cfe-commits
wrote:
> Author: ericwf
> Date: Fri Feb 19 18:16:41 2016
> New Revision: 261381
>
> URL: http://llvm.org/viewvc/llvm-project?rev=261381&view=rev
> Log:
> Add stdbool.h wrapper for libc++
>
> Summary:
> According to the C++ standard is
On Fri, Feb 19, 2016 at 4:41 PM, Nico Weber wrote:
> On Fri, Feb 19, 2016 at 4:29 PM, Richard Smith via cfe-commits
> wrote:
>>
>> On Thu, Feb 18, 2016 at 5:52 PM, Nico Weber via cfe-commits
>> wrote:
>> > Author: nico
>> > Date: Thu Feb 18 19:52:46 20
On 20 Feb 2016 6:54 p.m., "H.J. Lu" wrote:
>
> On Sat, Feb 20, 2016 at 4:57 PM, Matthijs van Duin
> wrote:
> > On 20 February 2016 at 23:35, H.J. Lu wrote:
> >> Can a compiler tell if a copy constructor or destructor is trivial
> >> from the class declaration without function body?
> >
> > Yes,
On 20 Feb 2016 10:01 p.m., "H.J. Lu" wrote:
>
> On Sat, Feb 20, 2016 at 9:47 PM, Richard Smith
wrote:
> > On 20 Feb 2016 6:54 p.m., "H.J. Lu" wrote:
> >>
> >> On Sat, Feb 20, 2016 at 4:57 PM, Matthijs van Duin
> >> wrote:
> >> > On 20 February 2016 at 23:35, H.J. Lu wrote:
> >> >> Can a compil
On 21 Feb 2016 6:29 p.m., "Faisal Vali via cfe-commits" <
cfe-commits@lists.llvm.org> wrote:
>
> Author: faisalv
> Date: Sun Feb 21 20:24:29 2016
> New Revision: 261506
>
> URL: http://llvm.org/viewvc/llvm-project?rev=261506&view=rev
> Log:
> Fix PR24473 : Teach clang to remember to substitute into
On 21 Feb 2016 8:21 p.m., "Faisal Vali" wrote:
>
> On Sun, Feb 21, 2016 at 10:06 PM, Richard Smith
wrote:
> > On 21 Feb 2016 6:29 p.m., "Faisal Vali via cfe-commits"
> > wrote:
> >>
> >> Author: faisalv
> >> Date: Sun Feb 21 20:24:29 2016
> >> New Revision: 261506
> >>
> >> URL: http://llvm.org/
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
LGTM for trunk and 3.8.
http://reviews.llvm.org/D17507
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mail
gt;>>> On Fri, Feb 19, 2016 at 4:41 PM, Nico Weber wrote:
>>>> > On Fri, Feb 19, 2016 at 4:29 PM, Richard Smith via cfe-commits
>>>> > wrote:
>>>> >>
>>>> >> On Thu, Feb 18, 2016 at 5:52 PM, Nico Weber via cfe-commits
>>
Author: rsmith
Date: Tue Feb 23 17:20:51 2016
New Revision: 261705
URL: http://llvm.org/viewvc/llvm-project?rev=261705&view=rev
Log:
PR24667: fix quadratic runtime if textually-included modular headers define
large numbers of macros.
Modified:
cfe/trunk/include/clang/Lex/Preprocessor.h
c
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
OK, let's go with this as a starting point. We don't need to get
maximally-precise type information in the first iteration.
http://reviews.llvm.org/D16821
_
Calling getMostRecentDecl seems like a slightly fragile way to avoid
iterator invalidation here. Instead...
@@ -214,6 +212,19 @@ namespace clang {
unsigned I = Record.size();
Record.push_back(0);
+ auto &Specializations = Common->Specializations;
+ auto &&PartialSpecializa
On Tue, Feb 23, 2016 at 8:28 AM, H.J. Lu wrote:
> On Tue, Feb 23, 2016 at 8:15 AM, Michael Matz wrote:
>> Hi,
>>
>> On Tue, 23 Feb 2016, H.J. Lu wrote:
>>
>>> I thought
>>>
>>> ---
>>> An empty type is a type where it and all of its subobjects (recursively)
>>> are of class, structure, union, or
rsmith added a subscriber: rsmith.
rsmith accepted this revision.
rsmith added a reviewer: rsmith.
rsmith added a comment.
This revision is now accepted and ready to land.
Sure, it makes sense to expose this.
Comment at: docs/LanguageExtensions.rst:1743
@@ +1742,3 @@
+
+This pro
2 PM, Nico Weber
>> > wrote:
>> >>
>> >> On Fri, Feb 19, 2016 at 10:02 PM, Nico Weber
>> >> wrote:
>> >>>
>> >>> On Fri, Feb 19, 2016 at 8:01 PM, Richard Smith
>> >>> wrote:
>> >>>>
>> >>>
On Wed, Feb 24, 2016 at 8:10 AM, Vassil Vassilev wrote:
>
> On 24/02/16 02:05, Richard Smith wrote:
>>
>> Calling getMostRecentDecl seems like a slightly fragile way to avoid
>> iterator invalidation here. Instead...
>>
>> @@ -214,6 +212,19 @@ namespace clang {
>> unsigned I = Record.size(
rsmith accepted this revision.
This revision is now accepted and ready to land.
Comment at: lib/Sema/SemaTemplateInstantiateDecl.cpp:2114
@@ -2113,1 +2113,3 @@
+EnterExpressionEvaluationContext ConstantEvaluated(SemaRef,
+
Author: rsmith
Date: Wed Feb 24 15:59:10 2016
New Revision: 261781
URL: http://llvm.org/viewvc/llvm-project?rev=261781&view=rev
Log:
PR26237: Fix iterator invalidation bug that occurs if serializing
specializations of a template manages to trigger deserialization of more
specializations of the sam
On Wed, Feb 24, 2016 at 2:01 PM, Vassil Vassilev wrote:
> On 24/02/16 22:50, Richard Smith wrote:
>>
>> On Wed, Feb 24, 2016 at 8:10 AM, Vassil Vassilev
>> wrote:
>>>
>>> On 24/02/16 02:05, Richard Smith wrote:
Calling getMostRecentDecl seems like a slightly fragile way to avoid
it
A UnaryTransformType should itself be a canonical type if its BaseType
is dependent -- that is, its CanonicalType should be QualType(this,
0). It should definitely not be treated as being canonically
equivalent to its BaseType.
On Wed, Feb 24, 2016 at 2:03 PM, Vassil Vassilev wrote:
> ping...
>
>
On Wed, Feb 24, 2016 at 5:36 AM, Alexander Kornienko via cfe-commits
wrote:
> Author: alexfh
> Date: Wed Feb 24 07:36:34 2016
> New Revision: 261738
>
> URL: http://llvm.org/viewvc/llvm-project?rev=261738&view=rev
> Log:
> [clang-tidy] introduce modernize-deprecated-headers check
>
> Summary:
> Th
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
Looks terrible, please commit :)
http://reviews.llvm.org/D17619
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi
rsmith added inline comments.
Comment at: lib/Basic/Targets.cpp:2603
@@ -2599,3 +2602,3 @@
case CK_i386:
case CK_i486:
case CK_i586:
echristo wrote:
> Waiting for rsmith to comment here.
>
>
This seems fine to me.
http://reviews.llvm.org/D13980
___
On Tue, Apr 26, 2016 at 1:55 PM, Akira Hatanaka via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> ahatanak added a comment.
>
> I'm looking for a way to avoid the assert in
> Sema::PerformDependentDiagnostics that is fired when a template parameter
> doesn't have a name.
>
> In order to avoid
On Tue, Apr 26, 2016 at 3:10 PM, Adrian Prantl via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
>
> On Apr 25, 2016, at 5:34 PM, Richard Smith wrote:
>
> On Mon, Apr 25, 2016 at 1:52 PM, Adrian Prantl via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
>
>> Author: adrian
>> Date: Mon Apr
Author: rsmith
Date: Tue Apr 26 18:40:43 2016
New Revision: 267632
URL: http://llvm.org/viewvc/llvm-project?rev=267632&view=rev
Log:
PR27513: When determining which declaration to put into an exported lookup
table for a module / PCH, never map from a normal declaration of a class to an
injected-cl
rsmith added inline comments.
Comment at: lib/Sema/SemaDecl.cpp:5044-5045
@@ -5043,4 +5043,4 @@
// function template specialization, add it to the scope stack.
- if (New->getDeclName() && AddToScope &&
- !(D.isRedeclaration() && New->isInvalidDecl())) {
+ if (New->getDe
As noted in PR27337, this only occurs in one WTL sample, and we have no
evidence that it actually occurs in real code. Have you seen uses of this
in the wild? We generally don't want to add compatibility for MSVC bugs
unless there's some real-world motivation.
On Tue, Apr 26, 2016 at 2:21 AM, Dmit
rsmith added a comment.
I'm a little concerned about the possibility of this breaking uses of this
feature on platforms where Clang is the system compiler. For instance, this
pattern would be broken by your change:
// stddef.h
#include "stddef-helper.h"
// stddef-helper.h
#include_ne
ght, it's the other direction ('A *' to 'B *') that this patch is
permitting.
> On Tue, Apr 26, 2016 at 6:33 PM, Richard Smith via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
>
>> As noted in PR27337, this only occurs in one WTL sample, and we ha
On Wed, Apr 27, 2016 at 11:50 AM, Stephan T. Lavavej via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> [Dmitry Polukhin]
> > As for 'A*' to 'B*" case, indeed it is not very common and it looks like
> someone just forgot to specify 'public'.
>
> If we're doing something gratuitously nonconform
On Wed, Apr 27, 2016 at 11:52 AM, Richard Smith
wrote:
> On Wed, Apr 27, 2016 at 11:50 AM, Stephan T. Lavavej via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
>
>> [Dmitry Polukhin]
>> > As for 'A*' to 'B*" case, indeed it is not very common and it looks
>> like someone just forgot to speci
rsmith accepted this revision.
This revision is now accepted and ready to land.
Comment at: docs/ControlFlowIntegrity.rst:268-269
@@ -262,1 +267,4 @@
+Normally, the compiler will disable CFI checks for classes that do not
+have hidden LTO visibility. With this flag enabled, the
rsmith added a subscriber: rsmith.
rsmith added a comment.
I don't know how the libc++ folks feel about this, but our usual policy in
Clang and LLVM is that we don't add bogus initializers to shut up incorrect
"maybe used uninitialized" compiler warnings, because these initializers
prevent tool
rsmith accepted this revision.
This revision is now accepted and ready to land.
Comment at: include/clang/Basic/DiagnosticGroups.td:341-342
@@ +340,4 @@
+
+def ShadowFieldInConstructor : DiagGroup<"shadow-field-in-constructor">;
+def ShadowFieldInConstructorModified :
DiagGroup<"
Author: rsmith
Date: Wed Apr 27 16:57:05 2016
New Revision: 267802
URL: http://llvm.org/viewvc/llvm-project?rev=267802&view=rev
Log:
[modules] When diagnosing a missing module import, suggest adding a #include if
the current language doesn't have an import syntax and we can figure out a
suitable f
rsmith added inline comments.
Comment at: lib/Parse/ParseTemplate.cpp:151-153
@@ -149,1 +150,5 @@
+ if (isSpecialization)
+TemplateParmScopePtr->setFlags(TemplateParmScopePtr->getFlags() ^
+ Scope::TemplateParamScope);
+
Us
rsmith added a comment.
LGTM
http://reviews.llvm.org/D18271
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rsmith added a subscriber: rsmith.
Comment at: lib/Driver/ToolChains.cpp:718-720
@@ +717,5 @@
+StringRef isysroot = A->getValue();
+// Assume SDK has path: SOME_PATH/SDKs/PlatformXX.YY.sdk
+size_t BeginSDK = isysroot.rfind("SDKs/");
+size_t EndSDK = isysroot.rfind(
rsmith added inline comments.
Comment at: lib/Driver/ToolChains.cpp:722
@@ +721,3 @@
+if (BeginSDK != StringRef::npos && EndSDK != StringRef::npos) {
+ StringRef SDK = isysroot.slice(BeginSDK + 5, EndSDK);
+ if (!SDK.startswith(getPlatformFamily()))
Author: rsmith
Date: Thu Apr 28 13:26:32 2016
New Revision: 267914
URL: http://llvm.org/viewvc/llvm-project?rev=267914&view=rev
Log:
Fix use of uninitialized value exposed by r267802. Accessors of an invalid
PresumedLoc should not be called.
Modified:
cfe/trunk/include/clang/Basic/SourceLocat
rsmith added inline comments.
Comment at: include/clang/Driver/Options.td:1102
@@ -1101,1 +1101,3 @@
def Wlarge_by_value_copy_EQ : Joined<["-"], "Wlarge-by-value-copy=">,
Flags<[CC1Option]>;
+def fubsan_strip_path_components_EQ : Joined<["-"],
"fubsan-strip-path-components=">,
Author: rsmith
Date: Thu Apr 28 14:54:51 2016
New Revision: 267926
URL: http://llvm.org/viewvc/llvm-project?rev=267926&view=rev
Log:
One more fix for use of invalid PresumedLocs missed by r267914.
Modified:
cfe/trunk/lib/Basic/SourceManager.cpp
Modified: cfe/trunk/lib/Basic/SourceManager.cpp
rsmith added a comment.
You give this example:
> 343 | Loc = ConvertBackendLocation(D, Context->getSourceManager());
> I | ^
> I | ^
How does this look for a case like `p->Foo()->Bar()` (where one or both of the
calls ar
On Thu, Apr 28, 2016 at 1:14 PM, Akira Hatanaka via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> ahatanak added a comment.
>
> If I try calling Init(AnyParent, F) in Scope::setFlags, clang fails to
> compile the following code because it cannot find the definition of struct
> "foo":
>
> vo
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
LGTM, thanks!
http://reviews.llvm.org/D19175
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listin
rsmith added inline comments.
Comment at: lib/Driver/ToolChains.cpp:722-733
@@ -700,1 +721,14 @@
+
+ if (const Arg *A = Args.getLastArg(options::OPT_isysroot)) {
+StringRef isysroot = A->getValue();
+// Assume SDK has path: SOME_PATH/SDKs/PlatformXX.YY.sdk
+size_t Beg
rsmith added inline comments.
Comment at: lib/Basic/Targets.cpp:1648-1684
@@ +1647,39 @@
+
+// Match the host's types.
+PointerWidth = HostTarget->getPointerWidth(/* AddrSpace = */ 0);
+PointerAlign = HostTarget->getPointerAlign(/* AddrSpace = */ 0);
+BoolWidth = H
3101 - 3200 of 4285 matches
Mail list logo
