ro added a comment.
It's been another two weeks, so: could someone please commit this for me?
Thanks.
https://reviews.llvm.org/D39640
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
Author: kosarev
Date: Mon Nov 27 01:39:29 2017
New Revision: 319012
URL: http://llvm.org/viewvc/llvm-project?rev=319012&view=rev
Log:
[CodeGen] Collect information about sizes of accesses and access types for TBAA
The information about access and type sizes is necessary for
producing TBAA metadat
This revision was automatically updated to reflect the committed changes.
Closed by commit rL319012: [CodeGen] Collect information about sizes of
accesses and access types for TBAA (authored by kosarev).
Changed prior to commit:
https://reviews.llvm.org/D40176?vs=124084&id=124332#toc
Repositor
miyuki added a comment.
In https://reviews.llvm.org/D40415#934939, @EricWF wrote:
> LGTM. Thanks.
I don't have write access. Please commit the patch on my behalf.
https://reviews.llvm.org/D40415
___
cfe-commits mailing list
cfe-commits@lists.llvm
ilya-biryukov added inline comments.
Comment at: clangd/ClangdLSPServer.cpp:254
+ std::move(Primary), llvm::make_unique(
+ ".", ArrayRef{}));
+}
Clangd still changes working directory when running the parsing, so `"."`
might en
klimek accepted this revision.
klimek added a comment.
Self-accepting and closing, as the underlying change has landed, and this diff
is incorrect now.
I also have an idea to solve the problem Typz has brought up.
https://reviews.llvm.org/D40068
__
Author: a.sidorin
Date: Mon Nov 27 02:30:00 2017
New Revision: 319015
URL: http://llvm.org/viewvc/llvm-project?rev=319015&view=rev
Log:
[ASTImporter] Support importing CXXPseudoDestructorExpr
Patch by Peter Szecsi!
Differential Revision: https://reviews.llvm.org/D38843
Modified:
cfe/trunk/
This revision was automatically updated to reflect the committed changes.
Closed by commit rL319015: [ASTImporter] Support importing
CXXPseudoDestructorExpr (authored by a.sidorin).
Changed prior to commit:
https://reviews.llvm.org/D38843?vs=124173&id=124338#toc
Repository:
rL LLVM
https://
nik created this revision.
The DeclRefExpr of CXXOperatorCallExpr refering to the custom operator
is visited before the arguments to the operator call. For the Call and
Subscript operator the range of this DeclRefExpr includes the whole call
expression, so that all tokens in that range were mapped
yvvan added inline comments.
Comment at: tools/libclang/CIndex.cpp:6445
+ struct PostChildrenAction {
+ CXCursor cursor;
+ enum Action { Invalid, Ignore, Postpone } action;
4 spaces instead of 2
Comment at: tools/libclang/CIndex.cpp:
klimek added inline comments.
Comment at: lib/Format/ContinuationIndenter.cpp:1518
+ unsigned RemainingTokenColumns = 0;
+ // The column number we're currently at.
+ unsigned ContentStartColumn = 0;
krasimir wrote:
> Could you please spell out the invariants t
nik updated this revision to Diff 124341.
nik added a comment.
Addressed coding style issues.
https://reviews.llvm.org/D40481
Files:
test/Index/annotate-operator-call-expr.cpp
tools/libclang/CIndex.cpp
Index: tools/libclang/CIndex.cpp
===
nik marked 4 inline comments as done.
nik added inline comments.
Comment at: tools/libclang/CIndex.cpp:6888
+ const SourceLocation fixedEnd =
+ RefNameRange.getEnd().getLocWithOffset(-1);
+ RefNameRange = SourceRange(RefNameRange.getBegin(), fixedEnd);
yvvan added inline comments.
Comment at: tools/libclang/CIndex.cpp:6888
+ const SourceLocation fixedEnd =
+ RefNameRange.getEnd().getLocWithOffset(-1);
+ RefNameRange = SourceRange(RefNameRange.getBegin(), fixedEnd);
nik wrote:
> yvvan wrote:
>
ioeric accepted this revision.
ioeric added a comment.
This revision is now accepted and ready to land.
Lg
(Sorry for losing track of this and the delay!)
Repository:
rL LLVM
https://reviews.llvm.org/D39706
___
cfe-commits mailing list
cfe-commi
bkramer added a comment.
There are a few users of the C++ API out there, do we have migration path for
them?
https://reviews.llvm.org/D40439
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-
Author: alexfh
Date: Mon Nov 27 04:42:04 2017
New Revision: 319021
URL: http://llvm.org/viewvc/llvm-project?rev=319021&view=rev
Log:
[clang-tidy] readability-non-const-parameter fixes should update all
declarations
Fixes http://llvm.org/PR34410.
Modified:
clang-tools-extra/trunk/clang-tidy/
alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.
LG, if this still works under Python 2.
https://reviews.llvm.org/D37482
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm
Author: d0k
Date: Mon Nov 27 04:48:26 2017
New Revision: 319022
URL: http://llvm.org/viewvc/llvm-project?rev=319022&view=rev
Log:
Make helper function static. NFC.
Modified:
clang-tools-extra/trunk/clangd/Protocol.cpp
Modified: clang-tools-extra/trunk/clangd/Protocol.cpp
URL:
http://llvm.or
ilya-biryukov created this revision.
Herald added a subscriber: mgorny.
It will be used to pass around things like Logger and Tracer throughout
clangd classes.
https://reviews.llvm.org/D40485
Files:
clangd/CMakeLists.txt
clangd/Context.cpp
clangd/Context.h
clangd/TypedValueMap.h
unitt
ilya-biryukov created this revision.
https://reviews.llvm.org/D40486
Files:
clangd/ClangdLSPServer.cpp
clangd/ClangdServer.cpp
clangd/ClangdServer.h
clangd/ClangdUnit.cpp
clangd/ClangdUnit.h
clangd/ClangdUnitStore.cpp
clangd/ClangdUnitStore.h
clangd/GlobalCompilationDatabase.cpp
ilya-biryukov created this revision.
https://reviews.llvm.org/D40488
Files:
clangd/ClangdUnit.cpp
clangd/JSONRPCDispatcher.cpp
clangd/JSONRPCDispatcher.h
clangd/ProtocolHandlers.cpp
clangd/Trace.cpp
clangd/Trace.h
clangd/tool/ClangdMain.cpp
unittests/clangd/TraceTests.cpp
Index:
alexfh created this revision.
Herald added subscribers: xazax.hun, mgorny.
rename_check.py misc-move-constructor-init performance-move-constructor-init
rename_check.py misc-inefficient-algorithm performance-inefficient-algorithm
https://reviews.llvm.org/D40487
Files:
clang-tidy/cert/CERTTidyM
ilya-biryukov created this revision.
Introduced begin_event, end_event and instant_event instead of a
string phase name.
https://reviews.llvm.org/D40489
Files:
clangd/Trace.cpp
clangd/Trace.h
Index: clangd/Trace.h
===
--- clan
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM!
https://reviews.llvm.org/D40487
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman
hokein accepted this revision.
hokein added a comment.
LGTM.
https://reviews.llvm.org/D40487
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ilya-biryukov planned changes to this revision.
ilya-biryukov added inline comments.
Comment at: clangd/JSONRPCDispatcher.h:80
+ // Ctx must be before Tracer!
Context Ctx;
JSONOutput &Out;
This is still wrong, `Context` is used by `Tracer` internally and s
bkramer requested changes to this revision.
bkramer added inline comments.
This revision now requires changes to proceed.
Comment at: clangd/Context.h:79
+/// Otherwise returns an empty Context.
+Context &globalCtx();
+
This is a giant code smell. If we want the
Author: vedranm
Date: Mon Nov 27 03:14:06 2017
New Revision: 319017
URL: http://llvm.org/viewvc/llvm-project?rev=319017&view=rev
Log:
configure.py: Add gfx900 (Vega, Raven)
Sort amdgcn-- and amdgcn--amdhsa in a consistent way.
Modified:
libclc/trunk/configure.py
Modified: libclc/trunk/confi
Author: AlexeySotkin
Date: Mon Nov 27 01:14:17 2017
New Revision: 319011
URL: http://llvm.org/viewvc/llvm-project?rev=319011&view=rev
Log:
[OpenCL] Add extensions cl_intel_subgroups and cl_intel_subgroups_short
Reviewers: yaxunl, Anastasia, bader
Reviewed By: Anastasia, bader
Subscribers: cfe-c
Author: alexfh
Date: Mon Nov 27 05:06:28 2017
New Revision: 319023
URL: http://llvm.org/viewvc/llvm-project?rev=319023&view=rev
Log:
[clang-tidy] Move checks from misc- to performance-
Summary:
rename_check.py misc-move-constructor-init performance-move-constructor-init
rename_check.py misc-ineff
This revision was automatically updated to reflect the committed changes.
Closed by commit rCTE319023: [clang-tidy] Move checks from misc- to
performance- (authored by alexfh).
Changed prior to commit:
https://reviews.llvm.org/D40487?vs=124356&id=124359#toc
Repository:
rCTE Clang Tools Extra
ilya-biryukov added inline comments.
Comment at: clangd/Context.h:79
+/// Otherwise returns an empty Context.
+Context &globalCtx();
+
bkramer wrote:
> This is a giant code smell. If we want the context route, please pass
> contexts everywhere. I really don't wa
Author: krasimir
Date: Mon Nov 27 05:23:45 2017
New Revision: 319024
URL: http://llvm.org/viewvc/llvm-project?rev=319024&view=rev
Log:
[clang-format] Add option to group multiple #include blocks when sorting
includes
Summary:
This patch allows grouping multiple #include blocks together and sort
This revision was automatically updated to reflect the committed changes.
Closed by commit rL319024: [clang-format] Add option to group multiple #include
blocks when sorting… (authored by krasimir).
Changed prior to commit:
https://reviews.llvm.org/D40288?vs=123924&id=124361#toc
Repository:
luckygeck added inline comments.
Comment at: clangd/Trace.h:41
+ /// Context.
static PtrKey CtxKey;
This is a (1)static non-pod member in an (2) interface. Is it really a good
idea? If we plan to have only one ctxkey, then maybe let's make it not bound to
This revision was automatically updated to reflect the committed changes.
Closed by commit rL319026: [lit] Set shlibpath_var on Solaris (authored by
fedor.sergeev).
Changed prior to commit:
https://reviews.llvm.org/D39640?vs=121626&id=124363#toc
Repository:
rL LLVM
https://reviews.llvm.org/
Author: fedor.sergeev
Date: Mon Nov 27 05:33:19 2017
New Revision: 319026
URL: http://llvm.org/viewvc/llvm-project?rev=319026&view=rev
Log:
[lit] Set shlibpath_var on Solaris
Summary:
During make check-all on Solaris, lit complains
llvm-lit: /vol/gcc/src/llvm/llvm/dist/tools/clang/test/Unit/lit.
aaron.ballman requested changes to this revision.
aaron.ballman added a comment.
This revision now requires changes to proceed.
Please add test cases for this new functionality.
Comment at: lib/CodeGen/TargetInfo.cpp:7665
+static llvm::APSInt getConstexprInt(const Expr *E, cons
malaperle requested changes to this revision.
malaperle added a comment.
This revision now requires changes to proceed.
There were some things I missed, sorry about that!
Comment at: clangd/main.cpp:1
+#define MACRO 1
+namespace ns1 {
This files needs to be rem
spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.
LGTM.
https://reviews.llvm.org/D40230
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-
malaperle added a comment.
@Nebiroth , Will this be compatible with your patch to change
CompilationDatabase at runtime? https://reviews.llvm.org/D39571
https://reviews.llvm.org/D40450
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://
Hahnfeld added reviewers: tra, jlebar.
Hahnfeld added a comment.
The change looks good in general and is small enough and last in the candidates
list to not break working setups.
What concerns me is that we have to do the same for every distribution (I know
that Arch Linux for example has CUDA
ilya-biryukov updated this revision to Diff 124371.
ilya-biryukov added a comment.
- Fixed a typo.
https://reviews.llvm.org/D40489
Files:
clangd/Trace.cpp
clangd/Trace.h
Index: clangd/Trace.h
===
--- clangd/Trace.h
+++ clangd/
ilya-biryukov marked an inline comment as done.
ilya-biryukov added inline comments.
Comment at: clangd/Trace.h:41
+ /// Context.
static PtrKey CtxKey;
luckygeck wrote:
> This is a (1)static non-pod member in an (2) interface. Is it really a good
> idea? I
Author: xazax
Date: Mon Nov 27 07:05:24 2017
New Revision: 319033
URL: http://llvm.org/viewvc/llvm-project?rev=319033&view=rev
Log:
[clang-tidy] Misc redundant expressions check updated for overloaded operators
Patch by: Lilla Barancsuk
Differential Revision: https://reviews.llvm.org/D39243
Mod
Author: alexfh
Date: Mon Nov 27 07:17:13 2017
New Revision: 319034
URL: http://llvm.org/viewvc/llvm-project?rev=319034&view=rev
Log:
[clang-tidy] Fix link error in clang-tidy after the recent check renames.
Modified:
clang-tools-extra/trunk/clang-tidy/cert/CMakeLists.txt
Modified: clang-tool
alexfh added a comment.
Is anything holding this patch?
https://reviews.llvm.org/D39800
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
alexfh added a comment.
Is anything holding this patch?
https://reviews.llvm.org/D39803
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: vvassilev
Date: Mon Nov 27 07:32:00 2017
New Revision: 319037
URL: http://llvm.org/viewvc/llvm-project?rev=319037&view=rev
Log:
Constify. NFC.
Modified:
cfe/trunk/include/clang/Sema/Sema.h
Modified: cfe/trunk/include/clang/Sema/Sema.h
URL:
http://llvm.org/viewvc/llvm-project/cfe/tru
luckygeck added inline comments.
Comment at: clangd/Trace.h:41
+ /// Context.
static PtrKey CtxKey;
ilya-biryukov wrote:
> luckygeck wrote:
> > This is a (1)static non-pod member in an (2) interface. Is it really a good
> > idea? If we plan to have only on
Author: marshall
Date: Mon Nov 27 07:51:36 2017
New Revision: 319038
URL: http://llvm.org/viewvc/llvm-project?rev=319038&view=rev
Log:
Implement LWG#2948: unique_ptr does not define operator<< for stream output
Added:
libcxx/trunk/test/std/utilities/memory/unique.ptr/unique.ptr.special/io.fa
klimek updated this revision to Diff 124381.
klimek added a comment.
Restructure based on code review feedback.
https://reviews.llvm.org/D40310
Files:
lib/Format/BreakableToken.cpp
lib/Format/BreakableToken.h
lib/Format/ContinuationIndenter.cpp
unittests/Format/FormatTest.cpp
unittest
klimek added a comment.
Restructured to make the invariants clearer based on a chat with Krasimir.
https://reviews.llvm.org/D40310
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: benhamilton
Date: Mon Nov 27 07:58:26 2017
New Revision: 319040
URL: http://llvm.org/viewvc/llvm-project?rev=319040&view=rev
Log:
[clang-tools-extra] Fix small typo in docs/ReleaseNotes.rst
Summary:
This is mainly a test diff to check the new Herald rule I
added in LLVM Phabricator to aut
Author: benhamilton
Date: Mon Nov 27 07:58:25 2017
New Revision: 319039
URL: http://llvm.org/viewvc/llvm-project?rev=319039&view=rev
Log:
[clang-tools-extra] Set up .arcconfig to point to new Diffusion CTE repository
Summary:
I'm testing out a new Diffusion repository `CTE`:
https://reviews.llvm
benhamilton created this revision.
We want to automatically copy cfe-commits@ on review requests
to the clang repository.
Similar to https://reviews.llvm.org/D40179, I set up a new Diffusion repository
with callsign
"C" for clang:
https://reviews.llvm.org/source/clang/
This explicitly updates
Author: marshall
Date: Mon Nov 27 08:17:19 2017
New Revision: 319042
URL: http://llvm.org/viewvc/llvm-project?rev=319042&view=rev
Log:
Fix failure on C++03 bots
Modified:
libcxx/trunk/include/ostream
libcxx/trunk/test/std/utilities/memory/unique.ptr/unique.ptr.special/io.fail.cpp
li
dcoughlin accepted this revision.
dcoughlin added a comment.
LGTM, but as noted inline you should update the new llvm_unreachable() to have
a more descriptive error message.
Do you have commit access or do you need someone to commit this for you?
Comment at: lib/StaticAnalyze
Nebiroth updated this revision to Diff 124391.
Nebiroth marked an inline comment as done.
Nebiroth added a comment.
Herald added a subscriber: klimek.
Removed temporary test file
Updated test to account for read-access symbol verification
Repository:
rCTE Clang Tools Extra
https://reviews.llv
Author: abataev
Date: Mon Nov 27 08:54:08 2017
New Revision: 319046
URL: http://llvm.org/viewvc/llvm-project?rev=319046&view=rev
Log:
[OPENMP] Improve handling of cancel directives in target-based
constructs, NFC.
Improved handling of cancel|cancellation point directives inside
target-based for d
benhamilton created this revision.
Herald added a subscriber: cfe-commits.
We want to automatically copy the appropriate mailing list
for review requests to the libc++ repository.
For context, see the proposal and discussion here:
http://lists.llvm.org/pipermail/cfe-dev/2017-November/056032.html
juliehockett updated this revision to Diff 124397.
juliehockett marked 7 inline comments as done.
juliehockett added a comment.
Added new tests and updated wording in warning.
https://reviews.llvm.org/D40108
Files:
clang-tidy/CMakeLists.txt
clang-tidy/fuchsia/CMakeLists.txt
clang-tidy/fuc
benhamilton created this revision.
Herald added a subscriber: cfe-commits.
We want to automatically copy the appropriate mailing list
for review requests to the libc++abi repository.
For context, see the proposal and discussion here:
http://lists.llvm.org/pipermail/cfe-dev/2017-November/056032.h
inspirer added inline comments.
Comment at: clangd/FuzzyMatch.cpp:254
+ // Penalty: matching inside a segment (and previous char wasn't matched).
+ if (WordRole[W] == Tail && P && !Matched[P - 1][W - 1])
+S -= 3;
You need a third boolean dimension in your D
Author: benhamilton
Date: Mon Nov 27 09:21:24 2017
New Revision: 319052
URL: http://llvm.org/viewvc/llvm-project?rev=319052&view=rev
Log:
[clang] Set up .arcconfig to point to new Diffusion C repository
Summary:
We want to automatically copy cfe-commits@ on review requests
to the clang repository
srhines added inline comments.
Comment at: lib/Driver/ToolChains/Gnu.cpp:660
+// of a cpu flag.
+Arg *A = Args.getLastArg(options::OPT_mcpu_EQ);
+if (A) {
Is it better to sink A into the if condition again?
Comment at: test/Driver/a
Author: dergachev
Date: Mon Nov 27 09:31:16 2017
New Revision: 319055
URL: http://llvm.org/viewvc/llvm-project?rev=319055&view=rev
Log:
[analyzer] pr34404: Fix a crash on modeling pointers to indirect members.
We were crashing whenever a C++ pointer-to-member was taken, that was pointing
to a mem
This revision was automatically updated to reflect the committed changes.
Closed by commit rL319055: [analyzer] pr34404: Fix a crash on modeling pointers
to indirect members. (authored by dergachev).
Changed prior to commit:
https://reviews.llvm.org/D39800?vs=122087&id=124404#toc
Repository:
NoQ added a comment.
Sorry, i wanted to quickly look at why this thing is lvalue, but this didn't
seem to be happening, so i guess i'd just commit for now.
Repository:
rL LLVM
https://reviews.llvm.org/D39800
___
cfe-commits mailing list
cfe-comm
Author: dergachev
Date: Mon Nov 27 09:37:09 2017
New Revision: 319058
URL: http://llvm.org/viewvc/llvm-project?rev=319058&view=rev
Log:
[analyzer] pr34766: Fix a crash on explicit std::initializer_list constructor.
We didn't support the following syntax:
(std::initializer_list){12}
which sudd
This revision was automatically updated to reflect the committed changes.
Closed by commit rL319058: [analyzer] pr34766: Fix a crash on explicit
std::initializer_list constructor. (authored by dergachev).
Changed prior to commit:
https://reviews.llvm.org/D39803?vs=122094&id=124406#toc
Reposito
malcolm.parsons created this revision.
Herald added subscribers: cfe-commits, xazax.hun, klimek.
The readability-else-after-return check was not warning about
an else after a throw of an exception that had arguments that needed
to be cleaned up.
Repository:
rCTE Clang Tools Extra
https://revi
sylvestre.ledru added a comment.
Debian packages don't update the PATH and we are aiming at providing packages
working out of the box.
In general, yeah, we have to do that for every distros... :/
https://reviews.llvm.org/D40453
___
cfe-commits mai
beanz added a comment.
@phosek, I would love to see compiler-rt fully refactored so that we target one
triple at a time always. I'd also love to see the clang driver refactored so
that the code to resolve paths to runtime libraries was shared so we follow the
same conventions across all driver
beanz accepted this revision.
beanz added a comment.
This revision is now accepted and ready to land.
LGTM.
Repository:
rL LLVM
https://reviews.llvm.org/D40258
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/
beanz accepted this revision.
beanz added a comment.
This revision is now accepted and ready to land.
LGTM.
Repository:
rL LLVM
https://reviews.llvm.org/D40280
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/
benhamilton added inline comments.
Comment at: clang-tidy/objc/AvoidSpinlockCheck.cpp:22-24
+ if (!getLangOpts().ObjC1 && !getLangOpts().ObjC2) {
+return;
+ }
Why? `OSSpinLock()` calls should also be avoided in C++.
I think you should remove this.
=
Hahnfeld added a comment.
In that case I don't see a way to be "clever", so we have to add the static
candidate. I'll let @tra or @jlebar have a final look.
https://reviews.llvm.org/D40453
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
htt
alexfh created this revision.
Herald added subscribers: xazax.hun, mgorny.
rename_check.py misc-move-const-arg performance-move-const-arg
rename_check.py misc-noexcept-move-constructor
performance-noexcept-move-constructor
https://reviews.llvm.org/D40507
Files:
clang-tidy/hicpp/HICPPTidyModu
pirama updated this revision to Diff 124410.
pirama added a comment.
Address review comments.
https://reviews.llvm.org/D40476
Files:
lib/Driver/ToolChains/Gnu.cpp
test/Driver/as-mcpu.c
Index: test/Driver/as-mcpu.c
===
--- /de
lebedev.ri added a comment.
How about also matching on call to functions with no-return attribute?
i.e.
[[noreturn]] my_die();
void do_stuff();
void fn(int x) {
if(!x)
my_die();
else // <- since `my_die()` will never return, `else` is not really needed.
do_stuff();
pirama updated this revision to Diff 124412.
pirama added a comment.
Add missing space.
https://reviews.llvm.org/D40476
Files:
lib/Driver/ToolChains/Gnu.cpp
test/Driver/as-mcpu.c
Index: test/Driver/as-mcpu.c
===
--- /dev/null
Author: alexfh
Date: Mon Nov 27 09:59:26 2017
New Revision: 319062
URL: http://llvm.org/viewvc/llvm-project?rev=319062&view=rev
Log:
[clang-tidy] Rename qualified references to check class + support inconsistent
names
Modified:
clang-tools-extra/trunk/clang-tidy/rename_check.py
Modified: cl
sepavloff created this revision.
Herald added a subscriber: JDevlieghere.
If a source file extensively uses templates, resulting LLVM IR may have
types with huge names. It may occur if a record type is defined in a class.
In this case its type name contains all declaration contexts and, if a
decla
srhines added inline comments.
Comment at: lib/Driver/ToolChains/Gnu.cpp:661
Arg *A;
-if ((A = Args.getLastArg(options::OPT_mcpu_EQ)) &&
-StringRef(A->getValue()).equals_lower("krait"))
- CmdArgs.push_back("-mcpu=cortex-a15");
-else
- Args.AddLastAr
pirama updated this revision to Diff 124415.
pirama added a comment.
Sink 'Arg *' declaration.
https://reviews.llvm.org/D40476
Files:
lib/Driver/ToolChains/Gnu.cpp
test/Driver/as-mcpu.c
Index: test/Driver/as-mcpu.c
===
--- /d
mcrosier added a comment.
Am I correct in assuming this is going to be a problem for Falkor and Saphira
as well? If so, can you add solutions for those as well? Cortex-a57 should be
good enough for those targets as well.
https://reviews.llvm.org/D40476
Eugene.Zelenko added inline comments.
Comment at: docs/clang-tidy/checks/fuchsia-default-arguments.rst:6
+
+Warns if a function is declared or called with default arguments.
+
Please synchronize with text in Release Notes.
https://reviews.llvm.org/D40108
___
malcolm.parsons added a comment.
In https://reviews.llvm.org/D40505#936170, @lebedev.ri wrote:
> How about also matching on call to functions with no-return attribute?
Great idea!
But I'll keep this patch for just the throw bugfix.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.o
Nebiroth marked 12 inline comments as done.
Nebiroth added inline comments.
Comment at: test/clangd/did-change-configuration.test:33
+
+{"jsonrpc":"2.0","method":"textDocument/didOpen","params":{"textDocument":{"uri":"file:///compile_commands.json","languageId":"json","version":1
pirama updated this revision to Diff 124417.
pirama added a comment.
Normalize falkor and saphira as well.
https://reviews.llvm.org/D40476
Files:
lib/Driver/ToolChains/Gnu.cpp
test/Driver/as-mcpu.c
Index: test/Driver/as-mcpu.c
==
juliehockett updated this revision to Diff 124421.
juliehockett marked an inline comment as done.
juliehockett added a comment.
Updating docs
https://reviews.llvm.org/D40108
Files:
clang-tidy/CMakeLists.txt
clang-tidy/fuchsia/CMakeLists.txt
clang-tidy/fuchsia/DefaultArgumentsCheck.cpp
c
jakehehrlich updated this revision to Diff 124422.
jakehehrlich added a comment.
This test was failing on windows machines. I'm hoping this change (adding
"_cc1" to "%clang") will resolve this because %clang_cc1 shouldn't behave any
differently on windows.
Repository:
rC Clang
https://revie
Author: marshall
Date: Mon Nov 27 11:03:30 2017
New Revision: 319074
URL: http://llvm.org/viewvc/llvm-project?rev=319074&view=rev
Log:
Fix PR#35438 - bitset constructor does not zero unused bits
Modified:
libcxx/trunk/include/bitset
libcxx/trunk/test/std/utilities/template.bitset/bitset.
phosek added a comment.
Can you please take a look at this one as well (this one is Clang, the other
one was LLVM)?
Repository:
rL LLVM
https://reviews.llvm.org/D40257
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org
beanz accepted this revision.
beanz added a comment.
This revision is now accepted and ready to land.
LGTM!
Repository:
rL LLVM
https://reviews.llvm.org/D40257
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/
Author: pirama
Date: Mon Nov 27 11:18:39 2017
New Revision: 319077
URL: http://llvm.org/viewvc/llvm-project?rev=319077&view=rev
Log:
Switch CPU names not recognized by GNU assembler
Summary:
Switch CPU names not recognized by GNU assembler to a close CPU that it
does recognize. In this patch, kr
This revision was automatically updated to reflect the committed changes.
Closed by commit rL319077: Switch CPU names not recognized by GNU assembler
(authored by pirama).
Repository:
rL LLVM
https://reviews.llvm.org/D40476
Files:
cfe/trunk/lib/Driver/ToolChains/Gnu.cpp
cfe/trunk/test/Dri
pirama added a comment.
Thanks for the review. Now let's just hope the windows bots stay happy :)
Repository:
rL LLVM
https://reviews.llvm.org/D40476
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/l
1 - 100 of 156 matches
Mail list logo
