Typz added a comment.
ping?
https://reviews.llvm.org/D33440
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Typz added a comment.
ping?
https://reviews.llvm.org/D37813
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hokein accepted this revision.
hokein added a comment.
This revision is now accepted and ready to land.
LGTM.
Comment at: clangd/clients/clangd-vscode/src/extension.ts:25
-const cppFileExtensions: string[] = ['cpp', 'c', 'cc', 'cxx', 'c++', 'm',
'mm', 'h', 'hh', 'hpp',
klimek added a comment.
I've talked with Daniel and we both believe this patch is not the right way to
go forward for clang-format. It is adding configuration mechanisms that we need
to maintain going forward, without addressing a general problem; specifically
for how to handle macros, I
ilya-biryukov requested changes to this revision.
ilya-biryukov added inline comments.
This revision now requires changes to proceed.
Comment at: clangd/ClangdServer.cpp:446
std::vector Result;
- Resources->getAST().get()->runUnderLock([Pos, , this](ParsedAST *AST)
{
+
sammccall created this revision.
sammccall added a reviewer: hokein.
Herald added subscribers: cfe-commits, ilya-biryukov, klimek.
The filtering wasn't previously working as intended - the string list is
interpreted as a list of editor modes, not file extensions.
(It happens to mostly work as "c"
lichray created this revision.
lichray added reviewers: mclow.lists, EricWF.
This is needed to implement ``.
Repository:
rCXX libc++
https://reviews.llvm.org/D41347
Files:
include/__errc
include/system_error
Index: include/system_error
klimek added a comment.
In https://reviews.llvm.org/D37813#958116, @Typz wrote:
> OK.
>
> So you mean a solution like the one discussed earlier would be the way to go?
>
> > I mean that we can configure macros in the format style, like "define A(X)
> > class X {". I'm not 100% sure whether we
ro added a comment.
In https://reviews.llvm.org/D41240#955391, @rsmith wrote:
> Mechanically, the code change looks fine, but I can't comment on whether this
> is a correct change for Solaris, or whether the type provided by `__float128`
> would use the right floating-point representation. You
ro updated this revision to Diff 127348.
ro added a comment.
Enables test/CodeGenCXX/float128-declarations.cpp on Solaris/x86, too.
Repository:
rC Clang
https://reviews.llvm.org/D41240
Files:
lib/Basic/Targets/OSTargets.h
test/CodeGenCXX/float128-declarations.cpp
Index:
Author: malaperle
Date: Mon Dec 18 06:59:01 2017
New Revision: 320988
URL: http://llvm.org/viewvc/llvm-project?rev=320988=rev
Log:
[clangd] Update documentation page with new features, instructions
Summary:
- Some features were implemented so mark them as such.
- Add installation instructions
This revision was automatically updated to reflect the committed changes.
Closed by commit rL320988: [clangd] Update documentation page with new
features, instructions (authored by malaperle, committed by ).
Repository:
rL LLVM
https://reviews.llvm.org/D41306
Files:
a.sidorin added a comment.
Hi Artem. This patch looks OK, just stylish issues.
Comment at: lib/StaticAnalyzer/Core/ExprEngineCXX.cpp:112
+ // It means that we cannot handle construction into null or garbage pointers.
+ // Such cosntructors need to be handled by checkers to
Author: sberg
Date: Mon Dec 18 05:51:48 2017
New Revision: 320982
URL: http://llvm.org/viewvc/llvm-project?rev=320982=rev
Log:
Revert r320978 "No -fsanitize=function warning when calling noexcept function
through non-noexcept pointer in C++17"
At least
sberg reopened this revision.
sberg added a comment.
Had to revert r320977/r320978 again with r320981/r320982: "At least
http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-android/builds/6013/steps/annotate/logs/stdio
complains about
ubsan::ubsan_handle_function_type_mismatch_abort
Author: kosarev
Date: Mon Dec 18 08:50:11 2017
New Revision: 320994
URL: http://llvm.org/viewvc/llvm-project?rev=320994=rev
Log:
Fix the reference to the now renamed member of TBAAStructField
See https://reviews.llvm.org/D39956 for details.
Modified:
cfe/trunk/lib/CodeGen/CodeGenTBAA.cpp
sammccall created this revision.
sammccall added a reviewer: ioeric.
Herald added subscribers: cfe-commits, ilya-biryukov, mgorny, klimek.
- Moved these functions to SourceCode.h
- added unit tests
- fix off by one in positionToOffset: Offset - 1 in final calculation was wrong
- fixed
a.sidorin added a comment.
This thing is very similar to https://reviews.llvm.org/D19979. Do we really
need to create a separate LoopContext or we can reuse ScopeContext instead?
https://reviews.llvm.org/D41151
___
cfe-commits mailing list
a.sidorin added a comment.
This patch still depends on scope implementation in CFG. There is no final
implementation; after initial implementation is done, I'll update the patch.
https://reviews.llvm.org/D19979
___
cfe-commits mailing list
ro added inline comments.
Comment at: lib/Driver/ToolChains/CommonArgs.cpp:528
StringRef Sanitizer) {
+ // Solaris ld doesn't need this. Inhibit use of non-existant
+ // --export-dynamic.
alekseyshl wrote:
> Can you
On 12/18/2017 03:05 PM, Richard Smith wrote:
Can we just strip the noexcept from the function type before emitting
the fsan type info?
As is apparently already done when emitting the (not noexcept-annotated)
__cxxabiv1::__function_type_info referenced from a noexcept-annotated
alekseyshl accepted this revision.
alekseyshl added inline comments.
This revision is now accepted and ready to land.
Comment at: lib/Driver/ToolChains/CommonArgs.cpp:528
StringRef Sanitizer) {
+ // Solaris ld doesn't need this. Inhibit use
Can we just strip the noexcept from the function type before emitting the
fsan type info?
On 18 Dec 2017 13:52, "Stephan Bergmann via cfe-commits" <
cfe-commits@lists.llvm.org> wrote:
Author: sberg
Date: Mon Dec 18 05:51:48 2017
New Revision: 320982
URL:
jgreenhalgh added a comment.
If this patch unconditionally defines _Float128, then I think it will conflict
with the typedef for _Float128 for IEEE754 128-bit long double systems in glibc:
/* The type _Float128 exists only since GCC 7.0. */
# if !__GNUC_PREREQ (7, 0) || defined
bcain added a comment.
In https://reviews.llvm.org/D39149#912260, @smeenai wrote:
> I confirmed that these warnings go away with https://reviews.llvm.org/D39462
> applied, and they reappear if I manually specify
> `-Wmaybe-tautological-constant-compare`. Thank you!
Let's resurrect these
dblaikie created this revision.
dblaikie added a reviewer: rsmith.
Herald added subscribers: cfe-commits, klimek.
This goes part-way down the path of moving the actual diagnostics out of
Clang's Basic library into the respective libraries that use those diagnostics.
The end goal would be that a
Typz added a comment.
OK.
So you mean a solution like the one discussed earlier would be the way to go?
> I mean that we can configure macros in the format style, like "define A(X)
> class X {". I'm not 100% sure whether we would just try to use the
> Preprocessor for this, or whether we'd
sammccall updated this revision to Diff 127320.
sammccall added a comment.
Fix quotes
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D41343
Files:
clangd/clients/clangd-vscode/src/extension.ts
Index: clangd/clients/clangd-vscode/src/extension.ts
Author: sammccall
Date: Mon Dec 18 03:29:45 2017
New Revision: 320972
URL: http://llvm.org/viewvc/llvm-project?rev=320972=rev
Log:
[clangd] in VSCode client, filter extensions properly and only accept file: URIs
Summary:
The filtering wasn't previously working as intended - the string list is
Author: mboehme
Date: Mon Dec 18 03:29:21 2017
New Revision: 320971
URL: http://llvm.org/viewvc/llvm-project?rev=320971=rev
Log:
[X86] Use {{.*}} instead of hardcoded %1 in knot test.
This makes the test more resilient and consistent with the other tests
introduced in r320919.
Modified:
This revision was automatically updated to reflect the committed changes.
Closed by commit rL320972: [clangd] in VSCode client, filter extensions
properly and only accept file: URIs (authored by sammccall, committed by ).
Repository:
rL LLVM
https://reviews.llvm.org/D41343
Files:
ilya-biryukov added a comment.
Here are a few more comments.
Comment at: clangd/CodeComplete.cpp:847
+ if (Opts.Index && SCInfo.SSInfo) {
+// FIXME: log warning with logger if sema code completion have collected
+// results.
NIT: FIXME(ioeric)?
Also,
ilya-biryukov added inline comments.
Comment at: clangd/ClangdLSPServer.cpp:284
+void ClangdLSPServer::onCodeHover(Ctx C, TextDocumentPositionParams ) {
+
+ auto H = Server.findHover(C,
NIT: remove empty line
Comment at:
sammccall added a comment.
Hope you don't mind me piling on... let me know!
Mostly readability, but i'd also like to reduce the scope w.r.t namespace
completion.
Comment at: clangd/CodeComplete.cpp:90
+CompletionItemKind getKindOfSymbol(index::SymbolKind Kind) {
+ using SK
ioeric created this revision.
ioeric added reviewers: hokein, sammccall.
Herald added subscribers: cfe-commits, ilya-biryukov, klimek.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D41345
Files:
clangd/ClangdUnit.cpp
clangd/ClangdUnit.h
clangd/index/FileIndex.cpp
miyuki updated this revision to Diff 127324.
miyuki added a comment.
Merged two diagnostics into one
https://reviews.llvm.org/D40705
Files:
include/clang/Basic/DiagnosticParseKinds.td
lib/Parse/ParseTemplate.cpp
test/CXX/temp/temp.param/p2-cpp11.cpp
test/CXX/temp/temp.param/p2.cpp
miyuki added a comment.
I don't have commit access. Please commit the patch on my behalf.
https://reviews.llvm.org/D40705
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mclow.lists accepted this revision.
mclow.lists added a comment.
This revision is now accepted and ready to land.
The rest of this LGTM.
Comment at:
test/std/numerics/numeric.ops/transform.reduce/transform_reduce_iter_iter_iter_init.pass.cpp:61
+inline MoveOnly
Author: sberg
Date: Mon Dec 18 05:05:42 2017
New Revision: 320978
URL: http://llvm.org/viewvc/llvm-project?rev=320978=rev
Log:
No -fsanitize=function warning when calling noexcept function through
non-noexcept pointer in C++17
As discussed in the mail thread
This revision was not accepted when it landed; it landed in state "Needs
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rC320978: No -fsanitize=function warning when calling noexcept
function through non… (authored by sberg, committed by ).
rjmccall added inline comments.
Comment at: lib/CodeGen/CGDecl.cpp:1421
+destroyer = CodeGenFunction::destroyNonTrivialCStruct;
+cleanupKind = getARCCleanupKind();
+break;
ahatanak wrote:
> rjmccall wrote:
> > This can only be getARCCleanupKind() if
sammccall added a comment.
Nice, thanks!
Some suggestions for further tightening the semantics, I'm happy to do these as
a followup if you think it's too invasive here.
Comment at: clangd/index/Index.h:127
/// \brief A query string for the fuzzy find. This is matched
sepavloff added inline comments.
Comment at: lib/Sema/SemaDecl.cpp:12184
Decl *Sema::ActOnSkippedFunctionBody(Decl *Decl) {
- if (FunctionDecl *FD = dyn_cast_or_null(Decl))
+ if (FunctionDecl *FD = Decl->getAsFunction())
FD->setHasSkippedBody();
In the
Author: kamil
Date: Mon Dec 18 23:10:33 2017
New Revision: 321060
URL: http://llvm.org/viewvc/llvm-project?rev=321060=rev
Log:
Teach clang/NetBSD about additional dependencies for sanitizers
Summary:
Sanitizers on NetBSD require additional linkage:
- libutil for forkpty(3)
- libexecinfo for
This revision was automatically updated to reflect the committed changes.
Closed by commit rC321060: Teach clang/NetBSD about additional dependencies for
sanitizers (authored by kamil, committed by ).
Repository:
rC Clang
https://reviews.llvm.org/D41054
Files:
a.sidorin updated this revision to Diff 127375.
a.sidorin added reviewers: xazax.hun, szepet.
a.sidorin added a comment.
Herald added a subscriber: rnkovacs.
Removed already fixed stuff, added a test for remaining.
Repository:
rC Clang
https://reviews.llvm.org/D6550
Files:
This operator bool should probably be explicit (as most/all of them should
be - and most of them in LLVM are) - to avoid undue implicit conversion to
other int types.
On Wed, Dec 13, 2017 at 7:43 AM Ilya Biryukov via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: ibiryukov
> Date:
Nebiroth created this revision.
Nebiroth added reviewers: malaperle, ilya-biryukov.
Adds BeforeExecute method to PrecompiledPreamble to be called before Execute().
This method can be overriden.
Repository:
rC Clang
https://reviews.llvm.org/D41365
Files:
JonasToth added a comment.
What happens if the operator is overloaded outside a class? Is that
allowed/disallowed and could you please mention the guideline on that in the
docs + tests.
Comment at: clang-tidy/fuchsia/OverloadedOperatorCheck.cpp:18
+
timshen updated this revision to Diff 127399.
timshen added a comment.
Formatted the files.
https://reviews.llvm.org/D41148
Files:
libcxx/include/experimental/__config
libcxx/include/experimental/simd
libcxx/test/std/experimental/simd/nothing_to_do.pass.cpp
Author: dim
Date: Mon Dec 18 11:46:56 2017
New Revision: 321016
URL: http://llvm.org/viewvc/llvm-project?rev=321016=rev
Log:
Fix more inconsistent line endings. NFC.
Modified:
clang-tools-extra/trunk/docs/modularize.rst
clang-tools-extra/trunk/docs/pp-trace.rst
Modified:
Author: dim
Date: Mon Dec 18 11:46:56 2017
New Revision: 321016
URL: http://llvm.org/viewvc/llvm-project?rev=321016=rev
Log:
Fix more inconsistent line endings. NFC.
Modified:
cfe/trunk/include/clang/AST/OperationKinds.def
cfe/trunk/lib/Parse/ParseTemplate.cpp
Eugene.Zelenko added inline comments.
Comment at: docs/ReleaseNotes.rst:141
+
+ Warns if an operator is overloaded, except for the copy and move operators.
+
assignment operators?
Comment at:
vsk added a comment.
Copying my comment from the diffusion thread to keep things in one place:
> You could make FunctionTypeMismatch an 'AlwaysRecoverable' check, just like
> the Vptr check, and remove the _abort handlers from the ubsan runtimes.
Repository:
rC Clang
vsk added a comment.
Please add a test.
Repository:
rC Clang
https://reviews.llvm.org/D40720
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
craig.topper added inline comments.
Comment at: lib/CodeGen/CGCall.cpp:1737
llvm::toStringRef(CodeGenOpts.NoSignedZeros));
+FuncAttrs.addAttribute("shstk-compatible",
+ llvm::toStringRef(CodeGenOpts.ShstkCompatible));
Nebiroth marked 21 inline comments as done.
Nebiroth added inline comments.
Comment at: clangd/ClangdServer.h:306
+ /// Run formatting for \p Rng inside \p File.
+ std::vector formatRange(PathRef File, Range Rng);
+ /// Run formatting for the whole \p File.
ioeric updated this revision to Diff 127409.
ioeric added a comment.
- minor cleanup.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D41367
Files:
clangd/index/Index.h
clangd/index/MemIndex.cpp
unittests/clangd/IndexTests.cpp
Index: unittests/clangd/IndexTests.cpp
bruno added a comment.
Hi Alex,
Thanks for improving this.
- Instead of adding `-cc1gen-reproducer`, why can't you run that through `-cc1`
and have a flag similar to `-###`, which just prints the reproducer line?
- I didn't understand how you can use the final output information, can you
szepet added subscribers: alexfh, szepet.
szepet requested changes to this revision.
szepet edited reviewers, added: xazax.hun, szepet; removed: dergachev.a.
szepet added a comment.
Herald added a subscriber: rnkovacs.
Hi Alexey!
Thank you for working on this!
Some general comments on the
nathawes updated this revision to Diff 127412.
nathawes marked an inline comment as done.
nathawes added a comment.
I've refactored the indexing/dependency data collection out from the writing
with the new IndexUnitDataConsumer class, and made other smaller changes to
address the feedback from
pcc added a comment.
Would it be possible to fix this by stripping the noexcept specifiers from both
the function type used in the check and the one that is embedded in the prefix
data? The downside is that we won't catch the case where the caller has a
noexcept specifier and the callee
Author: kcc
Date: Mon Dec 18 13:40:07 2017
New Revision: 321027
URL: http://llvm.org/viewvc/llvm-project?rev=321027=rev
Log:
[hwasan] update the design doc
Modified:
cfe/trunk/docs/HardwareAssistedAddressSanitizerDesign.rst
Modified: cfe/trunk/docs/HardwareAssistedAddressSanitizerDesign.rst
ioeric created this revision.
ioeric added a reviewer: sammccall.
Herald added subscribers: cfe-commits, ilya-biryukov, klimek.
When scopes are specified, only match symbols from scopes.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D41367
Files:
clangd/index/Index.h
ioeric added a comment.
Thanks for the review!
Comment at: clangd/CodeComplete.cpp:847
+ if (Opts.Index && SCInfo.SSInfo) {
+// FIXME: log warning with logger if sema code completion have collected
+// results.
ilya-biryukov wrote:
> NIT:
sberg added a comment.
In https://reviews.llvm.org/D40720#958677, @vsk wrote:
> Please add a test.
Note that the bot upon the first closing of this review changed the shown diff
from the combined cfe+compiler-rt diff to just the cfe part. See
https://reviews.llvm.org/rL320977 for the
ioeric updated this revision to Diff 127406.
ioeric marked 21 inline comments as done.
ioeric added a comment.
- Address comments in CodeComplete.cpp
- Split index changes into a separate patch.
- Merged with patch https://reviews.llvm.org/D41351.
- Remove Position.* files.
Repository:
rCTE
Nebiroth updated this revision to Diff 127413.
Nebiroth marked 3 inline comments as done.
Nebiroth added a comment.
findHover properly returns an error
Removed many cases of bad merging
Separated getHover in two functions
Minor indenting and NIT fixes
Repository:
rCTE Clang Tools
mclow.lists added a comment.
In https://reviews.llvm.org/D39149#958378, @bcain wrote:
> Let's resurrect these changes since https://reviews.llvm.org/D39462 was not
> the right short-term approach.
Let's not. IMHO, these changes are:
- a bad idea in principle - clang should not be warning
nathawes marked 45 inline comments as done.
nathawes added inline comments.
Comment at: lib/Index/FileIndexRecord.h:51
+public:
+ FileIndexRecord(FileID FID, bool isSystem) : FID(FID), IsSystem(isSystem) {}
+
ioeric wrote:
> s/isSystem/IsSystem/
>
> Also, I
vsk added a comment.
In https://reviews.llvm.org/D40720#958697, @sberg wrote:
> In https://reviews.llvm.org/D40720#958677, @vsk wrote:
>
> > Please add a test.
>
>
> Note that the bot upon the first closing of this review changed the shown
> diff from the combined cfe+compiler-rt diff to just
rnk added inline comments.
Comment at: lib/CodeGen/CodeGenModule.cpp:690-692
+ // Only handle ELF for now.
+ if (!CGM.getTriple().isOSBinFormatELF())
+return false;
Handling COFF here is probably trivial. Everything is dso_local unless it's
dllimport.
Reid Kleckner via Phabricator writes:
> rnk added inline comments.
>
>
>
> Comment at: lib/CodeGen/CodeGenModule.cpp:690-692
> + // Only handle ELF for now.
> + if (!CGM.getTriple().isOSBinFormatELF())
> +return false;
>
>
dcoughlin accepted this revision.
dcoughlin added a comment.
This revision is now accepted and ready to land.
Looks good to me, although I have a super nit about wording in a comment.
Comment at: lib/StaticAnalyzer/Core/ExprEngine.cpp:2181
+ // One of the forbidden LValue
Nebiroth updated this revision to Diff 127386.
Nebiroth marked 14 inline comments as done.
Nebiroth added a comment.
Herald added a subscriber: mgrang.
CppFilePreambleCallbacks no longer extends PPCallbacks
CppFilePreambleCallbacks no longer needs SourceManager parameter
Removed temporary
Thanks!
~Craig
On Mon, Dec 18, 2017 at 3:29 AM, Martin Bohme via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: mboehme
> Date: Mon Dec 18 03:29:21 2017
> New Revision: 320971
>
> URL: http://llvm.org/viewvc/llvm-project?rev=320971=rev
> Log:
> [X86] Use {{.*}} instead of hardcoded
juliehockett created this revision.
juliehockett added reviewers: aaron.ballman, alexfh, hokein.
juliehockett added a project: clang-tools-extra.
Herald added subscribers: xazax.hun, mgorny.
Adds a check to the Fuchsia module to warn if an operator is overloaded, except
move and copy operators.
NoQ marked 2 inline comments as done.
NoQ added a comment.
Sorry for the spam.
https://reviews.llvm.org/D40560
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
BillyONeal created this revision.
BillyONeal added reviewers: mclow.lists, EricWF.
- Remove _VSTD use and remove C++03-isms in C++17 tests for transform_reduce.
- Add tests for all the transform_reduce overloads that pass in a move-only
type.
- Repair missing move in libc++ product code.
efriedma created this revision.
efriedma added reviewers: vsk, davidxl.
efriedma added a project: clang.
Fixes regression from r320533.
This fixes the undefined behavior, but I'm not sure it's really right... I
think we end up with missing coverage for code in modules.
Repository:
rC Clang
BillyONeal added inline comments.
Comment at:
test/std/numerics/numeric.ops/transform.reduce/transform_reduce_iter_iter_iter_init.pass.cpp:61
+inline MoveOnly operator+(const MoveOnly& lhs, const MoveOnly& rhs)
+{
Should these go in MoveOnly.h ?
Repository:
This revision was automatically updated to reflect the committed changes.
Closed by commit rL321052: [Coverage] Fix use-after free in coverage emission
(authored by efriedma, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D41374?vs=127440=127451#toc
Repository:
rL LLVM
Author: efriedma
Date: Mon Dec 18 17:54:09 2017
New Revision: 321052
URL: http://llvm.org/viewvc/llvm-project?rev=321052=rev
Log:
[Coverage] Fix use-after free in coverage emission
Fixes regression from r320533.
This fixes the undefined behavior, but I'm not sure it's really right...
I think we
timshen updated this revision to Diff 127453.
timshen added a comment.
Update description.
https://reviews.llvm.org/D41376
Files:
libcxx/include/__config
libcxx/include/experimental/__config
libcxx/include/experimental/simd
libcxx/include/utility
ahatanak marked 9 inline comments as done.
ahatanak added a comment.
Address review comments.
Comment at: lib/CodeGen/CGDecl.cpp:1421
+destroyer = CodeGenFunction::destroyNonTrivialCStruct;
+cleanupKind = getARCCleanupKind();
+break;
rjmccall
timshen updated this revision to Diff 127458.
timshen added a comment.
Fix return type of reference::opreator--.
https://reviews.llvm.org/D41376
Files:
libcxx/include/__config
libcxx/include/experimental/__config
libcxx/include/experimental/simd
libcxx/include/utility
george.karpenkov abandoned this revision.
george.karpenkov added a comment.
Abandoned in favor of https://reviews.llvm.org/D41378
https://reviews.llvm.org/D40809
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
smeenai updated this revision to Diff 127416.
smeenai added a comment.
Remove stray comment
Repository:
rCXX libc++
https://reviews.llvm.org/D41368
Files:
include/istream
src/string.cpp
Index: src/string.cpp
===
---
vsk updated this revision to Diff 127424.
vsk added a comment.
- Make the patch cleaner by introducing an overload of EmitCall() which doesn't
require a SourceLocation argument.
https://reviews.llvm.org/D40698
Files:
docs/UndefinedBehaviorSanitizer.rst
lib/CodeGen/CGBuiltin.cpp
NoQ added a comment.
Comment at: lib/StaticAnalyzer/Core/ExprEngineCXX.cpp:522
+symVal = peekCXXNewAllocatorValue(State);
+ State = popCXXNewAllocatorValue(State);
+
a.sidorin wrote:
> Should this be under 'if' as well?
Whops!! Thanks!
vsk updated this revision to Diff 127426.
vsk added a comment.
- Tighten the IR test case.
https://reviews.llvm.org/D40698
Files:
docs/UndefinedBehaviorSanitizer.rst
lib/CodeGen/CGBuiltin.cpp
lib/CodeGen/CGCall.cpp
lib/CodeGen/CGExpr.cpp
lib/CodeGen/CGExprCXX.cpp
NoQ updated this revision to Diff 127437.
NoQ added a comment.
- Fix more `for(:)` whitespace.
https://reviews.llvm.org/D40560
Files:
include/clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h
lib/StaticAnalyzer/Core/ExprEngine.cpp
lib/StaticAnalyzer/Core/ExprEngineCXX.cpp
vsk accepted this revision.
vsk added a comment.
This revision is now accepted and ready to land.
Thanks, this lgtm as a stopgap.
As I mentioned offline, I think the goal should be to have non-modules and
modules-enabled builds of a project produce identical coverage reports. I'll
look into
BillyONeal added inline comments.
Comment at: include/numeric:245
for (; __first1 != __last1; ++__first1, (void) ++__first2)
__init = __b1(__init, __b2(*__first1, *__first2));
return __init;
It's arguable that this should be
timshen created this revision.
timshen added reviewers: mclow.lists, EricWF.
Herald added a subscriber: sanjoy.
This patch added a new macro _LIBCPP_HAS_VECTOR_EXTENSION for detecting
whether a vector extension (__attribute__((vector_size(num_bytes is
available.
This patch also backports
timshen updated this revision to Diff 127455.
timshen added a comment.
Fix reference's access control.
https://reviews.llvm.org/D41376
Files:
libcxx/include/__config
libcxx/include/experimental/__config
libcxx/include/experimental/simd
libcxx/include/utility
ahatanak updated this revision to Diff 127456.
ahatanak marked 5 inline comments as done.
ahatanak added a comment.
Update patch.
https://reviews.llvm.org/D41228
Files:
docs/LanguageExtensions.rst
include/clang/AST/Decl.h
include/clang/AST/Type.h
phosek added a comment.
ping?
Repository:
rL LLVM
https://reviews.llvm.org/D40513
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sammccall added inline comments.
Comment at: clangd/ClangdUnit.cpp:941
createInvocationFromCommandLine(ArgStrs, CommandLineDiagsEngine,
VFS);
+ CI->LangOpts->CommentOpts.ParseAllComments = true;
// createInvocationFromCommandLine sets DisableFree.
NoQ updated this revision to Diff 127420.
NoQ added a comment.
- Fix pop from empty stack.
- Add recursive operator new tests.
- Disable argument invalidation when the allocator was inlined (needed for
those tests to work)
In https://reviews.llvm.org/D40560#957653, @xazax.hun wrote:
> I
1 - 100 of 105 matches
Mail list logo