ilya-biryukov added inline comments.
Comment at: clangd/ClangdServer.h:289
+ /// ChangedSettings
+ void changeConfiguration(std::map ChangedSettings);
+
Nebiroth wrote:
> ilya-biryukov wrote:
> > This function is way too general for `ClangdServer`'s interface,
NoQ added a comment.
In https://reviews.llvm.org/D39803#919945, @george.karpenkov wrote:
> I think I lack context to completely get what is going on here: I assume we
> don't model the assignment here?
- We model `IntegralLiteral` as `nonloc::ConcreteInt` `12 S32b`.
- We model `InitListExpr`
xazax.hun added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/DynamicTypePropagation.cpp:587
if (TrackedType &&
+ !isa(CE) &&
!ASTCtxt.canAssignObjCInterfaces(DestObjectPtrType, *TrackedType) &&
george.karpenkov wrote:
> dcoughlin wrote:
Author: ctopper
Date: Wed Nov 8 20:10:46 2017
New Revision: 317766
URL: http://llvm.org/viewvc/llvm-project?rev=317766&view=rev
Log:
[X86] Rename the VEX scalar fma builtins to end with a '3' to match gcc
I think we need to use different builtins for the FMA4 instructions since those
instructio
martell added a comment.
@rnk ping.
Feel free to add anyone you think might be more suitable if you can't review
this.
Repository:
rL LLVM
https://reviews.llvm.org/D39588
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm
martell added a comment.
@rnk ping :)
Repository:
rL LLVM
https://reviews.llvm.org/D39673
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rsmith added a comment.
I'm not entirely sure what's happening with this and
https://reviews.llvm.org/D38818, but the direction looks good to me, and I left
a couple of comments on the other review thread.
https://reviews.llvm.org/D5767
___
cfe-co
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
Thank you! Please move the tests to a new subdirectory under test, say
test/Templight.
Comment at: tools/CMakeLists.txt:37
add_clang_subdirectory(libclang)
+add_subdirector
Author: vedantk
Date: Wed Nov 8 18:33:39 2017
New Revision: 317759
URL: http://llvm.org/viewvc/llvm-project?rev=317759&view=rev
Log:
[Coverage] Complete top-level deferred regions before labels
The area immediately after a terminated region in the function top-level
should have the same count as
Author: vedantk
Date: Wed Nov 8 18:33:40 2017
New Revision: 317760
URL: http://llvm.org/viewvc/llvm-project?rev=317760&view=rev
Log:
[Coverage] Emit deferred regions in headers
There are some limitations with emitting regions in macro expansions
because we don't gather file IDs within the expans
Author: vedantk
Date: Wed Nov 8 18:33:38 2017
New Revision: 317758
URL: http://llvm.org/viewvc/llvm-project?rev=317758&view=rev
Log:
[Coverage] Emit a gap area after if conditions
The area immediately after the closing right-paren of an if condition
should have a count equal to the 'then' block'
alexshap updated this revision to Diff 122194.
alexshap added a comment.
Fix
Repository:
rL LLVM
https://reviews.llvm.org/D39438
Files:
lib/StaticAnalyzer/Checkers/StackAddrEscapeChecker.cpp
test/Analysis/stack-capture-leak-arc.mm
test/Analysis/stack-capture-leak-no-arc.mm
Index: test
Hahnfeld added inline comments.
Comment at: CMakeLists.txt:236
+set(CLANG_DEFAULT_OBJCOPY "objcopy" CACHE STRING
+ "Default objcopy runtime to use.")
+
Nit: `runtime` -> `executable`
https://reviews.llvm.org/D39821
__
george.karpenkov added a comment.
@dcoughlin OK to commit?
https://reviews.llvm.org/D39584
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jdenny updated this revision to Diff 122172.
jdenny retitled this revision from "[VerifyDiagnosticConsumer] support
-verify=PREFIX" to "[VerifyDiagnosticConsumer] support -verify=".
jdenny edited the summary of this revision.
jdenny added a comment.
1. Extended -verify to accept multiple prefixes
george.karpenkov added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/DynamicTypePropagation.cpp:587
if (TrackedType &&
+ !isa(CE) &&
!ASTCtxt.canAssignObjCInterfaces(DestObjectPtrType, *TrackedType) &&
dcoughlin wrote:
> xazax.hun wrote:
george.karpenkov added a comment.
I think I lack context to completely get what is going on here: I assume we
don't model the assignment here?
Comment at: lib/StaticAnalyzer/Core/ExprEngineC.cpp:533
- if (isa(Init)) {
+ if (isa(Init) || isa(Init)) {
// No work needed.
jakehehrlich created this revision.
Herald added subscribers: aprantl, mgorny.
llvm-objcopy is getting to where it can be used in non-trivial ways (such as
for dwarf fission in clang). It now supports dwarf fission but this feature
hasn't been thoroughly tested yet. This change allows people to
george.karpenkov updated this revision to Diff 122170.
george.karpenkov added a comment.
Handle non-zero case, more tests cases, cleaner code.
https://reviews.llvm.org/D39707
Files:
lib/StaticAnalyzer/Core/RangeConstraintManager.cpp
test/Analysis/constant-folding.c
Index: test/Analysis/con
jlebar abandoned this revision.
jlebar added a comment.
Abandoning because it looks like we *can* implement nexttoward in CUDA, see
https://reviews.llvm.org/D39703. Will send a different patch.
https://reviews.llvm.org/D39702
___
cfe-commits maili
jlebar added a comment.
> Libdevice does provide implementation for __nv_nextafterf() and
> __nv_nextafter() and it has corresponding wrappers in math_functions.h[pp].
>
> Perhaps we should keep nextafter around.
Oh, that changes things, since nexttoward is basically the same thing as
nextafte
mikerice added inline comments.
Comment at: lib/Sema/SemaOpenMP.cpp:1197
+ Context, OMPDeclareTargetDeclAttr::MT_To);
+ D->addAttr(A);
+
When D already has an OMPDeclareTargetAttr this seems to add another.
Repository:
rL LLVM
https://r
Author: arphaman
Date: Wed Nov 8 14:47:15 2017
New Revision: 317737
URL: http://llvm.org/viewvc/llvm-project?rev=317737&view=rev
Log:
Remove redundant copy-pasted comment in test file from r317736
Modified:
cfe/trunk/test/SemaObjC/block-literal-with-attribute.m
Modified: cfe/trunk/test/Sema
Author: arphaman
Date: Wed Nov 8 14:44:34 2017
New Revision: 317736
URL: http://llvm.org/viewvc/llvm-project?rev=317736&view=rev
Log:
[ObjC] Fix function signature handling for blocks literals with attributes
Block literals can have a type with attributes in its signature, e.g.
ns_returns_retain
spatel updated this revision to Diff 122158.
spatel added a comment.
Updated again - the last upload was missing the driver's test file.
https://reviews.llvm.org/D39812
Files:
include/clang/Driver/CC1Options.td
include/clang/Frontend/CodeGenOptions.def
lib/CodeGen/CodeGenFunction.cpp
li
This revision was automatically updated to reflect the committed changes.
Closed by commit rL317734: [libcxx] Mark test cxa_deleted_virtual.pass.cpp as
failing for previous libcxx… (authored by vsapsai).
Changed prior to commit:
https://reviews.llvm.org/D39776?vs=122024&id=122157#toc
Repositor
Author: vsapsai
Date: Wed Nov 8 14:30:29 2017
New Revision: 317734
URL: http://llvm.org/viewvc/llvm-project?rev=317734&view=rev
Log:
[libcxx] Mark test cxa_deleted_virtual.pass.cpp as failing for previous libcxx
versions.
r313500 added a fix for undefined "___cxa_deleted_virtual" symbol.
Previo
spatel updated this revision to Diff 122155.
spatel added a comment.
Patch updated:
Match gcc's handling of -fassociative-math with -fno-signed-zeros and
-fno-trapping-math. I've created a new codegen option (-mreassociate) that maps
directly to LLVM's 'reassoc' flag, so we limit the mix-and-mat
Nebiroth added inline comments.
Comment at: clangd/ClangdLSPServer.cpp:51
+ "definitionProvider": true,
+ "configurationChangeProvider": true
}})");
malaperle wrote:
> Are you sure the existing tests don't fail? usually when we change t
tra accepted this revision.
tra added a comment.
This revision is now accepted and ready to land.
LGTM.
https://reviews.llvm.org/D39818
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
tra accepted this revision.
tra added a comment.
This revision is now accepted and ready to land.
Ouch. LGTM.
https://reviews.llvm.org/D39817
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-
jlebar created this revision.
Herald added a subscriber: sanjoy.
Previously we only tested on C++14 and newer, which let slip a bug where
std::min returned the max. :(
https://reviews.llvm.org/D39818
Files:
External/CUDA/algorithm.cu
Index: External/CUDA/algorithm.cu
==
jlebar created this revision.
Herald added a subscriber: sanjoy.
How embarrassing.
This is tested in the test-suite -- fix to come there in a separate
patch.
https://reviews.llvm.org/D39817
Files:
clang/lib/Headers/cuda_wrappers/algorithm
Index: clang/lib/Headers/cuda_wrappers/algorithm
==
malaperle added inline comments.
Comment at: include/indexstore/IndexStoreCXX.h:75
+ bool foreachRelation(llvm::function_ref receiver)
{
+#if INDEXSTORE_HAS_BLOCKS
+return indexstore_occurrence_relations_apply(obj,
^bool(indexstore_symbol_relation_t sym_rel) {
This revision was automatically updated to reflect the committed changes.
Closed by commit rL317727: [ObjC] Boxed strings should use the nullability from
stringWithUTF8String's… (authored by arphaman).
Changed prior to commit:
https://reviews.llvm.org/D39762?vs=121982&id=122148#toc
Repository:
Author: arphaman
Date: Wed Nov 8 13:33:15 2017
New Revision: 317727
URL: http://llvm.org/viewvc/llvm-project?rev=317727&view=rev
Log:
[ObjC] Boxed strings should use the nullability from stringWithUTF8String's
return type
Objective-C NSString has a class method stringWithUTF8String that creates
spatel added a comment.
In https://reviews.llvm.org/D39812#919699, @wristow wrote:
> Yes, I think we want to match that behavior. But by "light up" 'nsz' and the
> no-trapping-math attribute, do you mean automatically turn them on when
> '-fassociative-math' is specified? I'd think it should
Thanks for the patch, Reimar. Can you please add tests to ensure this
functionality doesn’t regress? As null character is required by the standard
(27.7.2.3p21), a good starting point seems to be
test/std/input.output/iostream.format/input.streams/istream.unformatted/getline_pointer_size.pass.cpp
Author: marco
Date: Wed Nov 8 11:21:54 2017
New Revision: 317709
URL: http://llvm.org/viewvc/llvm-project?rev=317709&view=rev
Log:
Add CoreOption flag to "-coverage" option to make it available for clang-cl
Summary:
The -coverage option is not a CoreOption, so it is not available to clang-cl.
Th
Author: marshall
Date: Wed Nov 8 12:25:47 2017
New Revision: 317722
URL: http://llvm.org/viewvc/llvm-project?rev=317722&view=rev
Log:
Added include for
Modified:
libcxx/trunk/fuzzing/fuzzing.cpp
Modified: libcxx/trunk/fuzzing/fuzzing.cpp
URL:
http://llvm.org/viewvc/llvm-project/libcxx/tru
Author: abataev
Date: Wed Nov 8 12:16:14 2017
New Revision: 317719
URL: http://llvm.org/viewvc/llvm-project?rev=317719&view=rev
Log:
[OPENMP] Codegen for `#pragma omp target parallel for`.
Added:
cfe/trunk/test/OpenMP/target_parallel_for_codegen_registration.cpp
cfe/trunk/test/OpenMP/tar
cameron314 added inline comments.
Comment at: cfe/trunk/lib/Sema/SemaOverload.cpp:6396
} else {
AddTemplateOverloadCandidate(FunTmpl, F.getPair(),
ExplicitTemplateArgs, Args,
The same slice that was added abov
frutiger added a comment.
@jklaehn do you know why the referenced cursor would point to line 2?
https://reviews.llvm.org/D39217
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: dlj
Date: Wed Nov 8 12:03:11 2017
New Revision: 317716
URL: http://llvm.org/viewvc/llvm-project?rev=317716&view=rev
Log:
Add a missing "REQUIRES: system-windows" to a Windows-only test.
This un-breaks builds on other platforms. Otherwise, they fail due to warnings
like:
warning: unable
frutiger added a comment.
Friendly poke @rsmith @compnerd
https://reviews.llvm.org/D39161
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
wristow added a comment.
In https://reviews.llvm.org/D39812#919687, @spatel wrote:
> I just reviewed the gcc docs:
> https://gcc.gnu.org/onlinedocs/gcc/Optimize-Options.html
>
> "[-fassociative-math] requires that both -fno-signed-zeros and
> -fno-trapping-math be in effect."
>
> If we want to
spatel added a comment.
I just reviewed the gcc docs:
https://gcc.gnu.org/onlinedocs/gcc/Optimize-Options.html
"[-fassociative-math] requires that both -fno-signed-zeros and
-fno-trapping-math be in effect."
If we want to match that behavior, I need to change this patch to light up
'nsz' and t
frutiger closed this revision.
frutiger added a comment.
Ah missed your last note. Committed as `r317706`.
Repository:
rL LLVM
https://reviews.llvm.org/D39804
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/
Author: frutiger
Date: Wed Nov 8 11:17:27 2017
New Revision: 317706
URL: http://llvm.org/viewvc/llvm-project?rev=317706&view=rev
Log:
[bindings] fix TLS test failure
Since cfe commit r237337, '__declspec(thread)' and 'thread_local' have
been the same since MSVC 2015. i.e. they are both consider
spatel created this revision.
Herald added a subscriber: mcrosier.
There are 2 parts to getting the -fassociative-math command-line flag
translated to LLVM FMF:
1. In the driver/frontend, we accept the flag and its 'no' inverse and deal
with the interactions with other flags like -ffast-math. T
mgorny added a comment.
I see that you haven't more luck than I did finding a better solution. How
about the code completion failure? I suppose I still have to figure that one
out on my own ;-).
I can commit this one myself, I have it ready for commit already.
Repository:
rL LLVM
https://r
mgorny abandoned this revision.
mgorny added a comment.
Duplicate of https://reviews.llvm.org/D39161.
Repository:
rL LLVM
https://reviews.llvm.org/D39810
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailma
frutiger added a comment.
I will commit this shortly.
@mgorny I have a fix for the `unique_external` issue here:
https://reviews.llvm.org/D39161
Repository:
rL LLVM
https://reviews.llvm.org/D39804
___
cfe-commits mailing list
cfe-commits@lists.
mgorny created this revision.
Starting with https://reviews.llvm.org/rL314037, anonymous namespaces no longer
give
unique-external linkage to variables. As a result, the relevant test no
longer works correctly. Comment it out until a replacement expression is
found.
Repository:
rL LLVM
https
dcoughlin added a comment.
I believe your motivating examples used errno_t, which is a signed type.
I'm fine with assuming a two's complement value representation for signed
integers, which would make Artem's suggestion work. That assumption definitely
deserves a comment, though.
https://revi
aprantl added a comment.
For clarification: what is the "symbols table" you are referring to in the
description?
Repository:
rL LLVM
https://reviews.llvm.org/D39622
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cg
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
https://reviews.llvm.org/D38221
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mgorny added a comment.
@frutiger, do you maybe happen to have fixes for the two other test failures?
Maybe I'm tracking them down unnecessarily ;-).
Repository:
rL LLVM
https://reviews.llvm.org/D39804
___
cfe-commits mailing list
cfe-commits@li
bsdjhb added a comment.
Ping
https://reviews.llvm.org/D38110
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ilya-biryukov added a comment.
In https://reviews.llvm.org/D39799#919494, @sammccall wrote:
> e.g. IIUC, things like `#include "sibling.h"` won't look for the h file next
> to the cc file?
Actually, this should work as quoted includes are always searched relative to
the current file before lo
This revision was automatically updated to reflect the committed changes.
Closed by commit rL317700: [analyzer] Fix a crash on logical operators with
vectors. (authored by dergachev).
Changed prior to commit:
https://reviews.llvm.org/D39682?vs=121856&id=122108#toc
Repository:
rL LLVM
https:
Author: dergachev
Date: Wed Nov 8 09:27:58 2017
New Revision: 317700
URL: http://llvm.org/viewvc/llvm-project?rev=317700&view=rev
Log:
[analyzer] Fix a crash on logical operators with vectors.
Do not crash when trying to compute x && y or x || y where x and y are
of a vector type.
For now we do
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
Repository:
rL LLVM
https://reviews.llvm.org/D39804
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailm
sammccall added a comment.
In https://reviews.llvm.org/D39799#919468, @ilya-biryukov wrote:
> In https://reviews.llvm.org/D39799#919415, @sammccall wrote:
>
> > Currently the working directory is always the parent of `compile_flags.txt`
> > as you suggest.
> > Maybe this isn't great though - so
ilya-biryukov added a comment.
In https://reviews.llvm.org/D39799#919415, @sammccall wrote:
> Currently the working directory is always the parent of `compile_flags.txt`
> as you suggest.
> Maybe this isn't great though - sometimes it might be important that the WD
> is the parent of the sourc
frutiger added a comment.
Thanks for posting this on Phabricator, I had sent it to the mailing list last
month.
Repository:
rL LLVM
https://reviews.llvm.org/D39804
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi
sammccall updated this revision to Diff 122100.
sammccall added a comment.
llvm::make_unique
https://reviews.llvm.org/D39799
Files:
docs/JSONCompilationDatabase.rst
include/clang/Tooling/CompilationDatabase.h
lib/Tooling/CompilationDatabase.cpp
test/Tooling/fixed-database.cpp
Index: te
malaperle added inline comments.
Comment at: lib/Index/IndexRecordWriter.cpp:155
+if (!IsNew) {
+ llvm::errs() << "Index: Duplicate USR! " << SymInfo.USR << "\n";
+ // FIXME: print more information so it's easier to find the declaration.
I'm getting
sammccall marked an inline comment as done.
sammccall added a comment.
In https://reviews.llvm.org/D39799#919388, @ilya-biryukov wrote:
> I can see that `FixedCompilationDatabase` does not set a working directory.
> Is this something we may want to have for `compile_flags.txt` or one would
> ne
mgorny created this revision.
The bump of default _MSC_VER which occurred in r315107 has caused
the '__declspec(thread)' TLSKind to change from STATIC to DYNAMIC.
Update the clang python binding tests accordingly. Test both for
behavior with old and new -fms-compatibility-version.
Repository:
ilya-biryukov added a comment.
I can see that `FixedCompilationDatabase` does not set a working directory. Is
this something we may want to have for `compile_flags.txt` or one would need to
resort to `compile_commands.json` to get this?
E.g., I'd find it useful to add includes with paths relativ
NoQ created this revision.
`std::initializer_list` objects can be constructed sort of explicitly, eg.
`(std::initializer_list){12}`. This produces an AST that looks like
CompoundLiteralExpr 0x11987f1a0 'std::initializer_list':'class
std::initializer_list'
`-CXXStdInitializerListExpr 0x11987
ioeric added a comment.
Implementation looks good. Just some nits.
Comment at: lib/Tooling/Refactoring/Extract/CaptureVariables.cpp:36
+ return true;
+// FIXME: Capture 'self'.
+if (!VD->isLocalVarDeclOrParm())
and `this`?
Commen
cameron314 added inline comments.
Comment at: cfe/trunk/lib/Sema/SemaOverload.cpp:6365
cast(FD)->getParent(), ObjectType,
- ObjectClassification, Args.slice(1), CandidateSet,
+ ObjectClassification, F
ahatanak accepted this revision.
ahatanak added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D39776
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/c
This revision was automatically updated to reflect the committed changes.
Closed by commit rL317696: [clang-refactor] Get rid of OccurrencesFinder in
RenamingAction, NFC (authored by hokein).
Repository:
rL LLVM
https://reviews.llvm.org/D39796
Files:
cfe/trunk/lib/Tooling/Refactoring/Rename
NoQ created this revision.
pr34404 points out that given
class C {
struct {
int x;
};
};
, taking pointer-to-member `&C::x` would crash the analyzer. We were not ready
to stumble upon an `IndirectFieldDecl`, which is used for representing a field
within an anonymous structure
Author: hokein
Date: Wed Nov 8 06:53:08 2017
New Revision: 317696
URL: http://llvm.org/viewvc/llvm-project?rev=317696&view=rev
Log:
[clang-refactor] Get rid of OccurrencesFinder in RenamingAction, NFC
Summary:
The OccurrencesFinder is only used in RenameOccurrences to find symbol
occurrences, th
hokein marked an inline comment as done.
hokein added inline comments.
Comment at: clangd/ClangdServer.cpp:67
+
+ // Using the handle(SymbolOccurrences) from parent class.
+ using tooling::RefactoringResultConsumer::handle;
sammccall wrote:
> why is this needed
sammccall accepted this revision.
sammccall added a comment.
This revision is now accepted and ready to land.
(This seems trivial enough to submit without review FWIW)
https://reviews.llvm.org/D39796
___
cfe-commits mailing list
cfe-commits@lists.ll
sammccall accepted this revision.
sammccall added inline comments.
This revision is now accepted and ready to land.
Comment at: clangd/ClangdServer.cpp:67
+
+ // Using the handle(SymbolOccurrences) from parent class.
+ using tooling::RefactoringResultConsumer::handle;
-
sammccall added inline comments.
Comment at: include/clang/Tooling/CompilationDatabase.h:188
+ static std::unique_ptr
+ loadFromFile(StringRef Path, std::string &ErrorMsg);
+
klimek wrote:
> Perhaps loadFromTextFile?
This mirrors JSONCompilationDatabase for con
uriel.k added inline comments.
Comment at: lib/Headers/avx512vlbwintrin.h:2425
+ return _mm_mask_cmpneq_epi8_mask (__U, _mm_and_si128 (__A, __B),
+ _mm_setzero_hi());
}
uriel.k wrote:
> For strange reason this line moved and is n
uriel.k added inline comments.
Comment at: lib/Headers/avx512vlbwintrin.h:2425
+ return _mm_mask_cmpneq_epi8_mask (__U, _mm_and_si128 (__A, __B),
+ _mm_setzero_hi());
}
For strange reason this line moved and is not aligned with t
uriel.k updated this revision to Diff 122082.
uriel.k marked 2 inline comments as done.
https://reviews.llvm.org/D38737
Files:
include/clang/Basic/BuiltinsX86.def
lib/Headers/avx512bwintrin.h
lib/Headers/avx512fintrin.h
lib/Headers/avx512vlbwintrin.h
lib/Headers/avx512vlintrin.h
test/
klimek added inline comments.
Comment at: include/clang/Tooling/CompilationDatabase.h:188
+ static std::unique_ptr
+ loadFromFile(StringRef Path, std::string &ErrorMsg);
+
Perhaps loadFromTextFile?
Comment at: lib/Tooling/CompilationDatabase.
sammccall created this revision.
Herald added a subscriber: ilya-biryukov.
This is an alternative to JSONCompilationDatabase for simple projects that
don't use a build system such as CMake.
(You can also drop one in ~, to make your tools use e.g. C++11 by default)
There's no facility for varying
sylvestre.ledru added a comment.
please thank Marco :)
https://reviews.llvm.org/D38124
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
belleyb added a comment.
@sylvestre.ledru Thanks for the test!
https://reviews.llvm.org/D38124
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sylvestre.ledru updated this revision to Diff 122077.
sylvestre.ledru edited the summary of this revision.
https://reviews.llvm.org/D38124
Files:
lib/profile/GCDAProfiling.c
test/profile/Inputs/instrprof-dlopen-dlclose-main.c
test/profile/instrprof-dlopen-dlclose-gcov.test
Index: test/prof
This revision was automatically updated to reflect the committed changes.
Closed by commit rL317692: Relax definitions.test to accept windows file paths.
(authored by sammccall).
Repository:
rL LLVM
https://reviews.llvm.org/D39797
Files:
clang-tools-extra/trunk/test/clangd/definitions.test
Author: sammccall
Date: Wed Nov 8 05:52:21 2017
New Revision: 317692
URL: http://llvm.org/viewvc/llvm-project?rev=317692&view=rev
Log:
Relax definitions.test to accept windows file paths.
Reviewers: ilya-biryukov
Subscribers: cfe-commits
Differential Revision: https://reviews.llvm.org/D39797
hokein added inline comments.
Comment at: clangd/ClangdServer.cpp:347
+
+void handle(tooling::SymbolOccurrences) override {}
+
sammccall wrote:
> hokein wrote:
> > sammccall wrote:
> > > I don't think you need to override this, assuming you don't expect any o
ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.
LGTM.
https://reviews.llvm.org/D39797
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman
hokein updated this revision to Diff 122074.
hokein marked 2 inline comments as done.
hokein added a comment.
Simplify the code.
https://reviews.llvm.org/D39676
Files:
clangd/CMakeLists.txt
clangd/ClangdLSPServer.cpp
clangd/ClangdLSPServer.h
clangd/ClangdServer.cpp
clangd/ClangdServer
sammccall created this revision.
https://reviews.llvm.org/D39797
Files:
test/clangd/definitions.test
Index: test/clangd/definitions.test
===
--- test/clangd/definitions.test
+++ test/clangd/definitions.test
@@ -27,7 +27,7 @@
# CH
hokein created this revision.
Herald added a subscriber: klimek.
The OccurrencesFinder is only used in RenameOccurrences to find symbol
occurrences, there is no need to inherit RefactoringRule.
Replace it with a single utility function to avoid code misleading.
https://reviews.llvm.org/D39796
krasimir updated this revision to Diff 122069.
krasimir added a comment.
- Remove dead code
https://reviews.llvm.org/D39786
Files:
docs/ClangFormatStyleOptions.rst
docs/tools/dump_format_style.py
include/clang/Format/Format.h
lib/Format/UsingDeclarationsSorter.cpp
unittests/Format/Usi
Author: krasimir
Date: Wed Nov 8 05:28:53 2017
New Revision: 317689
URL: http://llvm.org/viewvc/llvm-project?rev=317689&view=rev
Log:
[clang-tidy] Add a note about modernize-replace-random-shuffle
Summary:
This adds a note warning the users that the way the suggested fix seeds the
random number
This revision was automatically updated to reflect the committed changes.
Closed by commit rL317689: [clang-tidy] Add a note about
modernize-replace-random-shuffle (authored by krasimir).
Repository:
rL LLVM
https://reviews.llvm.org/D39787
Files:
clang-tools-extra/trunk/docs/clang-tidy/che
1 - 100 of 122 matches
Mail list logo