Author: rsmith
Date: Wed Nov 4 18:54:55 2015
New Revision: 252114
URL: http://llvm.org/viewvc/llvm-project?rev=252114&view=rev
Log:
[modules] If we're given a module file, via -fmodule-file=, for a module, but
we can't load that file due to a configuration mismatch, and implicit module
building i
Author: majnemer
Date: Wed Nov 4 19:01:47 2015
New Revision: 252115
URL: http://llvm.org/viewvc/llvm-project?rev=252115&view=rev
Log:
[Lex] Add __has_builtin support for __make_integer_seq
Differential Revision: http://reviews.llvm.org/D14349
Modified:
cfe/trunk/lib/Lex/PPMacroExpansion.cpp
This revision was automatically updated to reflect the committed changes.
Closed by commit rL252115: [Lex] Add __has_builtin support for
__make_integer_seq (authored by majnemer).
Changed prior to commit:
http://reviews.llvm.org/D14349?vs=39258&id=39294#toc
Repository:
rL LLVM
http://review
Author: majnemer
Date: Wed Nov 4 19:10:42 2015
New Revision: 252120
URL: http://llvm.org/viewvc/llvm-project?rev=252120&view=rev
Log:
Fix build break
Modified:
cfe/trunk/lib/Lex/PPMacroExpansion.cpp
Modified: cfe/trunk/lib/Lex/PPMacroExpansion.cpp
URL:
http://llvm.org/viewvc/llvm-project/c
aprantl added a comment.
Unless this is some kind of optimization that we specifically added to minimize
debug info size (I have never looked at our template support in detail), this
looks totally reasonable. Do other compilers do the same here?
http://reviews.llvm.org/D14358
__
djasper added a subscriber: djasper.
djasper accepted this revision.
djasper added a reviewer: djasper.
djasper added a comment.
This revision is now accepted and ready to land.
Looks good.
http://reviews.llvm.org/D14191
___
cfe-commits mailing list
djasper added a subscriber: djasper.
Comment at: clang-tidy/ClangTidy.cpp:46
@@ +45,3 @@
+
+#include "llvm/ADT/StringExtras.h"
+
This doesn't seem to be the right place for this #include.
Comment at: clang-tidy/ClangTidyOptions.h:89-93
@@ -86,1 +
Author: rsmith
Date: Wed Nov 4 19:30:19 2015
New Revision: 252125
URL: http://llvm.org/viewvc/llvm-project?rev=252125&view=rev
Log:
[modules] Don't merge an anonymous enum definition into a named enum definition.
Modified:
cfe/trunk/lib/Sema/SemaDecl.cpp
cfe/trunk/test/Modules/merge-enum
Author: chapuni
Date: Wed Nov 4 19:34:19 2015
New Revision: 252128
URL: http://llvm.org/viewvc/llvm-project?rev=252128&view=rev
Log:
TestModuleFileExtension.cpp: Include explicitly for fprintf(3).
Modified:
cfe/trunk/lib/Frontend/TestModuleFileExtension.cpp
Modified: cfe/trunk/lib/Frontend
probinson added a comment.
GCC 4.8.4 on Linux doesn't produce these, but DWARF 4 section 5.5.8 says a
class template instantiation is just like the equivalent non-template class
entry, with the exception of the template parameter entries. I read that as
meaning an incomplete description (i.e.
eugenis added inline comments.
Comment at: test/SemaCXX/internal_linkage.cpp:23
@@ +22,3 @@
+
+__attribute__((internal_linkage)) void A::f4() {} //
expected-error{{'internal_linkage' attribute does not appear on the first
declaration of 'f4'}}
+
Btw, this trigge
This revision was automatically updated to reflect the committed changes.
Closed by commit rL252134: Make ArgumentAdjuster aware of the current file
being processed. (authored by alexfh).
Changed prior to commit:
http://reviews.llvm.org/D14191?vs=38776&id=39303#toc
http://reviews.llvm.org/D141
Author: alexfh
Date: Wed Nov 4 20:19:53 2015
New Revision: 252134
URL: http://llvm.org/viewvc/llvm-project?rev=252134&view=rev
Log:
Make ArgumentAdjuster aware of the current file being processed.
Summary:
This is needed to handle per-project configurations when adding extra
arguments in clang-t
tinti updated this revision to Diff 39304.
tinti added a comment.
Clang format.
Repository:
rL LLVM
http://reviews.llvm.org/D14184
Files:
include/clang/Driver/Options.td
include/clang/Frontend/CodeGenOptions.h
lib/CodeGen/BackendUtil.cpp
lib/Driver/Tools.cpp
lib/Frontend/CompilerIn
tinti marked 2 inline comments as done.
Comment at: lib/Frontend/CompilerInvocation.cpp:459
@@ +458,3 @@
+StringRef Value = A->getValue();
+bool Valid = llvm::StringSwitch(Value)
+ .Case("default", true)
This part of the code does not i
Author: alexfh
Date: Wed Nov 4 20:30:21 2015
New Revision: 252138
URL: http://llvm.org/viewvc/llvm-project?rev=252138&view=rev
Log:
Accommodate interface change in r252134.
Modified:
clang-tools-extra/trunk/modularize/Modularize.cpp
Modified: clang-tools-extra/trunk/modularize/Modularize.cp
On Sun, Nov 1, 2015 at 10:20 AM, Manuel Klimek wrote:
>
>
> On Fri, Oct 23, 2015 at 9:31 PM Sean Silva wrote:
>
>> On Tue, Oct 20, 2015 at 1:52 AM, Manuel Klimek wrote:
>>
>>> On Tue, Oct 20, 2015 at 10:41 AM Sean Silva
>>> wrote:
>>>
On Tue, Oct 20, 2015 at 1:38 AM, Manuel Klimek
w
On Wed, Nov 4, 2015 at 1:07 PM, Richard Smith wrote:
> On Sun, Nov 1, 2015 at 10:20 AM, Manuel Klimek wrote:
>
>> On Fri, Oct 23, 2015 at 9:31 PM Sean Silva wrote:
>>
>>> On Tue, Oct 20, 2015 at 1:52 AM, Manuel Klimek
>>> wrote:
>>>
On Tue, Oct 20, 2015 at 10:41 AM Sean Silva
wrote:
Test case?
On Wed, Nov 4, 2015 at 3:44 PM, Tamas Berghammer via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> tberghammer created this revision.
> tberghammer added a reviewer: echristo.
> tberghammer added a subscriber: cfe-commits.
>
> Add new compiler flag to enable the generation of dwar
michaelwu added inline comments.
Comment at: test/Index/symbol-visibility.c:8
@@ +7,3 @@
+// CHECK: FunctionDecl=foo1:3:47visibility=Default
+// CHECK: FunctionDecl=foo2:4:49visibility=Protected
+// CHECK: FunctionDecl=foo3:5:46visibility=Hidden
skalinichev wrote:
nwilson updated this revision to Diff 39314.
nwilson added a comment.
- Cover variadic arguments in check for no params.
- Add tests to cover variadic arguments.
http://reviews.llvm.org/D14352
Files:
include/clang/Basic/DiagnosticSemaKinds.td
lib/Sema/SemaDecl.cpp
test/CXX/concepts-ts/dcl
Author: davidxl
Date: Wed Nov 4 23:46:39 2015
New Revision: 252147
URL: http://llvm.org/viewvc/llvm-project?rev=252147&view=rev
Log:
Use profile data template file for covmap func record (NFC)
Modified:
cfe/trunk/lib/CodeGen/CoverageMappingGen.cpp
Modified: cfe/trunk/lib/CodeGen/CoverageMap
Would citing PR20455 help? It wasn't actually my primary motivation but it's
not too far off. Having the template parameters there lets you know what's
going on in the DWARF, without having to fetch and parse the name string of
every struct you come across. Actually I'm not sure parsing the n
101 - 123 of 123 matches
Mail list logo
