rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
Alright, thanks, LGTM.
https://reviews.llvm.org/D49083
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/
0x8000- updated this revision to Diff 156430.
0x8000- added a comment.
Avoid parsing and reformatting the input literal - just print the original
source code.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D49114
Files:
clang-tidy/cppcoreguidelines/CppCoreGuidelinesTi
Author: rjmccall
Date: Thu Jul 19 22:40:12 2018
New Revision: 337525
URL: http://llvm.org/viewvc/llvm-project?rev=337525&view=rev
Log:
Document -fobjc-weak as an extension.
Fixes rdar://24091053.
Modified:
cfe/trunk/docs/LanguageExtensions.rst
Modified: cfe/trunk/docs/LanguageExtensions.rst
Author: rjmccall
Date: Thu Jul 19 22:40:09 2018
New Revision: 337524
URL: http://llvm.org/viewvc/llvm-project?rev=337524&view=rev
Log:
Fix and improve the ARC spec's wording about unmanaged objects.
Modified:
cfe/trunk/docs/AutomaticReferenceCounting.rst
Modified: cfe/trunk/docs/AutomaticRef
0x8000- added a comment.
./tools/clang/tools/extra/clang-tidy/tool/run-clang-tidy.py
-clang-tidy-binary ../llvm.rel/bin/clang-tidy
-checks="-*,readability-magic-numbers" -j 12 -p ../llvm.rel -j 12 -quiet >
/tmp/llvm.magic
grep "warning:" /tmp/llvm.magic | cut -d: -f5 | cut -d" " -f2 | s
sepavloff created this revision.
sepavloff added reviewers: rsmith, rjmccall, ikudrin.
With this change compiler generates alignment checks for wider range
of types. Previously such checks were generated only for the record types
with non-trivial default constructor. So the types like:
struct a
0x8000- updated this revision to Diff 156425.
0x8000- added a comment.
Filter out synthetic integers (such as _LINE_) from the report.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D49114
Files:
clang-tidy/cppcoreguidelines/CppCoreGuidelinesTidyModule.cpp
clang-tidy
yaxunl updated this revision to Diff 156424.
yaxunl marked an inline comment as done.
yaxunl added a comment.
Revised by John's comments.
https://reviews.llvm.org/D49083
Files:
lib/CodeGen/CGCUDANV.cpp
test/CodeGenCUDA/device-stub.cu
Index: test/CodeGenCUDA/device-stub.cu
=
phosek abandoned this revision.
phosek added a comment.
Herald added a subscriber: ldionne.
Replaced by https://reviews.llvm.org/D49587.
Repository:
rCXXA libc++abi
https://reviews.llvm.org/D41754
___
cfe-commits mailing list
cfe-commits@lists.ll
phosek abandoned this revision.
phosek added a comment.
Replaced by https://reviews.llvm.org/D49587.
Repository:
rUNW libunwind
https://reviews.llvm.org/D41753
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/
phosek abandoned this revision.
phosek added a comment.
Herald added a subscriber: ldionne.
Replaced by https://reviews.llvm.org/D49587.
Repository:
rCXX libc++
https://reviews.llvm.org/D41755
___
cfe-commits mailing list
cfe-commits@lists.llvm.o
0x8000- updated this revision to Diff 156422.
0x8000- added a comment.
Add a (presently failing) test for not tripping up on __LINE__ through several
layers of macro expansion (as in GoogleTest library). This creates a lot of
false positives in the unit tests and needs to be fixed.
Rep
yaxunl marked an inline comment as done.
yaxunl added inline comments.
Comment at: lib/CodeGen/CGCUDANV.cpp:578
+ DtorBuilder.CreateStore(Zero, GpuBinaryAddr);
+ DtorBuilder.CreateBr(ExitBlock);
+DtorBuilder.SetInsertPoint(ExitBlock);
rjmccall wrote
Author: ericwf
Date: Thu Jul 19 18:51:48 2018
New Revision: 337520
URL: http://llvm.org/viewvc/llvm-project?rev=337520&view=rev
Log:
Fix two test failures in
First, didn't correctly guard
against min/max macros. This adds the proper push/pop macro guards.
Second, an internal time helper had be
Author: ericwf
Date: Thu Jul 19 18:44:33 2018
New Revision: 337519
URL: http://llvm.org/viewvc/llvm-project?rev=337519&view=rev
Log:
Use _LIBCPP_UNREACHABLE to convince GCC that non-void functions actually always
return
Modified:
libcxx/trunk/include/experimental/filesystem
libcxx/trunk/
phosek added inline comments.
Comment at: libcxx/lib/CMakeLists.txt:269
+AND (TARGET cxxabi_static OR HAVE_LIBCXXABI))
+#if ((TARGET ${LIBCXX_CXX_ABI_LIBRARY}) OR
+#(${LIBCXX_CXX_ABI_LIBRARY} MATCHES "cxxabi(_static|_shared)?" AND
HAVE_LIBCXXABI))
---
NoQ added a comment.
Oh, this FIXME, i almost forgot about that. Not sure if we should focus on this
now because it's kinda premature optimization, especially after
@george.karpenkov has fixed a large performance problem that caused `VisitNode`
to be called like ~30 times more often than necess
phosek created this revision.
phosek added reviewers: ldionne, EricWF.
Herald added subscribers: cfe-commits, christof, mgorny.
This is a follow-up to r335809 and r337118. While libc++ headers are now
installed into the right location in both standard as well as multiarch
runtimes layout, turned
phosek updated this revision to Diff 156416.
phosek marked 8 inline comments as done.
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D49502
Files:
libcxx/CMakeLists.txt
libcxx/cmake/Modules/HandleLibCXXABI.cmake
libcxx/lib/CMakeLists.txt
libcxxabi
Author: ericwf
Date: Thu Jul 19 18:25:06 2018
New Revision: 337517
URL: http://llvm.org/viewvc/llvm-project?rev=337517&view=rev
Log:
cleanup test assertion inside library
Modified:
libcxx/trunk/src/experimental/filesystem/operations.cpp
Modified: libcxx/trunk/src/experimental/filesystem/oper
NoQ added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/DanglingInternalBufferChecker.cpp:253
+ allocation_state::getContainerObjRegion(N->getState(), PtrToBuf);
+ const auto *TypedRegion = dyn_cast(ObjRegion);
+ QualType ObjTy = TypedRegion->getValueType();
---
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
LGTM!
Repository:
rC Clang
https://reviews.llvm.org/D49553
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mai
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
Thanks. A couple tiny things and then LGTM.
Comment at: docs/Block-ABI-Apple.rst:69
+// block is a no-op, which is exactly how global blocks are handled.
+
+
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
Thanks, looks good.
https://reviews.llvm.org/D49294
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mai
rjmccall added inline comments.
Comment at: lib/CodeGen/CGCUDANV.cpp:466
+ CtorBuilder.CreateStore(RegisterFatbinCall, GpuBinaryAddr);
+ CtorBuilder.CreateBr(ExitBlock);
+CtorBuilder.SetInsertPoint(ExitBlock);
I meant more putting all the code for I
pcc requested changes to this revision.
pcc added a comment.
This revision now requires changes to proceed.
Sorry, I noticed that this patch is missing a test case. Can you add one please?
https://reviews.llvm.org/D49109
___
cfe-commits mailing list
pcc accepted this revision.
pcc added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D49109
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
acoomans added a comment.
An alternative to changing the whole behavior would be to make it a parameter /
configuration option.
Repository:
rC Clang
https://reviews.llvm.org/D49580
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://l
acoomans created this revision.
Herald added a subscriber: cfe-commits.
When running clang-format on specific lines, two types of indentation could be
expectedi:
- absolute indentation: the indentation one would get if the whole file were
formatted; or
- relative indentation: keep the indentati
ahatanak updated this revision to Diff 156404.
ahatanak marked 2 inline comments as done.
ahatanak added a comment.
Add a comment that explains the meaning of BLOCK_IS_NOESCAPE to the docs.
Rename function needsCopyDispose to needsCopyDisposeHelpers.
Repository:
rC Clang
https://reviews.llvm
tra marked 2 inline comments as done.
tra added a comment.
Ben, PTAL.
Comment at: clang/lib/Headers/__clang_cuda_device_functions.h:1080
+ unsigned int r;
+ asm("vabsdiff2.u32.u32.u32.sat %0,%1,%2,0;" : "=r"(r) : "r"(__a), "r"(__b));
+ return r;
bkramer wrot
yaxunl updated this revision to Diff 156403.
yaxunl marked 5 inline comments as done.
yaxunl added a comment.
Revised by John's comments.
https://reviews.llvm.org/D49294
Files:
lib/Sema/SemaCast.cpp
lib/Sema/SemaOverload.cpp
test/CodeGenCXX/address-space-cast.cpp
Index: test/CodeGenCXX/
tra updated this revision to Diff 156397.
tra added a comment.
Fixed the issues pointed out by bkramer@.
Apparently. sat does not matter for vabsdiff instruction with unsigned operands.
My tests were also missing __vabsssN.
https://reviews.llvm.org/D49274
Files:
clang/lib/Headers/__clang_cu
thomasanderson updated this revision to Diff 156399.
thomasanderson marked 5 inline comments as done.
https://reviews.llvm.org/D49109
Files:
lib/CodeGen/ItaniumCXXABI.cpp
Index: lib/CodeGen/ItaniumCXXABI.cpp
===
--- lib/CodeGen/It
0x8000- updated this revision to Diff 156392.
0x8000- added a comment.
Small refactoring and documentation update.
Revert built-in acceptable integers to -1, 0 and 1 and document them.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D49114
Files:
clang-tidy/cppcoreguid
tra updated this revision to Diff 156386.
tra added a comment.
Fixed inline asm syntax.
Added workaround for the bug in __vmaxs2() discovered during testing().
I've got set of tests for these functions that I'll add to test-suite shortly.
AFAICT this implementation matches nvidia's bit-to-bit.
Author: rsmith
Date: Thu Jul 19 16:24:41 2018
New Revision: 337508
URL: http://llvm.org/viewvc/llvm-project?rev=337508&view=rev
Log:
Fix typo causing assert in self-host.
Modified:
cfe/trunk/lib/CodeGen/CGExprConstant.cpp
cfe/trunk/test/CodeGen/init.c
Modified: cfe/trunk/lib/CodeGen/CGEx
yaxunl updated this revision to Diff 156383.
yaxunl marked 3 inline comments as done.
yaxunl added a comment.
Revised by John's comments.
https://reviews.llvm.org/D49083
Files:
lib/CodeGen/CGCUDANV.cpp
test/CodeGenCUDA/device-stub.cu
Index: test/CodeGenCUDA/device-stub.cu
=
yaxunl marked 5 inline comments as done.
yaxunl added inline comments.
Comment at: lib/CodeGen/CGCUDANV.cpp:444
+auto HandleValue =
+CtorBuilder.CreateAlignedLoad(GpuBinaryHandle, CGM.getPointerAlign());
+llvm::Constant *Zero =
llvm::Constant::getNullValue(Handle
phosek updated this revision to Diff 156382.
phosek edited subscribers, added: cfe-commits; removed: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D49573
Files:
libcxx/CMakeLists.txt
libcxx/lib/CMakeLists.txt
libcxxabi/CMakeLists.txt
libcxxabi/src/CMakeLists.txt
libunwin
arphaman added a comment.
In https://reviews.llvm.org/D49523#1167553, @malaperle wrote:
> Interesting! We also have a need for passing compilation commands in a
> context where there is no compile_commands.json, but we were thinking of
> putting this in a "didChangeConfiguration" message so tha
Author: maskray
Date: Thu Jul 19 15:45:41 2018
New Revision: 337505
URL: http://llvm.org/viewvc/llvm-project?rev=337505&view=rev
Log:
[docs] Correct -fvisibility-inlines-hidden description
Modified:
cfe/trunk/docs/ClangCommandLineReference.rst
cfe/trunk/include/clang/Driver/Options.td
Mo
echristo added a comment.
Getting close, one inline comment.
Comment at: lib/Driver/ToolChains/Cuda.h:161
bool SupportsProfiling() const override { return false; }
+ bool supportsDebugInfoOption(const llvm::opt::Arg *) const override {
+return false;
I'
pcc added inline comments.
Comment at: clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp:211-224
+ std::string ErrorMsg1;
+ EngineBuilder builder1(std::move(Owner1));
+ builder1.setMArch(MArch);
+ builder1.setMCPU(getCPUStr());
+ builder1.setMAttrs(getFeatureList());
+ b
morehouse added inline comments.
Comment at: clang/tools/clang-fuzzer/handle-llvm/CMakeLists.txt:21
+ set(handle_llvm_deps)
+endif()
How are you doing your diff? Some of these changes are already upstream.
Please rebase
Comment at: clang
rnkovacs created this revision.
rnkovacs added reviewers: NoQ, dcoughlin, xazax.hun, george.karpenkov.
Herald added subscribers: mikhail.ramalho, a.sidorin, dkrupp, szepet,
baloghadamsoftware, whisperity.
Following the discussion at https://reviews.llvm.org/D49360.
Added two more test cases that
0x8000- marked 3 inline comments as done.
0x8000- added inline comments.
Comment at: test/clang-tidy/readability-magic-numbers.cpp:16
+void BuggyFunction() {
+ int BadLocalInt = 6;
+ // CHECK-MESSAGES: :[[@LINE-1]]:21: warning: magic number integer literal 6
[readabili
ldionne added inline comments.
Comment at: libcxx/lib/CMakeLists.txt:269
+AND (TARGET cxxabi_static OR HAVE_LIBCXXABI))
+#if ((TARGET ${LIBCXX_CXX_ABI_LIBRARY}) OR
+#(${LIBCXX_CXX_ABI_LIBRARY} MATCHES "cxxabi(_static|_shared)?" AND
HAVE_LIBCXXABI))
--
emmettneyman updated this revision to Diff 156370.
emmettneyman added a comment.
- Fixed typo that broke build
Repository:
rC Clang
https://reviews.llvm.org/D49526
Files:
clang/tools/clang-fuzzer/CMakeLists.txt
clang/tools/clang-fuzzer/ExampleClangLLVMProtoFuzzer.cpp
clang/tools/clang-
xazax.hun added a comment.
Yeah, I would rather have the cleanups and do extra work in the visitor. But
lets wait what @NoQ thinks.
Repository:
rC Clang
https://reviews.llvm.org/D49568
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
jfb added a subscriber: yaxunl.
jfb added inline comments.
Comment at: lib/Sema/SemaChecking.cpp:3361
+if ((Form != Load && Form != LoadCopy && AtomTy.isConstQualified()) ||
AtomTy.getAddressSpace() == LangAS::opencl_constant) {
Diag(DRE->getLocStart(), diag::
akyrtzi accepted this revision.
akyrtzi added a comment.
This revision is now accepted and ready to land.
Good enough, thanks for looking into this!
Repository:
rC Clang
https://reviews.llvm.org/D49476
___
cfe-commits mailing list
cfe-commits@lis
Author: devnexen
Date: Thu Jul 19 14:50:03 2018
New Revision: 337499
URL: http://llvm.org/viewvc/llvm-project?rev=337499&view=rev
Log:
[CStringSyntaxChecker] Check strlcpy sizeof syntax
The last argument is expected to be the destination buffer size (or less).
Detects if it points to destina
emmettneyman updated this revision to Diff 156364.
emmettneyman added a comment.
- Cleaned up leftover code from mmap memcpy
Repository:
rC Clang
https://reviews.llvm.org/D49526
Files:
clang/tools/clang-fuzzer/CMakeLists.txt
clang/tools/clang-fuzzer/ExampleClangLLVMProtoFuzzer.cpp
clan
emmettneyman updated this revision to Diff 156362.
emmettneyman added a comment.
- Switched to JIT for compilation and execution
Repository:
rC Clang
https://reviews.llvm.org/D49526
Files:
clang/tools/clang-fuzzer/CMakeLists.txt
clang/tools/clang-fuzzer/ExampleClangLLVMProtoFuzzer.cpp
rnkovacs created this revision.
rnkovacs added reviewers: NoQ, xazax.hun, george.karpenkov.
Herald added subscribers: mikhail.ramalho, a.sidorin, dkrupp, szepet,
baloghadamsoftware, whisperity.
In order to avoid scanning the map at each node in the bug reporter visitor,
the `MemRegion` represent
Author: rsmith
Date: Thu Jul 19 14:38:56 2018
New Revision: 337498
URL: http://llvm.org/viewvc/llvm-project?rev=337498&view=rev
Log:
When we choose to use zeroinitializer for a trailing portion of an array
constant, don't convert the rest into a packed struct.
If an array constant has a large non
arsenm added a comment.
ping
https://reviews.llvm.org/D47154
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
phosek added inline comments.
Comment at: libcxx/lib/CMakeLists.txt:269
+AND (TARGET cxxabi_static OR HAVE_LIBCXXABI))
+#if ((TARGET ${LIBCXX_CXX_ABI_LIBRARY}) OR
+#(${LIBCXX_CXX_ABI_LIBRARY} MATCHES "cxxabi(_static|_shared)?" AND
HAVE_LIBCXXABI))
---
ioeric added a comment.
In https://reviews.llvm.org/D49476#1168604, @akyrtzi wrote:
> `CXIndexDataConsumer.cpp` uses `ASTNode.OrigD`, that code is exercised when
> you run `c-index-test -index-file <...>`. I'd recommend to at least check if
> that command would produce a different output for th
ioeric updated this revision to Diff 156356.
ioeric added a comment.
- try to add a test case.
Repository:
rC Clang
https://reviews.llvm.org/D49476
Files:
lib/Index/IndexingContext.cpp
test/Index/index-template-specialization.cpp
tools/c-index-test/c-index-test.c
tools/libclang/Index
This revision was automatically updated to reflect the committed changes.
Closed by commit rL337497: [Sema] Diagnose an invalid dependent function
template specialization (authored by epilk, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.o
Author: epilk
Date: Thu Jul 19 13:40:20 2018
New Revision: 337497
URL: http://llvm.org/viewvc/llvm-project?rev=337497&view=rev
Log:
[Sema] Diagnose an invalid dependent function template specialization
Previously, clang marked the specialization as invalid without emitting a
diagnostic. This lead
rsmith accepted this revision.
rsmith added inline comments.
This revision is now accepted and ready to land.
Comment at: lib/Driver/ToolChains/Gnu.cpp:2205-2209
+
+ // Deal with OpenEmbedded linux sysroots (like for arm-oe-linux-gnueabi)
+ // which are of the form /usr
erik.pilkington added a reviewer: aaron.ballman.
erik.pilkington added a comment.
Thanks for working on this! CCing Aaron, who is code owner for attributes.
Comment at: include/clang/Basic/AttrDocs.td:3355
+ let Content = [{
+The ``noderef`` attribute allows for showing a warn
rsmith added a comment.
The way in which you're checking for the problematic cases is unnecessarily
expensive. Instead of performing a separate AST traversal, please detect
whether you should be producing the warning directly when forming the
problematic expressions. (For example, you could sto
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
LGTM.
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:4111
+ "candidate template ignored: %select{template is not a function template"
+ "|is not a member
vsk added inline comments.
Comment at: docs/UndefinedBehaviorSanitizer.rst:97
+ is not equal to the original value before the downcast. This kind of
issues
+ may often be caused by an implicit integer promotions.
- ``-fsanitize=integer-divide-by-zero``: Integer divis
erik.pilkington added inline comments.
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:4111
+ "candidate template ignored: %select{template is not a function template"
+ "|is not a member of the enclosing namespace}0">;
rjmccall wrote:
> Your fir
pcc added a subscriber: cfe-commits.
pcc added inline comments.
Comment at: lib/CodeGen/ItaniumCXXABI.cpp:3215
// Emit the standard library with external linkage.
llvm::GlobalVariable::LinkageTypes Linkage;
if (IsStdLib)
`IsStdLib` will always be false a
apazos updated this revision to Diff 156343.
apazos added a comment.
- Rebased the patch.
- Udpated test case to check for IR attribute.
https://reviews.llvm.org/D48412
Files:
include/clang/Basic/Attr.td
include/clang/Basic/AttrDocs.td
include/clang/Basic/DiagnosticSemaKinds.td
lib/Code
erik.pilkington updated this revision to Diff 156338.
erik.pilkington added a comment.
Improve the diagnostics further.
https://reviews.llvm.org/D49085
Files:
clang/include/clang/Basic/DiagnosticSemaKinds.td
clang/lib/Sema/SemaTemplate.cpp
clang/test/CXX/temp/temp.decls/temp.friend/p1.cpp
rjmccall added inline comments.
Comment at: lib/Sema/SemaOverload.cpp:3150
+ !getLangOpts().OpenCLCPlusPlus)
+return false;
+
yaxunl wrote:
> rjmccall wrote:
> > yaxunl wrote:
> > > rjmccall wrote:
> > > > It's not really OpenCL C++ that's special here,
lebedev.ri added a comment.
In https://reviews.llvm.org/D49508#1168620, @rjmccall wrote:
> In https://reviews.llvm.org/D49508#1168599, @lebedev.ri wrote:
>
> > In https://reviews.llvm.org/D49508#1168584, @rjmccall wrote:
> >
> > > Hmm. I think the approach of flagging ICEs that are semantically
rjmccall added inline comments.
Comment at: docs/Block-ABI-Apple.rst:64
enum {
+BLOCK_IS_NOESCAPE = (1 << 23),
BLOCK_HAS_COPY_DISPOSE = (1 << 25),
Something happened to my older comments here, but please document the meaning
of this
rjmccall added a comment.
In https://reviews.llvm.org/D49508#1168599, @lebedev.ri wrote:
> In https://reviews.llvm.org/D49508#1168584, @rjmccall wrote:
>
> > Hmm. I think the approach of flagging ICEs that are semantically part of
> > an explicit cast is probably a better representation for too
Author: rsmith
Date: Thu Jul 19 12:05:13 2018
New Revision: 337483
URL: http://llvm.org/viewvc/llvm-project?rev=337483&view=rev
Log:
Fix failing testcase to actually be valid.
Modified:
cfe/trunk/test/SemaTemplate/pack-deduction.cpp
Modified: cfe/trunk/test/SemaTemplate/pack-deduction.cpp
UR
Author: rsmith
Date: Thu Jul 19 12:00:37 2018
New Revision: 337481
URL: http://llvm.org/viewvc/llvm-project?rev=337481&view=rev
Log:
Fix template argument deduction when a parameter pack has a value
provided by an outer template.
We made the incorrect assumption in various places that the only wa
Author: nico
Date: Thu Jul 19 11:59:38 2018
New Revision: 337480
URL: http://llvm.org/viewvc/llvm-project?rev=337480&view=rev
Log:
fix typo in comment
Modified:
cfe/trunk/lib/CodeGen/MicrosoftCXXABI.cpp
Modified: cfe/trunk/lib/CodeGen/MicrosoftCXXABI.cpp
URL:
http://llvm.org/viewvc/llvm-pro
On 07/19/2018 09:01 AM, Jonas Hahnfeld wrote:
> On 2018-07-19 15:43, Hal Finkel wrote:
>> On 07/16/2018 01:19 PM, Jonas Hahnfeld wrote:
>>> [ Moving discussion from https://reviews.llvm.org/D49386 to the
>>> relevant comment on cfe-commits, CC'ing Hal who commented on the
>>> original issue ]
>>>
akyrtzi added a comment.
`CXIndexDataConsumer.cpp` uses `ASTNode.OrigD`, that code is exercised when you
run `c-index-test -index-file <...>`. I'd recommend to at least check if that
command would produce a different output for the test case that you detected
originally.
Repository:
rC Clan
ahatanak updated this revision to Diff 156332.
ahatanak added a comment.
Fix a bug where the capture cleanups weren't pushed when
BlockDecl::doesNotEscape() returns true.
Test that, when ARC is enabled, captures are retained and copied into the stack
block object and destroyed when the stack bl
lebedev.ri added a comment.
In https://reviews.llvm.org/D49508#1168584, @rjmccall wrote:
> Hmm. I think the approach of flagging ICEs that are semantically part of an
> explicit cast is probably a better representation for tools across the board.
I could do that, but i couldn't find where it
yaxunl marked an inline comment as done.
yaxunl added inline comments.
Comment at: lib/Sema/SemaOverload.cpp:3150
+ !getLangOpts().OpenCLCPlusPlus)
+return false;
+
rjmccall wrote:
> yaxunl wrote:
> > rjmccall wrote:
> > > It's not really OpenCL C++ that
rjmccall added a comment.
Hmm. I think the approach of flagging ICEs that are semantically part of an
explicit cast is probably a better representation for tools across the board.
If we *are* going to do it this way, though, I think you should (1) make the
collection of skipped expressions opt
rjmccall added inline comments.
Comment at: lib/CodeGen/CGCUDANV.cpp:444
+auto HandleValue =
+CtorBuilder.CreateAlignedLoad(GpuBinaryHandle, CGM.getPointerAlign());
+llvm::Constant *Zero =
llvm::Constant::getNullValue(HandleValue->getType());
yax
kimgr added a comment.
Thank you for doing this, I'm going to guess you have IWYU in mind :-)
So as consumers of this, how do you envision we use the new variable? Something
like https://stackoverflow.com/a/13429998 to copy the resource dir into our
build root, and then an install rule to put i
rjmccall added inline comments.
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:4111
+ "candidate template ignored: %select{template is not a function template"
+ "|is not a member of the enclosing namespace}0">;
Your first explanation has a subj
kimgr added a comment.
Herald added a subscriber: llvm-commits.
Ping!
Repository:
rL LLVM
https://reviews.llvm.org/D32696
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kimgr added a comment.
Herald added a reviewer: javed.absar.
This can be closed, IWYU no longer officially supports in-tree builds.
https://reviews.llvm.org/D31696
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin
Gentle reminder to the reviewers to please review my patch:
https://reviews.llvm.org/D48862 [Fix lib paths for OpenEmbedded targets].
I wasn't sure who to add as reviewer for OpenEmbedded targets. So
apologies if you are not the correct person to review this.
Please feel free to add the correct
This revision was automatically updated to reflect the committed changes.
Closed by commit rL337477: [libc++] Allow running ABI list tests with different
ABI versions (authored by ldionne, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D49509?vs=156148&id=156323#toc
Reposito
EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.
LGTM. Though it might be worth noting that we don't really have an "ABI v2"
yet; we're just staging it ATM.
Repository:
rL LLVM
https://reviews.llvm.org/D49509
_
Author: rkovacs
Date: Thu Jul 19 10:43:09 2018
New Revision: 337474
URL: http://llvm.org/viewvc/llvm-project?rev=337474&view=rev
Log:
[analyzer] Fix memory sanitizer error in MallocChecker.
StringRef's data() returns a string that may be non-null-terminated.
Switch to using StringRefs from const
erik.pilkington updated this revision to Diff 156317.
erik.pilkington added a comment.
Improve the diagnostics. Thanks!
https://reviews.llvm.org/D49085
Files:
clang/include/clang/Basic/DiagnosticSemaKinds.td
clang/lib/Sema/SemaTemplate.cpp
clang/test/CXX/temp/temp.decls/temp.friend/p1.cpp
Author: erichkeane
Date: Thu Jul 19 10:19:16 2018
New Revision: 337473
URL: http://llvm.org/viewvc/llvm-project?rev=337473&view=rev
Log:
Fix unused variable warning.
Modified:
cfe/trunk/lib/CodeGen/CGOpenMPRuntime.cpp
Modified: cfe/trunk/lib/CodeGen/CGOpenMPRuntime.cpp
URL:
http://llvm.org/
This revision was automatically updated to reflect the committed changes.
Closed by commit rC337472: [analyzer] Memoize complexity of SymExpr (authored
by mramalho, committed by ).
Herald added a subscriber: cfe-commits.
Repository:
rC Clang
https://reviews.llvm.org/D49232
Files:
include/cl
Author: mramalho
Date: Thu Jul 19 10:03:12 2018
New Revision: 337472
URL: http://llvm.org/viewvc/llvm-project?rev=337472&view=rev
Log:
[analyzer] Memoize complexity of SymExpr
Summary:
This patch introduces a new member to SymExpr, which stores the symbol
complexity, avoiding recalculating it ev
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
I was thinking of two flags, but that'll work too.
https://reviews.llvm.org/D49536
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists
This revision was automatically updated to reflect the committed changes.
Closed by commit rC337470: [Sema] Add a new warning, -Wmemset-transposed-args
(authored by epilk, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D49112?vs=156178&id=156298#toc
Repository:
rC Clang
h
1 - 100 of 171 matches
Mail list logo