alexshap updated this revision to Diff 111622.
alexshap added a comment.
Skip the default binding for empty bases.
Repository:
rL LLVM
https://reviews.llvm.org/D36851
Files:
lib/StaticAnalyzer/Core/RegionStore.cpp
test/Analysis/ctor.mm
Index: test/Analysis/ctor.mm
=
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
Yeah, thanks!
After the FIXME about record layout is addressed, i guess the next action item
is to make sure the trivial constructor for the empty base is not evaluated
conservatively (doh!). Becau
sberg updated this revision to Diff 111632.
sberg added a comment.
(the original diff had inadvertently lacked full -U99 context; uploaded a
new diff)
https://reviews.llvm.org/D35562
Files:
clang/lib/AST/ExprConstant.cpp
clang/test/SemaCXX/constant-expression-cxx1y.cpp
Index: clang/t
sberg updated this revision to Diff 111634.
sberg added a comment.
(the original diff had inadvertently lacked full -U99 context; uploaded a
new diff)
https://reviews.llvm.org/D35564
Files:
clang/lib/Basic/DiagnosticIDs.cpp
clang/test/SemaCXX/warn-sysheader-macro.cpp
Index: clang/tes
ilya-biryukov added a comment.
I've also stumbled upon this crash. Here is an even simpler repro that crashes
for me (does not involve includes):
#ifndef HEADER_GUARD
#define FOO(X) int fhjdfhsdjkfhjkshfjk;
FOO(base)
struct X {
int a;
};
int test() {
X v;
v.
}
ilya-biryukov added inline comments.
Comment at: lib/Parse/Parser.cpp:519
ConsumeToken();
-
- PP.replayPreambleConditionalStack();
+ if (!PP.isCurrentLexer(nullptr))
+PP.replayPreambleConditionalStack();
ilya-biryukov wrote:
> This certainly fixes the cr
arphaman added a comment.
Ping.
Repository:
rL LLVM
https://reviews.llvm.org/D36075
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
yvvan updated this revision to Diff 111647.
yvvan marked 5 inline comments as done.
yvvan added a comment.
Review comments + solved https://bugs.llvm.org/show_bug.cgi?id=34207 issue.
I've also added the extra test for that issue
https://reviews.llvm.org/D36390
Files:
include/clang/Sema/Sema.h
arphaman accepted this revision.
arphaman added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D36777
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/c
arphaman added a comment.
FYI, this doesn't compiler after John's constant emitter refactoring (r310964)
https://reviews.llvm.org/D34992
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
rengolin added a comment.
This change seems to have nothing to do with adding core support but exposing
features from CPU names.
If this was required to "support" the new cores in Clang, why wasn't it needed
before?
If this is a new, more generic way, of getting support, shouldn't you remove
SjoerdMeijer added a comment.
"They also implement the RCpc AArch64 extension from ARMv8.3-A."
Perhaps you need to explain why a v8.2 core implements a v8.3 extension?
Comment at: lib/Driver/ToolChains/Arch/ARM.cpp:92
+ std::vector &Features
SjoerdMeijer added a comment.
Ah, my message crossed with Renato's; I only noticed it after submitting mine
(looks like we agree though:-))
https://reviews.llvm.org/D36731
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.or
samparker added a comment.
Thanks guys, I will sort my logic out.
Renato, the new Decode function is really just there to allow me to double
check the target features. This is already possible with AArch64 and it just
felt right to also be able to test ARM in the same way. I've renamed the titl
ilya-biryukov created this revision.
The crash occurs when the first token after a preamble is a macro
expansion.
Fixed by moving replayPreambleConditionalStack from Parser into
Preprocessor. It is now called right after the predefines file is
processed.
https://reviews.llvm.org/D36872
Files:
ilya-biryukov added a comment.
Here's an alternative fix that moves the `replay...` call into `Preprocessor`:
https://reviews.llvm.org/D36872.
It also fixes the crash and doesn't add invalid compiler errors about
non-matching `#ifdefs`.
https://reviews.llvm.org/D36458
__
erikjv added a comment.
Can you check if the example in https://bugs.llvm.org/show_bug.cgi?id=33574
works correctly?
Comment at: include/clang/Lex/Preprocessor.h:2029
+
private:
// Macro handling.
Duplicate "private:"
https://reviews.llvm.org/D36872
_
Author: joerg
Date: Fri Aug 18 05:57:36 2017
New Revision: 311156
URL: http://llvm.org/viewvc/llvm-project?rev=311156&view=rev
Log:
Spelling
Modified:
libcxx/trunk/include/__tree
Modified: libcxx/trunk/include/__tree
URL:
http://llvm.org/viewvc/llvm-project/libcxx/trunk/include/__tree?rev=3
samparker updated this revision to Diff 111664.
samparker added a comment.
Reverted the default cpu v8.2-a to generic, I will update
https://reviews.llvm.org/D36667 accordingly. Also fixed up the boolean issues.
https://reviews.llvm.org/D36731
Files:
lib/Driver/ToolChains/Arch/ARM.cpp
test
hamzasood added inline comments.
Comment at: lib/Parse/ParseExprCXX.cpp:1112
+ ParseScope TemplateParamScope(this, Scope::TemplateParamScope);
+ if (getLangOpts().CPlusPlus2a && Tok.is(tok::less)) {
faisalv wrote:
> We always create a template parameter scope
rengolin added a comment.
I'm happy with the patch, but I'll let @SjoerdMeijer approve.
https://reviews.llvm.org/D36731
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
JonasToth added a comment.
Did I run it over clang or athletic like that? I would be interested in an
overall output.
I added my thoughts. This check seems very valuable.
Comment at: clang-tidy/readability/FunctionCognitiveComplexityCheck.cpp:23
+ // Limit of 25 is the "upstr
erikjv added a comment.
I'll commit it, probably next Tuesday.
https://reviews.llvm.org/D35355
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
SjoerdMeijer accepted this revision.
SjoerdMeijer added a comment.
This revision is now accepted and ready to land.
Looks good to me too.
Two nits (no new review required): one is inlined, and the other one in the
summary: ARMv8.2-A => Armv8.2-A :-/
Comment at: test/Driver/
thakis added inline comments.
Comment at: lib/Driver/ToolChains/MSVC.cpp:135
+ llvm::SmallString<256> TestPath = PathEntry;
+ while(!TestPath.empty()) {
TestPath = llvm::sys::path::parent_path(TestPath);
Having a while loop going up the directo
hintonda added a comment.
Looks good, but I'll defer to the owners.
Minor nit...
Comment at: lib/Driver/ToolChains/MSVC.cpp:142
return true;
+} else if (llvm::sys::path::filename(ParentPath) == "x86ret"
+|| llvm::sys::path::filename(ParentPath) =
thakis added a comment.
Oh, I guess this is superseded by https://reviews.llvm.org/D36860 ?
https://reviews.llvm.org/D36857
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hintonda added a comment.
In https://reviews.llvm.org/D36857#845465, @thakis wrote:
> Oh, I guess this is superseded by https://reviews.llvm.org/D36860 ?
Yes, please ignore this patch -- it was just for brainstorming... ;-)
Comment at: lib/Driver/ToolChains/MSVC.cpp:135
+
thakis added a comment.
This approach looks good to me.
Is there any chance we could have a test for this? See `test/Driver` for
examples; use `python bin\llvm-lit.py
../llvm/tools/clang/test/Driver/my_test.cpp` to run a single test, `ninja
check-clang` to run all of them. You need some unix b
arphaman created this revision.
C++ allows us to reference static variables through member expressions. Prior
to this patch, non-integer static variables that were referenced using a member
expression were always emitted using lvalue loads. The old behaviour introduced
an inconsistency between
arphaman added inline comments.
Comment at: lib/Tooling/ASTDiff/ASTDiff.cpp:371
+ std::string ContextPrefix;
+ if (auto *Namespace = dyn_cast(Context))
+ContextPrefix = Namespace->getQualifiedNameAsString();
You don't need to check both `NamespaceDecl` and
Nebiroth updated this revision to Diff 111675.
Nebiroth added a comment.
Fixed more diff comments.
https://reviews.llvm.org/D36150
Files:
clangd/ClangdLSPServer.cpp
clangd/ClangdServer.cpp
clangd/ClangdServer.h
clangd/ProtocolHandlers.cpp
clangd/ProtocolHandlers.h
Index: clangd/Proto
faisalv added inline comments.
Comment at: lib/Parse/ParseExprCXX.cpp:1112
+ ParseScope TemplateParamScope(this, Scope::TemplateParamScope);
+ if (getLangOpts().CPlusPlus2a && Tok.is(tok::less)) {
hamzasood wrote:
> faisalv wrote:
> > We always create a templ
ilya-biryukov marked an inline comment as done.
ilya-biryukov added a comment.
In https://reviews.llvm.org/D36872#845364, @erikjv wrote:
> Can you check if the example in https://bugs.llvm.org/show_bug.cgi?id=33574
> works correctly?
Works correctly, added an extra test for it.
https://revie
ilya-biryukov updated this revision to Diff 111676.
ilya-biryukov added a comment.
- Removed redundant 'private' specifier.
- Added a test for https://bugs.llvm.org/show_bug.cgi?id=33574
https://reviews.llvm.org/D36872
Files:
include/clang/Lex/Preprocessor.h
lib/Lex/PPLexerChange.cpp
lib/
arphaman added a comment.
There's also a crash with the `( ... )` initializer:
namespace init_capture_undeclared_identifier {
auto a = [x = y]{};
int typo_foo;
auto b = [x (typo_boo )]{};
}
I think you should move the added correction code before `if (Init.isUsable())`
to en
hamzasood updated this revision to Diff 111681.
hamzasood added a comment.
It's Friday, so hopefully this is good to go.
I don't have svn access, so could someone commit this for me?
I've updated the patch so that it can be applied successfully to the latest
revision.
I've also renamed cxx1z->cx
arphaman added inline comments.
Comment at: clangd/ClangdServer.cpp:296
+
+ const int sourceSize = sizeof(DEFAULT_SOURCE_EXTENSIONS) /
sizeof(DEFAULT_SOURCE_EXTENSIONS[0]);
+ const int headerSize = sizeof(DEFAULT_HEADER_EXTENSIONS) /
sizeof(DEFAULT_HEADER_EXTENSIONS[0]);
anatol.pomozov added a comment.
Tim, had you chance to look at this patch?
Repository:
rL LLVM
https://reviews.llvm.org/D36272
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: krobelus
Date: Fri Aug 18 09:34:15 2017
New Revision: 311172
URL: http://llvm.org/viewvc/llvm-project?rev=311172&view=rev
Log:
[clang-diff] Fix some errors and inconsistencies
Fix to the computation of the rightmost descendant.
Prevents root nodes from being mapped if they are already ma
Author: krobelus
Date: Fri Aug 18 09:34:22 2017
New Revision: 311173
URL: http://llvm.org/viewvc/llvm-project?rev=311173&view=rev
Log:
[clang-diff] Add commandline arguments.
Summary:
Support command line options for build path and extra arguments
This emulates the options accepted by clang tools
This revision was automatically updated to reflect the committed changes.
Closed by commit rL311173: [clang-diff] Add commandline arguments. (authored by
krobelus).
Changed prior to commit:
https://reviews.llvm.org/D36177?vs=109502&id=111690#toc
Repository:
rL LLVM
https://reviews.llvm.org/
This revision was automatically updated to reflect the committed changes.
Closed by commit rL311172: [clang-diff] Fix some errors and inconsistencies
(authored by krobelus).
Repository:
rL LLVM
https://reviews.llvm.org/D36176
Files:
cfe/trunk/include/clang/Tooling/ASTDiff/ASTDiff.h
cfe/tr
ioeric added a comment.
This is great work and definitely a lot to digest! ;) Some high-level comments
for the first round.
In general, I really appreciate the high-level interfaces; I am just a bit
concerned about the implementation which is a bit hard to follow at this point,
especially with
alexshap added inline comments.
Comment at: include/clang/Basic/DiagnosticOr.h:40
+ /// diagnostic.
+ DiagnosticOr(PartialDiagnosticAt Diagnostic) : HasDiagnostic(true) {
+new (getDiagnosticStorage()) PartialDiagnosticAt(std::move(Diagnostic));
explicit ?
johannes updated this revision to Diff 111695.
johannes added a comment.
only use Enums as namespace prefix in C++11
https://reviews.llvm.org/D36187
Files:
include/clang/Tooling/ASTDiff/ASTDiff.h
lib/Tooling/ASTDiff/ASTDiff.cpp
test/Tooling/clang-diff-ast.cpp
test/Tooling/clang-diff-bas
johannes added inline comments.
Comment at: lib/Tooling/ASTDiff/ASTDiff.cpp:371
+ std::string ContextPrefix;
+ if (auto *Namespace = dyn_cast(Context))
+ContextPrefix = Namespace->getQualifiedNameAsString();
arphaman wrote:
> You don't need to check both `N
Author: kzhuravl
Date: Fri Aug 18 10:29:07 2017
New Revision: 311178
URL: http://llvm.org/viewvc/llvm-project?rev=311178&view=rev
Log:
AMDGPU: Rename r600-mcpu.cl->amdgpu-mcpu.cl
Added:
cfe/trunk/test/Driver/amdgpu-mcpu.cl
Removed:
cfe/trunk/test/Driver/r600-mcpu.cl
Added: cfe/trunk/test
rsmith added inline comments.
Comment at: lib/CodeGen/CGExprComplex.cpp:156
+auto Constant = tryEmitDeclRefOrMemberExprAsConstant(
+ME, ME->getMemberDecl(), /*AllowSideEffects=*/true);
+if (Constant) {
If we can (correctly) allow side-effects in t
morehouse created this revision.
Herald added subscribers: kristof.beyls, aemerson.
Arguments can be specified after -ignore_remaining_args=1 to modify
the compiler invocation. For example, the following command-line
will fuzz LLVM with a custom optimization level and target triple:
clang-prot
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
I'd like to also see a testcase for the situation where we trigger the emission
of a declaration with an `available_externally` definition and then find we
need to promote it to a "full" defin
rsmith added a comment.
Thanks, will you need someone to commit this for you?
Comment at: include/clang/Basic/DiagnosticGroups.td:794
+// earlier C++ versions.
+def CXX20 : DiagGroup<"c++20-extensions">;
+
We've historically avoided using precise year numbers u
vitalybuka accepted this revision.
vitalybuka added inline comments.
This revision is now accepted and ready to land.
Comment at: clang/tools/clang-fuzzer/ExampleClangProtoFuzzer.cpp:33
+ for (I = 1; I < *argc; I++)
+if (llvm::StringRef((*argv)[I]).equals("-ignore_remaining_
Author: alexshap
Date: Fri Aug 18 11:20:43 2017
New Revision: 311182
URL: http://llvm.org/viewvc/llvm-project?rev=311182&view=rev
Log:
[analyzer] Fix modeling of constructors
This diff fixes analyzer's crash (triggered assert) on the newly added test
case.
The assert being discussed is assert(!B
This revision was automatically updated to reflect the committed changes.
Closed by commit rL311182: [analyzer] Fix modeling of constructors (authored by
alexshap).
Changed prior to commit:
https://reviews.llvm.org/D36851?vs=111622&id=111711#toc
Repository:
rL LLVM
https://reviews.llvm.org/
morehouse updated this revision to Diff 111713.
morehouse added a comment.
- Refactor and use strcmp.
https://reviews.llvm.org/D36882
Files:
clang/tools/clang-fuzzer/ExampleClangProtoFuzzer.cpp
Index: clang/tools/clang-fuzzer/ExampleClangProtoFuzzer.cpp
=
Author: morehouse
Date: Fri Aug 18 11:34:39 2017
New Revision: 311185
URL: http://llvm.org/viewvc/llvm-project?rev=311185&view=rev
Log:
[clang-proto-fuzzer] Allow user-specified compiler arguments.
Summary:
Arguments can be specified after -ignore_remaining_args=1 to modify
the compiler invocatio
This revision was automatically updated to reflect the committed changes.
Closed by commit rL311185: [clang-proto-fuzzer] Allow user-specified compiler
arguments. (authored by morehouse).
Changed prior to commit:
https://reviews.llvm.org/D36882?vs=111713&id=111715#toc
Repository:
rL LLVM
ht
vsapsai updated this revision to Diff 111716.
vsapsai added a comment.
Address review comments.
Moved `CorrectDelayedTyposInExpr` according to review comment and added
mentioned test case.
Removed the assertion as it didn't catch the expression
ParenListExpr 0x7f8297060e40 'NULL TYPE'
`-Typ
Author: morehouse
Date: Fri Aug 18 11:43:30 2017
New Revision: 311186
URL: http://llvm.org/viewvc/llvm-project?rev=311186&view=rev
Log:
[SanitizerCoverage] Add stack depth tracing instrumentation.
Summary:
Augment SanitizerCoverage to insert maximum stack depth tracing for
use by libFuzzer. The
This revision was automatically updated to reflect the committed changes.
Closed by commit rL311186: [SanitizerCoverage] Add stack depth tracing
instrumentation. (authored by morehouse).
Changed prior to commit:
https://reviews.llvm.org/D36839?vs=111570&id=111718#toc
Repository:
rL LLVM
htt
Rakete updated this revision to Diff 111719.
Rakete marked 3 inline comments as done.
Rakete added a comment.
Yes please.
https://reviews.llvm.org/D36855
Files:
include/clang/Basic/DiagnosticGroups.td
include/clang/Basic/DiagnosticSemaKinds.td
lib/Sema/SemaExprCXX.cpp
test/S
STL_MSFT added a comment.
Ping?
https://reviews.llvm.org/D36503
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: epilk
Date: Fri Aug 18 13:20:56 2017
New Revision: 311191
URL: http://llvm.org/viewvc/llvm-project?rev=311191&view=rev
Log:
[Sema] Don't emit -Wunguarded-availability for switch cases
This made it awkward to switch over an enum where some entries
are partial and is unlikley to catch any b
This revision was automatically updated to reflect the committed changes.
Closed by commit rL311191: [Sema] Don't emit -Wunguarded-availability for
switch cases (authored by epilk).
Changed prior to commit:
https://reviews.llvm.org/D36777?vs=111528&id=111727#toc
Repository:
rL LLVM
https://
lebedev.ri created this revision.
lebedev.ri added a project: clang-tools-extra.
Herald added subscribers: xazax.hun, JDevlieghere.
Currently, there is two configured prefixes: `CHECK-FIXES` and `CHECK-MESSAGES`
`CHECK-MESSAGES` checks that there are no test output lines with
`warning:|error:`, w
Merged to 5.0 in r311194.
On Thu, Aug 17, 2017 at 12:35 PM, Richard Smith via cfe-commits
wrote:
> Author: rsmith
> Date: Thu Aug 17 12:35:50 2017
> New Revision: 35
>
> URL: http://llvm.org/viewvc/llvm-project?rev=35&view=rev
> Log:
> PR34161: support evaluation of 'void()' expressions i
lebedev.ri updated this revision to Diff 111731.
lebedev.ri marked 2 inline comments as done.
lebedev.ri added a comment.
Address review notes:
- `constexpr const` -> `constexpr`
- moved `test/clang-tidy/check_clang_tidy.py` changes into
https://reviews.llvm.org/D36892
Repository:
rL LLVM
h
lebedev.ri added a comment.
In https://reviews.llvm.org/D36836#845417, @JonasToth wrote:
> Did I run it over clang or athletic like that? I would be interested in an
> overall output.
No, not yet.
But if it is to be enabled for LLVM repos by default, then the `Threshold` will
most definitivel
Eric, sounds like there are no known issues with this, then. What do
you think about merging it?
On Thu, Aug 17, 2017 at 5:30 PM, Petr Hosek wrote:
> Confirmed, this changes is harmless. The failure we're seeing seems to have
> been introduced by Clang in r310983, sorry about the confusion.
>
> O
I'm OK merging it then.
On Fri, Aug 18, 2017 at 2:32 PM, Hans Wennborg wrote:
> Eric, sounds like there are no known issues with this, then. What do
> you think about merging it?
>
> On Thu, Aug 17, 2017 at 5:30 PM, Petr Hosek wrote:
> > Confirmed, this changes is harmless. The failure we're se
eduardo-elizondo added inline comments.
Comment at: include/clang/Lex/PreprocessorOptions.h:165
public:
- PreprocessorOptions() : UsePredefines(true), DetailedRecord(false),
+ PreprocessorOptions() : PPTimer("preprocessor", "Preprocessing"),
+ UsePrede
Author: hans
Date: Fri Aug 18 13:59:31 2017
New Revision: 311197
URL: http://llvm.org/viewvc/llvm-project?rev=311197&view=rev
Log:
Merging r309474:
r309474 | smeenai | 2017-07-28 19:54:41 -0700 (Fri, 28 Jul 2017) | 9 lines
[
r311197. Thanks!
On Fri, Aug 18, 2017 at 1:43 PM, Eric Fiselier wrote:
> I'm OK merging it then.
>
> On Fri, Aug 18, 2017 at 2:32 PM, Hans Wennborg wrote:
>>
>> Eric, sounds like there are no known issues with this, then. What do
>> you think about merging it?
>>
>> On Thu, Aug 17, 2017 at 5:30
Author: krobelus
Date: Fri Aug 18 14:26:13 2017
New Revision: 311199
URL: http://llvm.org/viewvc/llvm-project?rev=311199&view=rev
Log:
[clang-diff] Move the JSON export function to clang-diff
Reviewers: arphaman
Subscribers: klimek
Differential Revision: https://reviews.llvm.org/D36178
Added:
Author: krobelus
Date: Fri Aug 18 14:26:34 2017
New Revision: 311200
URL: http://llvm.org/viewvc/llvm-project?rev=311200&view=rev
Log:
[clang-diff] Move printing of matches and changes to clang-diff
Summary:
This also changes the output order of the changes. Now the matches are
printed in pre-ord
This revision was automatically updated to reflect the committed changes.
Closed by commit rL311199: [clang-diff] Move the JSON export function to
clang-diff (authored by krobelus).
Repository:
rL LLVM
https://reviews.llvm.org/D36178
Files:
cfe/trunk/include/clang/Tooling/ASTDiff/ASTDiff.h
This revision was automatically updated to reflect the committed changes.
Closed by commit rL311200: [clang-diff] Move printing of matches and changes to
clang-diff (authored by krobelus).
Repository:
rL LLVM
https://reviews.llvm.org/D36179
Files:
cfe/trunk/include/clang/Tooling/ASTDiff/AST
Should we mention this in the release notes?
Is the flag documented somewhere?
On Wed, Jan 18, 2017 at 4:44 PM, Dehao Chen via cfe-commits
wrote:
> Author: dehao
> Date: Wed Jan 18 18:44:21 2017
> New Revision: 292458
>
> URL: http://llvm.org/viewvc/llvm-project?rev=292458&view=rev
> Log:
> Add
Should we add a mention of this to the release notes?
On Fri, Mar 3, 2017 at 1:49 AM, Nemanja Ivanovic via cfe-commits
wrote:
> Author: nemanjai
> Date: Fri Mar 3 03:49:17 2017
> New Revision: 296861
>
> URL: http://llvm.org/viewvc/llvm-project?rev=296861&view=rev
> Log:
> [PowerPC] Enable -fomi
Author: vlad.tsyrklevich
Date: Fri Aug 18 16:21:10 2017
New Revision: 311210
URL: http://llvm.org/viewvc/llvm-project?rev=311210&view=rev
Log:
Revert "[clang-diff] Move printing of matches and changes to clang-diff"
This reverts commit r311200, it was causing widespread build failures.
Modified:
Author: vlad.tsyrklevich
Date: Fri Aug 18 16:21:11 2017
New Revision: 311211
URL: http://llvm.org/viewvc/llvm-project?rev=311211&view=rev
Log:
Revert "[clang-diff] Move the JSON export function to clang-diff"
This reverts commit r311199, it was causing widespread build failures.
Removed:
cfe
On Fri, Aug 18, 2017 at 2:46 PM, Hans Wennborg wrote:
> Should we mention this in the release notes?
> Is the flag documented somewhere?
>
It's documented in the Options.td file, do you think there are other
documents that need to be added/updated?
Thanks,
Dehao
>
> On Wed, Jan 18, 2017 at
STL_MSFT updated this revision to Diff 111776.
STL_MSFT edited the summary of this revision.
STL_MSFT added a comment.
This addresses Don Hinton's feedback. It also unifies `IsVS2017OrNewer` and
`IsDevDivInternal` into a 3-state `enum class ToolsetLayout`.
https://reviews.llvm.org/D36860
Files
STL_MSFT marked an inline comment as done.
STL_MSFT added a comment.
In https://reviews.llvm.org/D36860#845469, @thakis wrote:
> This approach looks good to me.
Thanks!
> Is there any chance we could have a test for this?
Unfortunately, testing this involves having a VS toolset with the
DevD
Author: krobelus
Date: Fri Aug 18 17:57:38 2017
New Revision: 311216
URL: http://llvm.org/viewvc/llvm-project?rev=311216&view=rev
Log:
Revert "Revert "[clang-diff] Move the JSON export function to clang-diff""
This reverts commit eac4c13ac9ea8f12bc049e040c7b9c8a517f54e7, the
original commit *shou
Author: krobelus
Date: Fri Aug 18 18:34:24 2017
New Revision: 311220
URL: http://llvm.org/viewvc/llvm-project?rev=311220&view=rev
Log:
[clang-diff] Fix test for python 3
Modified:
cfe/trunk/test/Tooling/clang-diff-json.cpp
Modified: cfe/trunk/test/Tooling/clang-diff-json.cpp
URL:
http://llv
thakis accepted this revision.
thakis added a comment.
This revision is now accepted and ready to land.
Many driver tests check in a basic representative directory structure (e.g.
test/Driver/Inputs/basic_freebsd_tree/ and its many siblings).
But if you're happy with others breaking this code, I
STL_MSFT added a comment.
In https://reviews.llvm.org/D36860#846232, @thakis wrote:
> Many driver tests check in a basic representative directory structure (e.g.
> test/Driver/Inputs/basic_freebsd_tree/ and its many siblings).
>
> But if you're happy with others breaking this code, I suppose hav
thakis created this revision.
This makes -Wunreachable-code work for programs containing SEH (except for
__finally, which is still missing for now).
__try is modeled like try (but simpler since it can only have a single __except
or __finally), __except is fairly similar to catch (but simpler,
ahatanak created this revision.
This patch fixes an assertion failure that occurs when compiling the following
invalid code:
struct S {
template
S(T &&) {}
};
template
void foo1(int a0, S a1 = [](){ (void)&a0; } ) { // a0 cannot be used in the
default argument for a1
}
Author: krobelus
Date: Fri Aug 18 19:56:35 2017
New Revision: 311222
URL: http://llvm.org/viewvc/llvm-project?rev=311222&view=rev
Log:
Revert "Revert "[clang-diff] Move printing of matches and changes to
clang-diff""
Fix build by renaming ChangeKind -> Change
This reverts commit 0c78c5729f29315
Author: faisalv
Date: Fri Aug 18 20:43:07 2017
New Revision: 311224
URL: http://llvm.org/viewvc/llvm-project?rev=311224&view=rev
Log:
[c++2a] Implement P0409R2 - Allow lambda capture [=,this] (by hamzasood)
This patch, by hamzasood, implements P0409R2, and allows [=, this] pre-C++2a as
an extens
faisalv closed this revision.
faisalv added a comment.
Committed as r311224, on behalf of Hamza.
https://reviews.llvm.org/rL311224
https://reviews.llvm.org/D36572
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin
ahatanak created this revision.
Sema::CheckAddressOfMemberAccess was disregarding the context in which the
member pointer was referenced.
This patch fixes PR32898.
rdar://problem/33737747
https://reviews.llvm.org/D36918
Files:
lib/Sema/SemaAccess.cpp
test/SemaCXX/access.cpp
Index: test
Rakete updated this revision to Diff 111800.
Rakete added a comment.
Rebased because another commit already added the definitions of the C++20
groups.
https://reviews.llvm.org/D36855
Files:
include/clang/Basic/DiagnosticSemaKinds.td
lib/Sema/SemaExprCXX.cpp
test/SemaCXX/cxx2a-poi
Rakete updated this revision to Diff 111801.
https://reviews.llvm.org/D36855
Files:
include/clang/Basic/DiagnosticSemaKinds.td
lib/Sema/SemaExprCXX.cpp
test/SemaCXX/cxx2a-pointer-to-const-ref-member.cpp
Index: test/SemaCXX/cxx2a-pointer-to-const-ref-member.cpp
yamaguchi added inline comments.
Comment at: llvm/utils/TableGen/OptParserEmitter.cpp:319-320
OS << ", Values);\n";
OS << "(void)ValuesWereAdded;\nassert(ValuesWereAdded &&";
OS << " \"Couldn't add values to OptTable!\");\n";
}
r
yamaguchi updated this revision to Diff 111804.
yamaguchi added a comment.
Update diff. Thank you for your comments!
https://reviews.llvm.org/D36820
Files:
clang/include/clang/Driver/Options.td
clang/test/Driver/autocomplete.c
llvm/utils/TableGen/OptParserEmitter.cpp
Index: llvm/utils/T
ahatanak updated this revision to Diff 111805.
ahatanak added a comment.
Test access to protected member functions.
https://reviews.llvm.org/D36918
Files:
lib/Sema/SemaAccess.cpp
test/SemaCXX/access.cpp
Index: test/SemaCXX/access.cpp
===
1 - 100 of 103 matches
Mail list logo