This revision was automatically updated to reflect the committed changes.
Closed by commit rC327206: [ObjC] Allow declaring __weak pointer fields in C
structs in ARC. (authored by ahatanak, committed by ).
Repository:
rC Clang
https://reviews.llvm.org/D44095
Files:
include/clang/AST/Decl.h
This revision was automatically updated to reflect the committed changes.
Closed by commit rL327206: [ObjC] Allow declaring __weak pointer fields in C
structs in ARC. (authored by ahatanak, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.or
Author: ahatanak
Date: Fri Mar 9 22:36:08 2018
New Revision: 327206
URL: http://llvm.org/viewvc/llvm-project?rev=327206&view=rev
Log:
[ObjC] Allow declaring __weak pointer fields in C structs in ARC.
This patch uses the infrastructure added in r326307 for enabling
non-trivial fields to be declar
Author: ahatanak
Date: Fri Mar 9 21:55:21 2018
New Revision: 327204
URL: http://llvm.org/viewvc/llvm-project?rev=327204&view=rev
Log:
[Driver] Pass Default=false to hasFlag.
I forgot to do this in r326530.
Modified:
cfe/trunk/lib/Driver/ToolChains/Clang.cpp
cfe/trunk/test/Driver/fno-esc
Eugene.Zelenko added inline comments.
Comment at: clang-tidy/fuchsia/ZxTemporaryObjectsCheck.cpp:24
+ std::string QualifiedName = Node.getQualifiedNameAsString();
+ return llvm::any_of(Names,
+ [&](StringRef Name) { return QualifiedName == Name; });
---
NoQ added a comment.
> Because string length is for now only composed of `CStringChecker`-tagged
> metadata symbols and constants...
Even if this was not the case, it is stil certain that string length is a
`NonLoc`. And as such it is either a plain `nonloc::SymbolVal` that is
unaffected or a
rtrieu added inline comments.
Herald added a subscriber: JDevlieghere.
Comment at: include/clang/AST/CHashVisitor.h:72-79
+ template
+ void addData(const llvm::iterator_range &x) {
+addData(std::distance(x.begin(), x.end()));
+ }
+ template
+ void addData(const llvm::Arr
NoQ updated this revision to Diff 137892.
NoQ added a comment.
Slightly simplify one of the tests.
https://reviews.llvm.org/D44347
Files:
include/clang/StaticAnalyzer/Core/PathSensitive/SVals.h
test/Analysis/symbol-reaper.c
Index: test/Analysis/symbol-reaper.c
NoQ added a comment.
Leak false-negatives that result from bugs in
`Environment::removeDeadBindings()` and
`RegionStoreManager::removeDeadBindings()` are also only appearing due to the
overall zombie symbol problem we have (https://reviews.llvm.org/D18860). The
bugs are in the code that popula
NoQ created this revision.
NoQ added reviewers: dcoughlin, xazax.hun, a.sidorin, george.karpenkov, szepet.
Herald added subscribers: cfe-commits, rnkovacs, baloghadamsoftware.
I've been investigating a false positive that had a pointer-type symbol with a
non-zero range which had its range forgott
Hi,
This change increases the size of a CallArg, and thus that of a
CallArgList, dramatically (an LValue is *much* larger than an RValue, and
unlike an RValue, does not appear to be at all optimized for size). This
results in CallArgList (which contains inline storage for 16 CallArgs)
ballooning i
Author: rsmith
Date: Fri Mar 9 17:47:22 2018
New Revision: 327195
URL: http://llvm.org/viewvc/llvm-project?rev=327195&view=rev
Log:
Revert r326946. It caused stack overflows by significantly increasing the size
of a CallArgList.
Removed:
cfe/trunk/test/CodeGenCXX/amdgcn-func-arg.cpp
Modifie
juliehockett created this revision.
juliehockett added reviewers: aaron.ballman, hokein, ilya-biryukov.
juliehockett added a project: clang-tools-extra.
Herald added subscribers: xazax.hun, mgorny.
Adding a checker to fuchsia-zx (for zircon) to flag instances where specific
objects are temporaril
This revision was automatically updated to reflect the committed changes.
Closed by commit rL327194: [CMake] Copy the generated __config header into
build directory (authored by phosek, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D4
Author: phosek
Date: Fri Mar 9 17:20:11 2018
New Revision: 327194
URL: http://llvm.org/viewvc/llvm-project?rev=327194&view=rev
Log:
[CMake] Copy the generated __config header into build directory
When the generated __config file is being used, it is currently only
copied during installation proc
majnemer accepted this revision.
majnemer added a comment.
This revision is now accepted and ready to land.
LGTM
Comment at: lib/Transforms/ObjCARC/ObjCARCOpts.cpp:692
+ DenseMap &BlockColors) {
+ auto *CI = dyn_cast(&I);
+ assert(CI && "CloneCallInst must r
Author: gbiv
Date: Fri Mar 9 17:11:17 2018
New Revision: 327192
URL: http://llvm.org/viewvc/llvm-project?rev=327192&view=rev
Log:
[CodeGen] Try to not call a dtor after lifetime.end
If CodeGenFunction::EmitCall is:
- asked to emit a call with an indirectly returned value,
- given an invalid retu
rtrieu added inline comments.
Comment at: lib/Sema/SemaStmt.cpp:2953
+ ExprResult &Res)
+{
+ ImplicitCastExpr AsRvalue(ImplicitCastExpr::OnStack, Value->getType(),
Opening brace should follow the closing paren on previous line.
Author: sqlbyme
Date: Fri Mar 9 16:53:05 2018
New Revision: 327191
URL: http://llvm.org/viewvc/llvm-project?rev=327191&view=rev
Log:
[libcxx][test] Reverting r327178 and r327190.
Reverting changes made to iter_alloc_deduction.fail.cpp
as my changes seem to be making several Linux bots angry.
M
Author: sqlbyme
Date: Fri Mar 9 16:19:25 2018
New Revision: 327190
URL: http://llvm.org/viewvc/llvm-project?rev=327190&view=rev
Log:
[libcxx][test] Marking libcpp-no-deduction-guides unsupported.
This fixes linux bot failures with r327178.
Modified:
libcxx/trunk/test/std/strings/basic.st
compnerd updated this revision to Diff 137867.
compnerd added a comment.
Use the BB colorizer to detect the token. Fortunately, there is no BB
removal/splitting happening here, so there is no state to maintain.
Repository:
rL LLVM
https://reviews.llvm.org/D44327
Files:
lib/Transforms/Obj
vsapsai added inline comments.
Comment at: clang/lib/Serialization/ASTReaderDecl.cpp:1811
if (Update || Canon != D) {
Canon->DefinitionData = D->DefinitionData;
Reader.PendingDefinitions.insert(D);
This store seems to be dead too. Need to spend more
Author: az
Date: Fri Mar 9 15:39:34 2018
New Revision: 327189
URL: http://llvm.org/viewvc/llvm-project?rev=327189&view=rev
Log:
[ARM] Add ARMv8.2-A FP16 vector intrinsic
Add the fp16 neon vector intrinsic for ARM as described in the ARM ACLE
document.
Reviews in https://reviews.llvm.org/D43650
This revision was automatically updated to reflect the committed changes.
juliehockett marked an inline comment as done.
Closed by commit rCTE327186: [clang-tidy] Update run-clang-tidy.py with config
arg (authored by juliehockett, committed by ).
Changed prior to commit:
https://reviews.llvm.or
Author: juliehockett
Date: Fri Mar 9 15:26:56 2018
New Revision: 327186
URL: http://llvm.org/viewvc/llvm-project?rev=327186&view=rev
Log:
[clang-tidy] Update run-clang-tidy.py with config arg
Updating the run-clang-tidy.py script to allow specification of the
config argument to the clang-tidy in
Author: morehouse
Date: Fri Mar 9 15:02:22 2018
New Revision: 327184
URL: http://llvm.org/viewvc/llvm-project?rev=327184&view=rev
Log:
[clangd-fuzzer] Update ClangdLSPServer constructor call.
Build was broken by r326719.
Modified:
clang-tools-extra/trunk/clangd/fuzzer/ClangdFuzzer.cpp
Modi
Author: compnerd
Date: Fri Mar 9 15:00:29 2018
New Revision: 327183
URL: http://llvm.org/viewvc/llvm-project?rev=327183&view=rev
Log:
test: repair windows build after SVN r327105
Thanks to Nico Weber for pointing out the failure. Add an explicit
target for the test.
Modified:
cfe/trunk/tes
phosek added inline comments.
Comment at: libcxx/include/CMakeLists.txt:19
+DEPENDS ${LIBCXX_BINARY_DIR}/__generated_config)
+ set(generated_config_deps generate_config_header)
+endif()
compnerd wrote:
> Where is `generated_config_deps` used?
Line 69
Repos
phosek created this revision.
phosek added a reviewer: chandlerc.
Herald added subscribers: cfe-commits, mgorny.
This CMake flag allows setting the default value for the
-f[no]-experimental-new-pass-manager flag.
Repository:
rC Clang
https://reviews.llvm.org/D44330
Files:
clang/CMakeLists
compnerd accepted this revision.
compnerd added inline comments.
This revision is now accepted and ready to land.
Comment at: libcxx/include/CMakeLists.txt:19
+DEPENDS ${LIBCXX_BINARY_DIR}/__generated_config)
+ set(generated_config_deps generate_config_header)
+endif()
-
majnemer added inline comments.
Comment at: lib/Transforms/ObjCARC/ObjCARCOpts.cpp:701
+
+ if (auto *CleanupPad = dyn_cast(BB.getFirstNonPHI()))
+OpBundles.emplace_back("funclet", CleanupPad);
What if the cleanuppad was introduced in a block which branched t
This revision was automatically updated to reflect the committed changes.
Closed by commit rL327178: XFAIL: libcpp-no-deduction-guides in
libcxx/test/std/strings/basic. (authored by sqlbyme, committed by ).
Herald added subscribers: llvm-commits, christof.
Changed prior to commit:
https://revie
Author: sqlbyme
Date: Fri Mar 9 14:13:12 2018
New Revision: 327178
URL: http://llvm.org/viewvc/llvm-project?rev=327178&view=rev
Log:
XFAIL: libcpp-no-deduction-guides in
libcxx/test/std/strings/basic.string/string.cons/iter_alloc_deduction.fail.cpp
Summary: Refactor the previous version method
compnerd created this revision.
compnerd added a reviewer: gottesmm.
Herald added a subscriber: cfe-commits.
compnerd added a reviewer: ahatanak.
compnerd added subscribers: rnk, majnemer.
In the case that the CallInst that is being moved has an associated
operand bundle which is a funclet, the mo
arsenm added a comment.
In https://reviews.llvm.org/D43281#1023962, @dfukalov wrote:
> The problem is that if set addrspace "2" in description string,
> CanT.getAddressSpace() returns target addrspace value "11" (shifted in the
> enum) and compares it with input LangAS addrspace ("2", "opencl_l
Eugene.Zelenko added inline comments.
Comment at: clang-tidy/bugprone/ParentVirtualCallCheck.cpp:12
+
+#include
+#include
Please run Clang-format and remove empty line between headers.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D44295
_
dfukalov added a comment.
ping...
Repository:
rC Clang
https://reviews.llvm.org/D43281
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
pfultz2 updated this revision to Diff 137822.
https://reviews.llvm.org/D44231
Files:
clang-tidy/misc/SizeofExpressionCheck.cpp
clang-tidy/misc/SizeofExpressionCheck.h
docs/clang-tidy/checks/misc-sizeof-expression.rst
test/clang-tidy/misc-sizeof-expression.cpp
Index: test/clang-tidy/misc-
thegameg added a comment.
Thanks for working on this! Few remarks in the comments.
Comment at: lib/CodeGen/CGBuiltin.cpp:934
+static Value *dumpRecord(CodeGenFunction &CGF, QualType RType,
+ Value*& RecordPtr, CharUnits Align,
`llvm::V
pfultz2 added a comment.
> I don't have a script for it. I've used "bear" with at least some of those
> projects because they use makefiles rather than cmake
> (https://github.com/rizsotto/Bear). I'm not tied to those projects
> specifically, either, so if you have a different corpus you'd pref
rsmith added inline comments.
Comment at: clang/lib/Serialization/ASTReaderDecl.cpp:1789
+ // set fake one.
+ if (!Canon->DefinitionData) {
+Canon->DefinitionData = DD;
No braces around single-line `if` bodies, please.
Comment at: clang/l
Author: pcc
Date: Fri Mar 9 11:37:28 2018
New Revision: 327166
URL: http://llvm.org/viewvc/llvm-project?rev=327166&view=rev
Log:
Fix Clang test case.
Modified:
cfe/trunk/test/CodeGen/thinlto-distributed-cfi-devirt.ll
Modified: cfe/trunk/test/CodeGen/thinlto-distributed-cfi-devirt.ll
URL:
h
This revision was automatically updated to reflect the committed changes.
Closed by commit rC327165: Don't use -pie in relocatable link. (authored
by eugenis, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D44229?vs=137493&id=137809#toc
Repository:
rC Clang
https://review
Author: eugenis
Date: Fri Mar 9 11:35:16 2018
New Revision: 327165
URL: http://llvm.org/viewvc/llvm-project?rev=327165&view=rev
Log:
Don't use -pie in relocatable link.
Summary:
Android, in particular, got PIE enabled by default in r316606. It resulted in
relocatable links passing both -r and -p
Quuxplusone added a comment.
@rtrieu ping?
Repository:
rC Clang
https://reviews.llvm.org/D43898
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
george.karpenkov added a comment.
> we often run out of inlining stack depth limit
Can we consider increasing that limit? I'd much rather have a limit on maximum
path *length* (which we currently don't have), as longer paths are more likely
to be false positives.
On the other hand, I don't see
lebedev.ri added a comment.
Herald added subscribers: llvm-commits, xazax.hun, mgorny, klimek.
Any further thoughts here?
I was slightly bitten by this recently, and i though that it already existed as
a clang-tidy check (:
Repository:
rL LLVM
https://reviews.llvm.org/D16008
_
vsapsai updated this revision to Diff 137801.
vsapsai added a comment.
- Claim the definition data more eagerly.
Not sure that added "different" in the existing comment is actually useful.
It makes sense to me but don't know about others.
https://reviews.llvm.org/D43494
Files:
clang/lib/Seri
malcolm.parsons added inline comments.
Comment at: clang-tidy/bugprone/ParentVirtualCallCheck.cpp:152
+Member->getQualifierLoc().getSourceRange(),
+GetNameAsString(*(Parents.front())) + "::");
+ }
zinovy.nis wrote:
> malcolm.parsons wrote:
> > Wh
dcoughlin added a comment.
This looks good. Some minor post-commit review inline.
Comment at: include/clang/StaticAnalyzer/Checkers/Checkers.td:615
+
+def GCDAsyncSemaphoreChecker : Checker<"GCDAsyncSemaphore">,
+ HelpText<"Checker for performance anti-pattern when using semap
Eugene.Zelenko added inline comments.
Comment at: clang-tidy/bugprone/ParentVirtualCallCheck.cpp:26
+
+bool IsParentOf(const CXXRecordDecl *Parent, const CXXRecordDecl *ThisClass) {
+ assert(Parent);
Please make this function static and remove anonymous namespac
sammccall created this revision.
sammccall added reviewers: ioeric, hokein.
Herald added subscribers: cfe-commits, jkorous-apple, ilya-biryukov, klimek.
This is an important ranking signal.
It's off for the dynamic index for now. Correspondingly, tell the index
infrastructure only to report declar
sammccall created this revision.
Herald added subscribers: cfe-commits, ioeric, jkorous-apple, ilya-biryukov,
klimek.
This is an important ranking signal.
It's off for the dynamic index for now.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D44314
Files:
clangd/global-symbol
zinovy.nis updated this revision to Diff 137787.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D44295
Files:
clang-tidy/bugprone/BugproneTidyModule.cpp
clang-tidy/bugprone/CMakeLists.txt
clang-tidy/bugprone/ParentVirtualCallCheck.cpp
clang-tidy/bugprone/ParentVirtualCallC
zinovy.nis added inline comments.
Comment at: clang-tidy/bugprone/ParentVirtualCallCheck.cpp:152
+Member->getQualifierLoc().getSourceRange(),
+GetNameAsString(*(Parents.front())) + "::");
+ }
malcolm.parsons wrote:
> What does this do for templat
zinovy.nis updated this revision to Diff 137781.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D44295
Files:
clang-tidy/bugprone/BugproneTidyModule.cpp
clang-tidy/bugprone/CMakeLists.txt
clang-tidy/bugprone/ParentVirtualCallCheck.cpp
clang-tidy/bugprone/ParentVirtualCallC
zinovy.nis updated this revision to Diff 137783.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D44295
Files:
clang-tidy/bugprone/BugproneTidyModule.cpp
clang-tidy/bugprone/CMakeLists.txt
clang-tidy/bugprone/ParentVirtualCallCheck.cpp
clang-tidy/bugprone/ParentVirtualCallC
paulsemel updated this revision to Diff 137775.
paulsemel added a comment.
Added recursive type pretty-printing as suggested by Aaron.
Repository:
rC Clang
https://reviews.llvm.org/D44093
Files:
include/clang/Basic/Builtins.def
lib/CodeGen/CGBuiltin.cpp
lib/Sema/SemaChecking.cpp
Index
paulsemel added inline comments.
Comment at: lib/CodeGen/CGBuiltin.cpp:1252
+ Types[getContext().getPointerType(getContext().CharTy)] = "%s";
+ GENERATE_TYPE_QUALIFIERS_NO_RESTRICT(getContext().CharTy, "%s")
+}
aaron.ballman wrote:
> paulsemel wrote
malcolm.parsons added inline comments.
Comment at: clang-tidy/bugprone/ParentVirtualCallCheck.cpp:152
+Member->getQualifierLoc().getSourceRange(),
+GetNameAsString(*(Parents.front())) + "::");
+ }
What does this do for templated parent classes?
aaron.ballman added a comment.
In https://reviews.llvm.org/D44231#1032782, @pfultz2 wrote:
> > Again, that only works for C++ and not C.
>
> Typedef has always worked in C.
This is true.
I think what I'm struggling to say is: I don't think this is a common pattern
in either C or C++. It's als
Eric,
Any chance you could have one more look at this and let me know if the
patch is acceptable?
Thanks,
Mike
On Mon, Mar 5, 2018 at 4:16 PM, Mike Edwards via Phabricator <
revi...@reviews.llvm.org> wrote:
> sqlbyme updated this revision to Diff 137104.
> sqlbyme added a comment.
>
> I copied w
gtbercea added inline comments.
Comment at: test/Driver/openmp-offload-gpu.c:150
+/// bitcode library and add it to the LIBRARY_PATH.
+// RUN: touch %T/libomptarget-nvptx-sm_60.bc
+// RUN: env LIBRARY_PATH=%T %clang -### -fopenmp=libomp
-fopenmp-targets=nvptx64-nvidia-cuda \
gtbercea updated this revision to Diff 137769.
gtbercea added a comment.
Fix test.
Repository:
rC Clang
https://reviews.llvm.org/D43197
Files:
include/clang/Basic/DiagnosticDriverKinds.td
lib/Driver/ToolChains/Cuda.cpp
test/Driver/Inputs/lib/libomptarget-nvptx-sm_20.bc
test/Driver/op
ioeric added inline comments.
Comment at: clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp:188
-bool mergeAndDeduplicate(const TUReplacements &TUs,
- FileToReplacementsMap &GroupedReplacements,
- clang::SourceManager &SM
simark marked 2 inline comments as done.
simark added inline comments.
Comment at: clangd/ClangdServer.h:159
/// constructor will receive onDiagnosticsReady callback.
void addDocument(PathRef File, StringRef Contents,
+ WantDiagnostics WantDiags = WantDiag
Hahnfeld added inline comments.
Comment at: test/Driver/openmp-offload-gpu.c:150
+/// bitcode library and add it to the LIBRARY_PATH.
+// RUN: touch %T/libomptarget-nvptx-sm_60.bc
+// RUN: env LIBRARY_PATH=%T %clang -### -fopenmp=libomp
-fopenmp-targets=nvptx64-nvidia-cuda \
Apoloiges all, I spoke too soon. Some tests fail due to mir changes, and one
which a slight different version of the memset intrinsic.
Attached is a patch which resolves the failures.
Thanks,
Simon
From: llvm-commits [llvm-commits-boun...@lists.llvm.org]
simark marked 2 inline comments as done.
simark added inline comments.
Comment at: clangd/ClangdLSPServer.cpp:101
json::obj{
-{"textDocumentSync", 1},
+{"textDocumentSync", 2},
{"documentFormattingProvider", true},
il
pfultz2 added a comment.
> Again, that only works for C++ and not C.
Typedef has always worked in C.
> Did it report any true positives that would need correcting?
Not for LLVM, but it has in other projects like I mentioned.
> Can you check some other large repos (both C++ and C), such as: Qt,
gtbercea added inline comments.
Comment at: test/Driver/openmp-offload-gpu.c:150
+/// bitcode library and add it to the LIBRARY_PATH.
+// RUN: touch %T/libomptarget-nvptx-sm_60.bc
+// RUN: env LIBRARY_PATH=%T %clang -### -fopenmp=libomp
-fopenmp-targets=nvptx64-nvidia-cuda \
gtbercea updated this revision to Diff 137755.
gtbercea added a comment.
Revert to c_str().
Repository:
rC Clang
https://reviews.llvm.org/D43197
Files:
include/clang/Basic/DiagnosticDriverKinds.td
lib/Driver/ToolChains/Cuda.cpp
test/Driver/Inputs/lib/libomptarget-nvptx-sm_60.bc
test/
gtbercea added inline comments.
Comment at: lib/Driver/ToolChains/Cuda.cpp:592
+Twine("lib") + CLANG_LIBDIR_SUFFIX);
+LibraryPaths.emplace_back(DefaultLibPath.c_str());
+
ABataev wrote:
> Do you still need `.c_str()` here?
Doesn't compile without it b
Hahnfeld added inline comments.
Comment at: test/Driver/openmp-offload-gpu.c:150
+/// bitcode library and add it to the LIBRARY_PATH.
+// RUN: touch %T/libomptarget-nvptx-sm_60.bc
+// RUN: env LIBRARY_PATH=%T %clang -### -fopenmp=libomp
-fopenmp-targets=nvptx64-nvidia-cuda \
gtbercea updated this revision to Diff 137754.
gtbercea added a comment.
Change test.
Repository:
rC Clang
https://reviews.llvm.org/D43197
Files:
include/clang/Basic/DiagnosticDriverKinds.td
lib/Driver/ToolChains/Cuda.cpp
test/Driver/Inputs/lib/libomptarget-nvptx-sm_60.bc
test/Driver
grokos added inline comments.
Comment at: test/Driver/openmp-offload-gpu.c:150
+/// bitcode library and add it to the LIBRARY_PATH.
+// RUN: touch %T/libomptarget-nvptx-sm_60.bc
+// RUN: env LIBRARY_PATH=%T %clang -### -fopenmp=libomp
-fopenmp-targets=nvptx64-nvidia-cuda \
-
ioeric created this revision.
ioeric added a reviewer: sammccall.
Herald added subscribers: cfe-commits, jkorous-apple, ilya-biryukov, klimek.
Potential use case: argument go-to-definition result with symbol
information (e.g. function definition in cc file) that might not be in the AST.
Reposito
Author: abataev
Date: Fri Mar 9 07:20:30 2018
New Revision: 327136
URL: http://llvm.org/viewvc/llvm-project?rev=327136&view=rev
Log:
[OPENMP] Fix the address of the original variable in task reductions.
If initialization of the task reductions requires pointer to original
variable, which is stor
gtbercea marked an inline comment as done.
gtbercea added inline comments.
Comment at: lib/Driver/ToolChains/Cuda.cpp:536-542
+ StringRef CompilerPath = env;
+ while (!CompilerPath.empty()) {
+std::pair Split =
+CompilerPath.split(llvm::sys::EnvPathS
ABataev added inline comments.
Comment at: lib/Driver/ToolChains/Cuda.cpp:592
+Twine("lib") + CLANG_LIBDIR_SUFFIX);
+LibraryPaths.emplace_back(DefaultLibPath.c_str());
+
Do you still need `.c_str()` here?
Comment at: test/Driver/ope
Hi,
Any downstream users should be able to apply this, the corresponding clang
patch (r325651) and the LLD patch (r325657, and r325713) cleanly to the
release sources of LLVM 6.0, should they require it.
Thanks,
Simon
Author: sdardis
Date: Tue Feb 20 16:06:53 2018
New Revision: 325653
URL: http
Looks like this broke the Windows bot:
http://lab.llvm.org:8011/builders/clang-x64-ninja-win7/builds/9365 Can you
fix or revert, please?
On Fri, Mar 9, 2018 at 2:06 AM, Saleem Abdulrasool via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: compnerd
> Date: Thu Mar 8 23:06:42 2018
> Ne
This revision was automatically updated to reflect the committed changes.
Closed by commit rL327134: [SemaOverload] Fixed crash on code completion
(authored by ibiryukov, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D44300
Files:
cfe/
Author: ibiryukov
Date: Fri Mar 9 06:43:29 2018
New Revision: 327134
URL: http://llvm.org/viewvc/llvm-project?rev=327134&view=rev
Log:
[SemaOverload] Fixed crash on code completion
Summary:
The relevant failing assertion message is:
../tools/clang/lib/Sema/SemaInit.cpp:8411: PerformCopyInitializ
ilya-biryukov updated this revision to Diff 137743.
ilya-biryukov added a comment.
- Added a comment
Repository:
rC Clang
https://reviews.llvm.org/D44300
Files:
lib/Sema/SemaOverload.cpp
test/CodeCompletion/enable-if-attr-crash.cpp
Index: test/CodeCompletion/enable-if-attr-crash.cpp
==
sammccall accepted this revision.
sammccall added inline comments.
This revision is now accepted and ready to land.
Comment at: lib/Sema/SemaOverload.cpp:6251
+ : P->getDefaultArg();
+ if (!DefArg)
+return false;
comment this
ilya-biryukov created this revision.
ilya-biryukov added reviewers: bkramer, sammccall, ioeric, hokein.
The relevant failing assertion message is:
../tools/clang/lib/Sema/SemaInit.cpp:8411: PerformCopyInitialization():
Assertion `InitE && "No initialization expression?"' failed.
See the added te
Author: hokein
Date: Fri Mar 9 06:16:46 2018
New Revision: 327131
URL: http://llvm.org/viewvc/llvm-project?rev=327131&view=rev
Log:
[clangd] Fix failing lit test.
This test is missed in r327129.
Modified:
clang-tools-extra/trunk/test/clangd/xrefs.test
Modified: clang-tools-extra/trunk/test
Author: ioeric
Date: Fri Mar 9 06:06:43 2018
New Revision: 327130
URL: http://llvm.org/viewvc/llvm-project?rev=327130&view=rev
Log:
[clangd-vscode] Add package-lock.json to .gitignore
Modified:
clang-tools-extra/trunk/clangd/clients/clangd-vscode/.gitignore
Modified: clang-tools-extra/trunk
This revision was automatically updated to reflect the committed changes.
Closed by commit rL327129: [clangd] Use identifier range as the definition
range. (authored by hokein, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D44247?vs=1
This revision was automatically updated to reflect the committed changes.
Closed by commit rCTE327129: [clangd] Use identifier range as the definition
range. (authored by hokein, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D44247?vs=137720&id=137736#toc
Repository:
rL L
Author: hokein
Date: Fri Mar 9 06:00:34 2018
New Revision: 327129
URL: http://llvm.org/viewvc/llvm-project?rev=327129&view=rev
Log:
[clangd] Use identifier range as the definition range.
Summary: This also matches the range in symbol index.
Reviewers: sammccall
Subscribers: klimek, ilya-biryuk
aaron.ballman added a comment.
In https://reviews.llvm.org/D44231#1032011, @pfultz2 wrote:
> > If the return type of foo() is changed, then the use for s1 will be
> > automatically updated
>
> But usually you write it as:
>
> using foo_return = uint16_t;
> foo_return foo();
> ...
> size_
This revision was automatically updated to reflect the committed changes.
Closed by commit rL327127: [clangd] Don't index template specializations.
(authored by sammccall, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D44298?vs=137731
Author: sammccall
Date: Fri Mar 9 05:25:29 2018
New Revision: 327127
URL: http://llvm.org/viewvc/llvm-project?rev=327127&view=rev
Log:
[clangd] Don't index template specializations.
Summary:
These have different USRs than the underlying entity, but are not typically
interesting in their own righ
ioeric accepted this revision.
ioeric added a comment.
This revision is now accepted and ready to land.
lgtm
Comment at: unittests/clangd/SymbolCollectorTests.cpp:200
+ UnorderedElementsAreArray(
+ {QName("Foo"), QName("f1"), QName("f2"), QName("KInt"),
QName("Tm
sammccall updated this revision to Diff 137731.
sammccall added a comment.
verify that we don't emit extra symbols in other template-related cases
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D44298
Files:
clangd/index/SymbolCollector.cpp
unittests/clangd/SymbolCollectorTe
sammccall updated this revision to Diff 137729.
sammccall added a comment.
merge matchers
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D44298
Files:
clangd/index/SymbolCollector.cpp
unittests/clangd/SymbolCollectorTests.cpp
Index: unittests/clangd/SymbolCollectorTests.cp
sammccall created this revision.
sammccall added a reviewer: ioeric.
Herald added subscribers: cfe-commits, jkorous-apple, ilya-biryukov, klimek.
These have different USRs than the underlying entity, but are not typically
interesting in their own right and can be numerous (e.g. generated traits).
This patch exposes an interface to libclang's clang_getOverriddenCursors
and clang_disposeOverriddenCursors functions to the python api.
The patch is adapted from this stack overflow response but cleaned up to
fit within the existing cindex.py codebase
https://stackoverflow.com/questions/35962473/
1 - 100 of 120 matches
Mail list logo
