Author: maskray
Date: Tue Jul 24 23:57:31 2018
New Revision: 337892
URL: http://llvm.org/viewvc/llvm-project?rev=337892&view=rev
Log:
cc1_main: fix -Wsign-compare on FreeBSD
Its __rlim_t is intentionally signed (__int64_t) because of legacy code
that uses -1 for RLIM_INFINITY.
Modified:
cfe/
ilya-biryukov added a comment.
The mode of operation where compile commands come from the client seems useful,
but I wonder if there's any value in mixing it with `compile_commands.json` and
other CDB plugins.
Do you plan to use the overridden commands in conjunction with CDB plugins or
do you
ilya-biryukov added a comment.
In https://reviews.llvm.org/D49523#1174086, @arphaman wrote:
> We actually need both mechanisms. I posted the didChangeConfiguration
> extension to https://reviews.llvm.org/D49758.
Why do we need both? Can we have only the one from
https://reviews.llvm.org/D4975
yvvan added a comment.
@ilya-biryukov I would name this revision "for test purpose". It works at some
extent with the current clangd version and can be used by someone else. But
since we had a discussion about YAML being a temporary solution I never planned
to commit this revision.
So yes, I t
kbobyrev added inline comments.
Comment at: clang-tools-extra/clangd/index/dex/Trigram.cpp:56
+ for (int I = LowercaseIdentifier.size() - 1; I >= 0; --I) {
+Next[I] = {{NextTail, NextHead, NextNextHead}};
+NextTail = Roles[I] == Tail ? I : 0;
sammccall w
kbobyrev updated this revision to Diff 157196.
kbobyrev marked 7 inline comments as done.
kbobyrev added a comment.
Address last round of comments.
https://reviews.llvm.org/D49591
Files:
clang-tools-extra/clangd/CMakeLists.txt
clang-tools-extra/clangd/index/dex/Token.h
clang-tools-extra/c
rjmccall added a comment.
In https://reviews.llvm.org/D49718#1174281, @ahatanak wrote:
> Merge pushBlockObjectRelease and enterByrefCleanup.
>
> In https://reviews.llvm.org/D49718#1174113, @rjmccall wrote:
>
> > Heh, okay. It looks like the runtime doesn't do anything different, but I
> > think
EricWF added a comment.
In https://reviews.llvm.org/D49774#1174565, @mclow.lists wrote:
> I haven't reviewed this closely, but you might want to look at
> http://wg21.link/P0355, where we added a `file_clock` and `file_time` types.
Thanks for the information. It doesn't look to have too much b
mgorny accepted this revision.
mgorny added a comment.
This revision is now accepted and ready to land.
Looks good, thanks!
Repository:
rCXXA libc++abi
https://reviews.llvm.org/D49765
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http:/
Author: ericwf
Date: Tue Jul 24 21:46:32 2018
New Revision: 337888
URL: http://llvm.org/viewvc/llvm-project?rev=337888&view=rev
Log:
Fix bugs in create_directory implementation.
Libc++ was incorrectly reporting an error when the target of create_directory
already exists, but was not a directory.
This revision was automatically updated to reflect the committed changes.
Closed by commit rL337887: CodeGen: use non-zero memset when possible for
automatic variables (authored by jfb, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D49771
Author: jfb
Date: Tue Jul 24 21:29:03 2018
New Revision: 337887
URL: http://llvm.org/viewvc/llvm-project?rev=337887&view=rev
Log:
CodeGen: use non-zero memset when possible for automatic variables
Summary:
Right now automatic variables are either initialized with bzero followed by a
few stores,
mclow.lists added a comment.
I haven't reviewed this closely, but you might want to look at
http://wg21.link/P0355, where we added a `file_clock` and `file_time` types.
Repository:
rCXX libc++
https://reviews.llvm.org/D49774
___
cfe-commits mail
jfb marked an inline comment as done.
jfb added a comment.
Addressed all comments.
Comment at: lib/CodeGen/CGDecl.cpp:956-957
+class BytePattern {
+ uint8_t Val;
+ enum class ValueType { Specific, Any, None } Type;
+ BytePattern(ValueType Type) : Type(Type) {}
--
mclow.lists closed this revision.
mclow.lists added a comment.
Committed as 337885 after changing `constexpr` -> `TEST_CONSTEXPR_CXX14`
https://reviews.llvm.org/D49773
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi
jfb updated this revision to Diff 157191.
jfb marked 2 inline comments as done.
jfb added a comment.
- Use short to test padding between array elements.
- Define enum class storage type; swap order of if / else to make it more
readable.
Repository:
rC Clang
https://reviews.llvm.org/D49771
F
Author: marshall
Date: Tue Jul 24 21:21:21 2018
New Revision: 337885
URL: http://llvm.org/viewvc/llvm-project?rev=337885&view=rev
Log:
New test support for comparisons. Reviewed as https://reviews.llvm.org/D49773
Added:
libcxx/trunk/test/support/test_comparisons.h
Added: libcxx/trunk/test/su
Author: ericwf
Date: Tue Jul 24 21:21:59 2018
New Revision: 337886
URL: http://llvm.org/viewvc/llvm-project?rev=337886&view=rev
Log:
Fix missing includes in format_string.hpp helper
Modified:
libcxx/trunk/test/support/format_string.hpp
Modified: libcxx/trunk/test/support/format_string.hpp
UR
EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.
Otherwise, I'm quite happy with this patch.
The only downside I see is if `testComparisons6` fails, it will report the
source of the failure as being in the header, and not provide information
Author: ericwf
Date: Tue Jul 24 20:41:31 2018
New Revision: 337884
URL: http://llvm.org/viewvc/llvm-project?rev=337884&view=rev
Log:
Make explicitly require C++11.
Previously the didn't guard its
contents in any dialect. However, the implementation implicitly
requires at least C++11, and the te
Author: ericwf
Date: Tue Jul 24 20:31:48 2018
New Revision: 337883
URL: http://llvm.org/viewvc/llvm-project?rev=337883&view=rev
Log:
Ensure path::iterator and PathParser share the same enumeration values.
To avoid exposing implementation details, path::iterator and PathParser
both implicitly used
mclow.lists added a comment.
> Can they be non-constexpr in C++11 and still function?
Sure. I'll fix that.
https://reviews.llvm.org/D49773
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co
EricWF added a comment.
This is the final change I need to land before adding
``, so I don't plan to let this review linger long.
If you have any strong objections, please speak now or forever hold your peace
(or at least until the next ABI break, whichever comes first).
Repository:
rCXX li
EricWF added a comment.
Some of this stuff is marked `constexpr` despite it requiring C++14 constexpr
semantics. Can they be non-constexpr in C++11 and still function? Or should
the header explicitly require C++14 to be included?
https://reviews.llvm.org/D49773
Author: phosek
Date: Tue Jul 24 20:01:35 2018
New Revision: 337881
URL: http://llvm.org/viewvc/llvm-project?rev=337881&view=rev
Log:
[profile] Support profiling runtime on Fuchsia
This ports the profiling runtime on Fuchsia and enables the
instrumentation. Unlike on other platforms, Fuchsia doesn
EricWF created this revision.
EricWF added reviewers: mclow.lists, ldionne, joerg, arthur.j.odwyer.
Herald added subscribers: cfe-commits, christof.
The ``file_time_type`` time point is used to represent the write times for
files.
Its job is to act as part of a C++ wrapper for less ideal system i
Author: ericwf
Date: Tue Jul 24 19:53:53 2018
New Revision: 337880
URL: http://llvm.org/viewvc/llvm-project?rev=337880&view=rev
Log:
Add design docs for upcoming file_time_type change.
In upcoming changes to filesystem I plan to change file_time_type
to use __int128_t as its underlying representa
0x8000- updated this revision to Diff 157187.
0x8000- added a comment.
Update links in documentation
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D49114
Files:
clang-tidy/cppcoreguidelines/CppCoreGuidelinesTidyModule.cpp
clang-tidy/readability/CMakeLists.txt
clan
Eugene.Zelenko added inline comments.
Comment at: docs/clang-tidy/checks/readability-magic-numbers.rst:56
+By default only `0`, `1` and `-1` integer values are accepted without a
warning.
+This can be overridden with the 'IgnoredIntegerValues' option. In addition,
+the `0.0` fl
0x8000- updated this revision to Diff 157186.
0x8000- added a comment.
Fix a few typos
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D49114
Files:
clang-tidy/cppcoreguidelines/CppCoreGuidelinesTidyModule.cpp
clang-tidy/readability/CMakeLists.txt
clang-tidy/readabi
mclow.lists added a comment.
Usage looks like this:
AssertComparisons6AreNoexcept();
AssertComparisons6ReturnBool();
static_assert(testComparisons6Values( 5U, 5U), "");
static_assert(testComparisons6Values( 5U, 10U), "");
static_assert(testComparisons6Values(10U, 5U), "");
https:
mclow.lists created this revision.
mclow.lists added reviewers: ldionne, EricWF.
As I've been plowing through the `` stuff, I found myself writing tests
to make sure that the comparisons for a bunch of different types are "sane".
So I factored them out into something that can be used over and ov
Eugene.Zelenko added inline comments.
Comment at: clang-tidy/readability/MagicNumbersCheck.h:42
+return false;
+ };
+ bool isSyntheticValue(const clang::SourceManager *SourceManager,
Unnecessary semicolon. Please also add empty line after.
===
0x8000- added a comment.
@aaron.ballman , @JonasToth , @Eugene.Zelenko - would you be so kind to do one
more review pass and let me know if there are any blockers or if this is ready
to merge? I do not have commit privileges, so if it is alright, feel free to
merge it.
Repository:
rCTE
Author: marshall
Date: Tue Jul 24 18:55:25 2018
New Revision: 337879
URL: http://llvm.org/viewvc/llvm-project?rev=337879&view=rev
Log:
Mark as complete
Modified:
libcxx/trunk/www/cxx2a_status.html
Modified: libcxx/trunk/www/cxx2a_status.html
URL:
http://llvm.org/viewvc/llvm-project/libcxx/
This revision was automatically updated to reflect the committed changes.
Closed by commit rL337877: [CMake] Link static libunwind and libc++abi into
libc++ in Fuchsia toolchain (authored by phosek, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://review
Author: phosek
Date: Tue Jul 24 18:44:22 2018
New Revision: 337877
URL: http://llvm.org/viewvc/llvm-project?rev=337877&view=rev
Log:
[CMake] Link static libunwind and libc++abi into libc++ in Fuchsia toolchain
When building libc++ for Fuchsia, we want to distribute shared libc++,
libc++abi and li
phosek updated this revision to Diff 157182.
Repository:
rC Clang
https://reviews.llvm.org/D49628
Files:
clang/cmake/caches/Fuchsia-stage2.cmake
Index: clang/cmake/caches/Fuchsia-stage2.cmake
===
--- clang/cmake/caches/Fuchsia
This revision was automatically updated to reflect the committed changes.
Closed by commit rC337876: [analyzer] Syntactic matcher for leaks associated
with run loop and… (authored by george.karpenkov, committed by ).
Herald added a subscriber: cfe-commits.
Changed prior to commit:
https://revie
Author: george.karpenkov
Date: Tue Jul 24 18:27:15 2018
New Revision: 337876
URL: http://llvm.org/viewvc/llvm-project?rev=337876&view=rev
Log:
[analyzer] Syntactic matcher for leaks associated with run loop and
autoreleasepool
A checker for detecting leaks resulting from allocating temporary
aut
vsapsai added a comment.
In https://reviews.llvm.org/D49518#1168038, @bruno wrote:
> Hi Volodymyr, thanks for improving this.
>
> > Need to double check what tests we have when using relative path names at
> > the root level.
>
> Another interesting place to look at is
> `unittests/Basic/Virtua
bogner accepted this revision.
bogner added a comment.
This revision is now accepted and ready to land.
Seems straightforward and correct to me.
Comment at: lib/CodeGen/CGDecl.cpp:956-957
+class BytePattern {
+ uint8_t Val;
+ enum class ValueType { Specific, Any, None } Type;
morehouse added inline comments.
Comment at: clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp:190
+ builder.setMCJITMemoryManager(
+ std::unique_ptr(RTDyldMM));
+ builder.setOptLevel(OLvl);
emmettneyman wrote:
> morehouse wrote:
> > These 3 lines can b
Quuxplusone added inline comments.
Comment at: test/CodeGen/init.c:202
+ union U { char c; int i; };
+ struct S { union U u; int i; };
+ struct S arr[5] = { { {0xF0}, 0xF0F0F0F0 }, { {0xF0}, 0xF0F0F0F0 }, {
{0xF0}, 0xF0F0F0F0 }, { {0xF0}, 0xF0F0F0F0 }, { {0xF0}, 0xF0F0F0F0 },
emmettneyman added inline comments.
Comment at: clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp:190
+ builder.setMCJITMemoryManager(
+ std::unique_ptr(RTDyldMM));
+ builder.setOptLevel(OLvl);
morehouse wrote:
> These 3 lines can be combined to `builde
jfb created this revision.
jfb added a reviewer: dexonsmith.
Herald added a subscriber: cfe-commits.
Right now automatic variables are either initialized with bzero followed by a
few stores, or memcpy'd from a synthesized global. We end up encountering a
fair amount of code where memcpy of non-z
tra planned changes to this revision.
tra added a comment.
Ugh. Apparently moving this code up just disabled module destructor. :-( That
explains why we no longer crash.
https://reviews.llvm.org/D49763
___
cfe-commits mailing list
cfe-commits@lists
arphaman added a comment.
Thanks for working on this!
It looks like we only have one client of `getNearestOption`. Wouldn't it make
sense to fold the check into the function itself?
Repository:
rC Clang
https://reviews.llvm.org/D49736
___
cfe-co
Author: phosek
Date: Tue Jul 24 16:42:51 2018
New Revision: 337868
URL: http://llvm.org/viewvc/llvm-project?rev=337868&view=rev
Log:
[CMake] Include CMakeDependentOption in libunwind
This should resolve the breakage introduced in r337867 which introduced
the use of cmake_dependent_option without
ssijaric created this revision.
ssijaric added reviewers: rnk, mstorsjo, TomTan, haripul.
Herald added a reviewer: javed.absar.
Herald added subscribers: cfe-commits, chrib, kristof.beyls.
Microsoft's C++ object model for ARM64 is the same as that for X86_64. For
example, small structs with non-
This revision was automatically updated to reflect the committed changes.
Closed by commit rCXX337867: [CMake] Option to control whether shared/static
library is installed (authored by phosek, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D49573?vs=156382&id=157170#toc
Repo
Author: phosek
Date: Tue Jul 24 16:27:51 2018
New Revision: 337867
URL: http://llvm.org/viewvc/llvm-project?rev=337867&view=rev
Log:
[CMake] Option to control whether shared/static library is installed
Currently it's only possible to control whether shared or static library
build of libc++, libc+
Author: phosek
Date: Tue Jul 24 16:27:51 2018
New Revision: 337867
URL: http://llvm.org/viewvc/llvm-project?rev=337867&view=rev
Log:
[CMake] Option to control whether shared/static library is installed
Currently it's only possible to control whether shared or static library
build of libc++, libc+
Author: phosek
Date: Tue Jul 24 16:27:51 2018
New Revision: 337867
URL: http://llvm.org/viewvc/llvm-project?rev=337867&view=rev
Log:
[CMake] Option to control whether shared/static library is installed
Currently it's only possible to control whether shared or static library
build of libc++, libc+
joerg added a comment.
Depends a bit on the platform, __cxa_atexit on most modern ELF systems,
fallback to atexit. If the global dtor is run too late, it smells like a
missing library dependency. They are executed in topological order after all.
https://reviews.llvm.org/D49763
_
Author: george.karpenkov
Date: Tue Jul 24 16:23:33 2018
New Revision: 337866
URL: http://llvm.org/viewvc/llvm-project?rev=337866&view=rev
Log:
Revert "[analyzer] Extend NoStoreFuncVisitor to insert a note on IVars"
This reverts commit a9e21bd727112cd69eabc1af648c5da6b773d06e.
Reverted because the
Author: george.karpenkov
Date: Tue Jul 24 16:14:29 2018
New Revision: 337864
URL: http://llvm.org/viewvc/llvm-project?rev=337864&view=rev
Log:
[analyzer] Extend NoStoreFuncVisitor to insert a note on IVars
The note is added in the following situation:
- We are throwing a nullability-related war
tra added a comment.
In https://reviews.llvm.org/D49763#1174283, @joerg wrote:
> Can this ever end up in a shared library? If yes, please use the normal logic
> for creating a global destructor. atexit is not very friendly to dlopen...
Yes, it can end up in a shared library. What would be the
bviyer created this revision.
bviyer added reviewers: arphaman, erik.pilkington, dexonsmith.
Herald added a subscriber: cfe-commits.
When the error has occurred for one of the variables inside a template, the Loc
function will try to find the end-location of the final item. If this variable
is d
morehouse added inline comments.
Comment at: clang/tools/clang-fuzzer/handle-llvm/CMakeLists.txt:17
-# Depend on LLVM IR intrinsic generation.
+# Depend on LLVM IR instrinsic generation.
set(handle_llvm_deps intrinsics_gen)
Typo introduced here.
phosek created this revision.
phosek added reviewers: mgorny, EricWF.
Herald added subscribers: cfe-commits, ldionne, christof.
This was changed unintentionally in r335809.
Repository:
rCXXA libc++abi
https://reviews.llvm.org/D49765
Files:
libcxxabi/CMakeLists.txt
Index: libcxxabi/CMakeL
joerg added a comment.
Can this ever end up in a shared library? If yes, please use the normal logic
for creating a global destructor. atexit is not very friendly to dlopen...
https://reviews.llvm.org/D49763
___
cfe-commits mailing list
cfe-commits
ahatanak updated this revision to Diff 157149.
ahatanak added a comment.
Merge pushBlockObjectRelease and enterByrefCleanup.
In https://reviews.llvm.org/D49718#1174113, @rjmccall wrote:
> Heh, okay. It looks like the runtime doesn't do anything different, but I
> think it's probably more corre
Author: d0k
Date: Tue Jul 24 15:47:16 2018
New Revision: 337862
URL: http://llvm.org/viewvc/llvm-project?rev=337862&view=rev
Log:
[Sema] Destroy tokens in DeclaratorChunk params
Otherwise this leaks in some edge cases.
Modified:
cfe/trunk/include/clang/Sema/DeclSpec.h
Modified: cfe/trunk/in
jlebar accepted this revision.
jlebar added inline comments.
This revision is now accepted and ready to land.
Comment at: clang/lib/CodeGen/CGCUDANV.cpp:379
+ // Create destructor and register it with atexit() the way NVCC does it.
Doing
+ // it during regular destructor phase
mikerice added a comment.
In https://reviews.llvm.org/D46652#1164010, @thakis wrote:
> Also, were you planning on also adding support for the (filename-less version
> of) hdrstop pragma? After this change, that should probably be fairly
> straightforward.
Thanks for taking a look. I'll be go
tra created this revision.
tra added reviewers: jlebar, timshen.
Herald added subscribers: bixia, sanjoy.
There's apparently a race between fatbin destructors registered by us
and some internal calls registered by CUDA runtime from cudaRegisterFatbin.
Moving fatbin de-registration to atexit() was
vsk added inline comments.
Comment at: lib/CodeGen/CodeGenFunction.h:380
+ /// True if sanitizer checks for current pointer value are generated.
+ bool PointerChecksAreEmitted = false;
+
rjmccall wrote:
> sepavloff wrote:
> > rjmccall wrote:
> > > I don't think
mclow.lists added a comment.
With this change, only one copy of the c++abi headers gets copied - into
`include/c++/v1/`.
LGTM.
Repository:
rCXX libc++
https://reviews.llvm.org/D49752
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http:/
emmettneyman updated this revision to Diff 157138.
emmettneyman added a comment.
Cleaned up code
Tried to get rid of ParseCommandLineOptions() call but could not figure out
how to initialize a PassInfo object without it.
Repository:
rC Clang
https://reviews.llvm.org/D49526
Files:
clang/to
vsk accepted this revision.
vsk added a comment.
This revision is now accepted and ready to land.
In https://reviews.llvm.org/D49760#1174141, @nickdesaulniers wrote:
> Thanks for the review. Today is my first day committing to clang!
Welcome :).
LGTM. Feel free to commit this yourself once you
Author: d0k
Date: Tue Jul 24 14:50:06 2018
New Revision: 337860
URL: http://llvm.org/viewvc/llvm-project?rev=337860&view=rev
Log:
[clangd] Guard fuzzer against empty inputs.
Modified:
clang-tools-extra/trunk/clangd/fuzzer/ClangdFuzzer.cpp
Modified: clang-tools-extra/trunk/clangd/fuzzer/Clang
nickdesaulniers updated this revision to Diff 157134.
nickdesaulniers added a comment.
- prefer auto when type is specified on RHS of assignment.
Repository:
rC Clang
https://reviews.llvm.org/D49760
Files:
include/clang/Sema/SemaInternal.h
lib/Sema/SemaTemplateDeduction.cpp
lib/Sema/Se
nickdesaulniers marked an inline comment as done.
nickdesaulniers added a comment.
Thanks for the review. Today is my first day committing to clang!
Repository:
rC Clang
https://reviews.llvm.org/D49760
___
cfe-commits mailing list
cfe-commits@lis
Hello everyone,
LLVM buildmaster will be updated and restarted after 6PM Pacific time today.
Thanks
Galina
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vsk added a comment.
Thanks for doing this!
Comment at: include/clang/Sema/SemaInternal.h:120
+ if (const TemplateTypeParmType *TTP =
+ UPP.first.dyn_cast())
+return std::make_pair(TTP->getDepth(), TTP->getIndex());
Perhaps 'const auto *TTP = ...'
nickdesaulniers created this revision.
nickdesaulniers added reviewers: srhines, pirama.
Herald added a subscriber: cfe-commits.
Continuing off of:
https://reviews.llvm.org/D38382
Fixes:
https://bugs.llvm.org/show_bug.cgi?id=12176
Repository:
rC Clang
https://reviews.llvm.org/D49760
Files:
rjmccall added a comment.
In https://reviews.llvm.org/D49718#1173883, @ahatanak wrote:
> Address review comments.
>
> I think I should be able to merge pushBlockObjectRelease and
> enterByrefCleanup, but the BlockFieldFlags that are passed are different. We
> set BLOCK_FIELD_IS_WEAK in addition
Author: rsmith
Date: Tue Jul 24 14:18:30 2018
New Revision: 337857
URL: http://llvm.org/viewvc/llvm-project?rev=337857&view=rev
Log:
Don't lifetime-extend or track lifetime problems through the LHS of '->*'.
Fixes a false-positive warning found by selfhost.
Modified:
cfe/trunk/lib/AST/Expr.c
rjmccall added inline comments.
Comment at: lib/CodeGen/CodeGenFunction.h:380
+ /// True if sanitizer checks for current pointer value are generated.
+ bool PointerChecksAreEmitted = false;
+
sepavloff wrote:
> rjmccall wrote:
> > I don't think this is a good w
arphaman added a comment.
In https://reviews.llvm.org/D49523#1171484, @ilya-biryukov wrote:
> The extensions itself seems like a reasonable way to provide compile commands
> for the individual files. In case didChangeConfiguration does not work for
> you for some reason, happy to take a look at
arphaman created this revision.
arphaman added reviewers: malaperle-ericsson, ilya-biryukov, sammccall, jkorous.
arphaman added a project: clang-tools-extra.
Herald added subscribers: dexonsmith, MaskRay, ioeric.
This patch is based on https://reviews.llvm.org/D49523.
It extends the 'workspace/di
steveire updated this revision to Diff 157116.
steveire retitled this revision from "Inline DeclarationNameInfo::getLocEnd
into callers" to "Avoid returning an invalid end source loc".
steveire edited the summary of this revision.
Repository:
rC Clang
https://reviews.llvm.org/D49100
Files:
Author: rupprecht
Date: Tue Jul 24 13:28:07 2018
New Revision: 337850
URL: http://llvm.org/viewvc/llvm-project?rev=337850&view=rev
Log:
Revert "[VFS] Cleanups to VFS interfaces."
This reverts commit r337834 due to test failures.
Modified:
cfe/trunk/include/clang/Basic/VirtualFileSystem.h
jhibbits created this revision.
Herald added subscribers: cfe-commits, kbarton, nemanjai.
r337347 added support for the Signal Processing Engine (SPE) to LLVM.
This follows that up with the clang side.
Repository:
rC Clang
https://reviews.llvm.org/D49754
Files:
include/clang/Driver/Options
rupprecht added a comment.
Looks like this caused some test failures
(http://lab.llvm.org:8011/builders/clang-cmake-aarch64-quick/builds/14441/steps/ninja%20check%201),
e.g.:
[ RUN ] GoToInclude.All
/home/buildslave/buildslave/clang-cmake-aarch64-quick/llvm/tools/clang/tools/extra/uni
phosek added a comment.
This was pointed out by @mclow.lists on IRC in our discussion related to
https://reviews.llvm.org/D49711, I believe this change should resolve that
issue.
Repository:
rCXX libc++
https://reviews.llvm.org/D49752
___
cfe-c
phosek created this revision.
phosek added reviewers: mclow.lists, EricWF, ldionne.
Herald added subscribers: cfe-commits, christof, mgorny.
This is a refinement on r337833. Previously we were installing two
copies of c++abi headers in libc++ build directory, one in
include/c++build and another on
george.karpenkov requested changes to this revision.
george.karpenkov added inline comments.
This revision now requires changes to proceed.
Comment at: lib/StaticAnalyzer/Core/SimpleSValBuilder.cpp:1300
+ Loc::isLocType(S->getRHS()->getType())) {
+SVal V = SVB.ma
mgrang updated this revision to Diff 157106.
mgrang added a comment.
Thanks @rsmith. I have addressed your comments.
https://reviews.llvm.org/D48862
Files:
lib/Driver/ToolChains/Gnu.cpp
lib/Driver/ToolChains/Linux.cpp
test/Driver/Inputs/openembedded_aarch64_linux_tree/usr/include/c++/6.3
sepavloff added inline comments.
Comment at: lib/CodeGen/CodeGenFunction.h:380
+ /// True if sanitizer checks for current pointer value are generated.
+ bool PointerChecksAreEmitted = false;
+
rjmccall wrote:
> I don't think this is a good way of doing this. U
lebedev.ri added a comment.
In https://reviews.llvm.org/D48958#1173860, @vsk wrote:
> LGTM, although I think it'd be helpful to have another +1 just to be safe.
Thank you for the review!
It would indeed be great if someone else could take a look, especially since we
are **so** close to the bra
NoQ created this revision.
NoQ added reviewers: dcoughlin, xazax.hun, a.sidorin, george.karpenkov, szepet,
rnkovacs, mikhail.ramalho.
Herald added subscribers: cfe-commits, baloghadamsoftware.
Add one more defensive check to prevent crashes on trying to simplify a
`SymSymExpr` with different `Lo
sepavloff updated this revision to Diff 157102.
sepavloff added a comment.
Updated patch
- Modified check generation. Now for each 'new' expression compiler tries to
generate the checks. It solves problem of 'nested' new expressions, which
appear when 'new' is used in default arguments.
- Field
george.karpenkov requested changes to this revision.
george.karpenkov added a comment.
This revision now requires changes to proceed.
Minor nits mostly.
Comment at: include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h:411
+ /// for bad reasons, returning null, even in p
ahatanak added inline comments.
Comment at: lib/CodeGen/CGBlocks.cpp:1905
+case BlockCaptureEntityKind::None:
+ llvm_unreachable("unexpected BlockCaptureEntityKind");
}
rjmccall wrote:
> These two switches differ only in (1) what kind of cleanup the
ahatanak updated this revision to Diff 157095.
ahatanak marked 2 inline comments as done.
ahatanak added a comment.
Address review comments.
I think I should be able to merge pushBlockObjectRelease and enterByrefCleanup,
but the BlockFieldFlags that are passed are different. We set
BLOCK_FIELD_
vsk accepted this revision.
vsk added a comment.
This revision is now accepted and ready to land.
LGTM, although I think it'd be helpful to have another +1 just to be safe.
I did two small experiments with this using a Stage1 Rel+Asserts compiler:
Stage2 Rel+Asserts build of llvm-tblgen:
ninja l
rsmith added a comment.
Does this check destructors of nested aggregate initializations in the case
where brace elision occurs? I don't think just checking the top level of the
SK_ListInitialization is enough. Perhaps it would make more sense to mark the
dtors used from InitListChecker (in non-
pcc requested changes to this revision.
pcc added a comment.
This revision now requires changes to proceed.
As I mentioned in https://bugs.llvm.org/show_bug.cgi?id=38281 I don't see a
good reason to do this. Also extern_weak is not an appropriate linkage for
definitions, only declarations.
htt
1 - 100 of 188 matches
Mail list logo
