Re: [PATCH] D10834: Added functions to retrieve information about variable storage in libclang and its python bindings.

RedX2501 added a comment. Ping http://reviews.llvm.org/D10834 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D10833: Retrieve BinaryOperator::getOpcode and BinaryOperator::getOpcodeStr via libclang and its python interface

RedX2501 added a comment. Ping http://reviews.llvm.org/D10833 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D12244: Implement ACLE 2.0 macros of chapters 6.4 and 6.5 for [ARM] and [Aarch64] targets

labrinea updated this revision to Diff 32934. labrinea added a comment. Reordered macro definitions to make review easier. http://reviews.llvm.org/D12244 Files: lib/Basic/Targets.cpp test/Preprocessor/aarch64-target-features.c test/Preprocessor/arm-acle-6.4.c test/Preprocessor/arm-acle-

Re: [PATCH] D12244: Implement ACLE 2.0 macros of chapters 6.4 and 6.5 for [ARM] and [Aarch64] targets

labrinea added inline comments. Comment at: lib/Basic/Targets.cpp:4757 @@ -4679,2 +4756,3 @@ Builder.defineMacro("__ARM_NEON__"); + Builder.defineMacro("__ARM_NEON_FP", "0x" + llvm::utohexstr(HW_FP & ~HW_FP_DP)); } According to the ACLE document,

RE: [PATCH] RE: [cfe-dev] missing return statement for non-void functions in C++

I've taken the latest suggestions into account, see attached patch. Could you have a look? Thanks. -Original Message- From: scott douglass [mailto:sdougl...@arm.com] Sent: 18 August 2015 14:16 To: Sjoerd Meijer Cc: 'Marshall Clow'; 'Gabriel Dos Reis'; 'Richard Smith'; 'cfe-commits' Subje

Re: [PATCH] D12240: [AArch64] Define the macro __ARM_FP16_ARGS

This revision was automatically updated to reflect the committed changes. Closed by commit rL245833: [AArch64] Define the macro __ARM_FP16_ARGS (authored by kwalker). Changed prior to commit: http://reviews.llvm.org/D12240?vs=32832&id=32938#toc Repository: rL LLVM http://reviews.llvm.org/D1

r245833 - [AArch64] Define the macro __ARM_FP16_ARGS

Author: kwalker Date: Mon Aug 24 05:11:14 2015 New Revision: 245833 URL: http://llvm.org/viewvc/llvm-project?rev=245833&view=rev Log: [AArch64] Define the macro __ARM_FP16_ARGS The ACLE (ARM C Language Extensions) 2.0 defines that the predefined macro __ARM_FP16_ARGS should be defined if __fp16 c

Re: [PATCH] D12148: [ARM] Allow passing/returning of __fp16 arguments

rengolin added inline comments. Comment at: lib/Driver/Tools.cpp:4254 @@ -4253,2 +4253,3 @@ if (getToolChain().getArch() == llvm::Triple::aarch64 || - getToolChain().getArch() == llvm::Triple::aarch64_be) + getToolChain().getArch() == llvm::Triple::aarch64_be || +

[PATCH] D12278: [X86] Add MSVC-compatible intrinsics for clac, stac, lgdt and sgdt

mkuper created this revision. mkuper added reviewers: rnk, majnemer. mkuper added a subscriber: cfe-commits. LLVM part was posted as D12277 http://reviews.llvm.org/D12278 Files: include/clang/Basic/BuiltinsX86.def lib/Headers/Intrin.h test/CodeGen/builtins-x86.c Index: lib/Headers/Intrin.

Re: [PATCH] D12244: Implement ACLE 2.0 macros of chapters 6.4 and 6.5 for [ARM] and [Aarch64] targets

olista01 added a subscriber: olista01. Comment at: lib/Basic/Targets.cpp:4685-4686 @@ +4684,4 @@ +// ACLE 6.5.2 Half-precision (16-bit) floating-point format +if (HW_FP & HW_FP_HP) + Builder.defineMacro("__ARM_FP16_FORMAT_IEEE", "1"); + The __fp16 typ

r245837 - clang-format: Properly handle braced lists in macros.

Author: djasper Date: Mon Aug 24 08:23:37 2015 New Revision: 245837 URL: http://llvm.org/viewvc/llvm-project?rev=245837&view=rev Log: clang-format: Properly handle braced lists in macros. Before: #define A\ { a, a } \ , After: #define A {a, a}, Modified: cfe/trunk/lib/Fo

Re: [PATCH] D12244: Implement ACLE 2.0 macros of chapters 6.4 and 6.5 for [ARM] and [Aarch64] targets

labrinea added inline comments. Comment at: lib/Basic/Targets.cpp:4685-4686 @@ +4684,4 @@ +// ACLE 6.5.2 Half-precision (16-bit) floating-point format +if (HW_FP & HW_FP_HP) + Builder.defineMacro("__ARM_FP16_FORMAT_IEEE", "1"); + olista01 wrote: > The

[PATCH] D12281: Tests no longer need the 'REQUIRES: SHELL' line.

angelgarcia created this revision. angelgarcia added a reviewer: alexfh. angelgarcia added subscribers: cfe-commits, klimek. Update python script, so that it doesn't print that line in new tests. http://reviews.llvm.org/D12281 Files: clang-tidy/add_new_check.py Index: clang-tidy/add_new_check

Re: [PATCH] D12281: Tests no longer need the 'REQUIRES: SHELL' line.

LGTM! ~Aaron On Mon, Aug 24, 2015 at 9:45 AM, Angel Garcia via cfe-commits wrote: > angelgarcia created this revision. > angelgarcia added a reviewer: alexfh. > angelgarcia added subscribers: cfe-commits, klimek. > > Update python script, so that it doesn't print that line in new tests. > > http

Re: [PATCH] D12244: Implement ACLE 2.0 macros of chapters 6.4 and 6.5 for [ARM] and [Aarch64] targets

labrinea added inline comments. Comment at: lib/Basic/Targets.cpp:4685-4686 @@ +4684,4 @@ +// ACLE 6.5.2 Half-precision (16-bit) floating-point format +if (HW_FP & HW_FP_HP) + Builder.defineMacro("__ARM_FP16_FORMAT_IEEE", "1"); + olista01 wrote: > lab

Re: [PATCH] D11832: [Patch] [Analyzer] false positive: Potential leak connected with memcpy (PR 22954)

pgousseau updated this revision to Diff 32955. pgousseau added a comment. Encapsulate 'RegionAndSymbolInvalidationTraits' by overriding 'AddToWorkList' following Anton's review. Also as pointed out by Devin's review: Add tests cases and handling for regions, bindings in super regions, and aliase

r245843 - clang-format: Make formatting of member function reference qualifiers

Author: djasper Date: Mon Aug 24 09:28:08 2015 New Revision: 245843 URL: http://llvm.org/viewvc/llvm-project?rev=245843&view=rev Log: clang-format: Make formatting of member function reference qualifiers more consistent. Before: SomeType MemberFunction(const Deleted &)&&; SomeType MemberFunct

r245846 - clang-format: Always allow break after leading annotations.

Author: djasper Date: Mon Aug 24 10:10:01 2015 New Revision: 245846 URL: http://llvm.org/viewvc/llvm-project?rev=245846&view=rev Log: clang-format: Always allow break after leading annotations. Before: DEPRECATED("Use NewClass::NewFunction instead.") int OldFunction( const string ¶meter) {}

Re: r245843 - clang-format: Make formatting of member function reference qualifiers

On Mon, Aug 24, 2015 at 7:28 AM, Daniel Jasper via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: djasper > Date: Mon Aug 24 09:28:08 2015 > New Revision: 245843 > > URL: http://llvm.org/viewvc/llvm-project?rev=245843&view=rev > Log: > clang-format: Make formatting of member function r

Re: [PATCH] D12164: Stop treating -static as overriding -fPIC: they are distinct.

jyknight closed this revision. jyknight added a comment. D'oh, I copy/pasted the wrong review link when submitting this. It was submitted as http://reviews.llvm.org/rL245667. http://reviews.llvm.org/D12164 ___ cfe-commits mailing list cfe-commits@l

Re: r245843 - clang-format: Make formatting of member function reference qualifiers

On Mon, Aug 24, 2015 at 5:46 PM, David Blaikie wrote: > > > On Mon, Aug 24, 2015 at 7:28 AM, Daniel Jasper via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > >> Author: djasper >> Date: Mon Aug 24 09:28:08 2015 >> New Revision: 245843 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=24584

[libcxx] r245849 - Fix a crasher found by libFuzzer

Author: marshall Date: Mon Aug 24 10:57:09 2015 New Revision: 245849 URL: http://llvm.org/viewvc/llvm-project?rev=245849&view=rev Log: Fix a crasher found by libFuzzer Modified: libcxx/trunk/include/regex libcxx/trunk/test/std/re/re.alg/re.alg.search/grep.pass.cpp Modified: libcxx/trunk/

[PATCH] D12287: Add replace-auto_ptr check.

angelgarcia created this revision. angelgarcia added a reviewer: alexfh. angelgarcia added subscribers: cfe-commits, klimek. Migrate replace-auto_ptr check from clang-modernize to modernize module in clang-tidy. http://reviews.llvm.org/D12287 Files: clang-tidy/modernize/CMakeLists.txt clang

Re: [PATCH] D12148: [ARM] Allow passing/returning of __fp16 arguments

olista01 removed rL LLVM as the repository for this revision. olista01 updated this revision to Diff 32967. olista01 added a comment. Use a switch statement to check the target architecture. http://reviews.llvm.org/D12148 Files: lib/Basic/Targets.cpp lib/CodeGen/TargetInfo.cpp lib/Driver/

Re: [PATCH] D12271: [X86] Expose the various _rot intrinsics on non-MS platforms

rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land. This looks good. As a larger issue, LLVM fast isel definitely won't pattern match this series of shifts and selects to rotl at -O0. There are some users who want branchless constant time rotates re

Re: [PATCH] D6551: Improvements to scan-build.

ayartsev updated this revision to Diff 32969. ayartsev added a comment. Updated the patch with r245621. OK to commit? http://reviews.llvm.org/D6551 Files: tools/scan-build/scan-build Index: tools/scan-build/scan-build === --- to

LLVM buildmaster will be tonight

Hello everyone, LLVM buildmaster will be restarted after 5 PM Pacific time today. Thanks Galina ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D11182: [OPENMP 4.0] Initial support for 'omp declare reduction' construct.

fraggamuffin added a comment. Got it. Test results look good with this patch. Expected Passes: 7011 Expected Failures : 21 Unsupported Tests : 89 Unexpected Failures: 47 http://reviews.llvm.org/D11182 ___ cfe-commits mailing list cfe-

Re: [PATCH] D10732: [OPENMP 4.0] Initial support for array sections.

ABataev marked 15 inline comments as done. ABataev added a comment. Richard, thanks for the review! Comment at: include/clang/Basic/DiagnosticSemaKinds.td:7680 @@ +7679,3 @@ +def err_omp_section_length_undefined : Error< + "section length is unspecified, but subscripted value i

Re: [PATCH] D10732: [OPENMP 4.0] Initial support for array sections.

ABataev updated this revision to Diff 32974. ABataev marked 10 inline comments as done. ABataev added a comment. Update after review http://reviews.llvm.org/D10732 Files: include/clang-c/Index.h include/clang/AST/ASTContext.h include/clang/AST/BuiltinTypes.def include/clang/AST/DataRecu

Re: [PATCH] D11468: [Static Analyzer] The first implementation of nullability checker.

xazax.hun marked 11 inline comments as done. xazax.hun added a comment. Thank you for the review. I will upload the new version of the patch once the rest of it was reviewed. Comment at: lib/StaticAnalyzer/Checkers/NullabilityChecker.cpp:12 @@ +11,3 @@ +// checker is that, the

Re: [PATCH] D12169: Relax constexpr rules to improve __builtin_object_size's accuracy

george.burgess.iv updated this revision to Diff 32985. george.burgess.iv added a comment. Mostly redid the patch; removed OutermostExprEvaluator, moved evaluation intent into `EvaluationMode`, made it so the more interesting rulebreaking that `__builtin_object_size` allows is only allowed when e

Re: [PATCH] D12022: Refactored dtor sanitizing into EHScopeStack

nmusgrave updated this revision to Diff 32986. nmusgrave added a comment. - Refactored for simpler trivial field checking http://reviews.llvm.org/D12022 Files: lib/CodeGen/CGCXX.cpp lib/CodeGen/CGClass.cpp lib/CodeGen/CodeGenModule.h test/CodeGenCXX/sanitize-dtor-nontrivial-virtual-base

r245861 - Fixing some copy-pasta comments; NFC.

Author: aaronballman Date: Mon Aug 24 13:50:01 2015 New Revision: 245861 URL: http://llvm.org/viewvc/llvm-project?rev=245861&view=rev Log: Fixing some copy-pasta comments; NFC. Modified: cfe/trunk/include/clang/AST/Decl.h Modified: cfe/trunk/include/clang/AST/Decl.h URL: http://llvm.org/vie

[libcxx] r245864 - Add release goals to TODO.txt

Author: ericwf Date: Mon Aug 24 14:33:40 2015 New Revision: 245864 URL: http://llvm.org/viewvc/llvm-project?rev=245864&view=rev Log: Add release goals to TODO.txt Modified: libcxx/trunk/TODO.TXT Modified: libcxx/trunk/TODO.TXT URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/TODO.TXT?r

r245866 - [ARM] Error out on apple darwin platforms if float-abi is "hard".

Author: ahatanak Date: Mon Aug 24 14:50:35 2015 New Revision: 245866 URL: http://llvm.org/viewvc/llvm-project?rev=245866&view=rev Log: [ARM] Error out on apple darwin platforms if float-abi is "hard". Error out if the user tries to use float-abi="hard" since it isn't supported on darwin platforms

r245867 - [libclang] For convenience to clients, make sure that nullability and __kindof annotations do not hide

Author: akirtzidis Date: Mon Aug 24 14:50:45 2015 New Revision: 245867 URL: http://llvm.org/viewvc/llvm-project?rev=245867&view=rev Log: [libclang] For convenience to clients, make sure that nullability and __kindof annotations do not hide the underlying type. rdar://22063577 Modified: cfe/

Re: [PATCH] D11690: [CUDA] Added stubs for new attributes used by CUDA headers.

tra abandoned this revision. tra added a comment. There is no way to close revision. Abandoning it to get it off my list of pending changes. http://reviews.llvm.org/D11690 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.or

Re: [PATCH] D12155: ARM: Error out on apple darwin platforms if float-abi is "hard"

This revision was automatically updated to reflect the committed changes. Closed by commit rL245866: [ARM] Error out on apple darwin platforms if float-abi is "hard". (authored by ahatanak). Changed prior to commit: http://reviews.llvm.org/D12155?vs=32561&id=32989#toc Repository: rL LLVM ht

Re: patch: clarify diagnostic when returned value doesn't match function return type

On 19 August 2015 at 15:18, Nick Lewycky wrote: > On 10 August 2015 at 19:08, Nick Lewycky wrote: > >> This simple-minded patch extends the case where we report "no viable >> conversion from 'X' to 'Y'" to emit a more useful diagnostic "no viable >> conversion from returned value of type 'X' to

Re: [PATCH] D11664: [CUDA] Implemented additional processing steps needed to link with CUDA libdevice bitcode.

echristo added inline comments. Comment at: lib/CodeGen/CodeGenAction.cpp:166-170 @@ +165,7 @@ +std::vector ModuleFuncNames; +// We need to internalize contents of the linked module but it +// has to be done *after* the linking because internalized +

Re: [PATCH] Have clang list the imported modules in the debug info

> On Aug 19, 2015, at 1:20 PM, David Blaikie wrote: > > > > On Mon, Aug 10, 2015 at 5:00 PM, Adrian Prantl > wrote: > >> On Jul 24, 2015, at 12:33 PM, David Blaikie > > wrote: >> >> *reads back through the thread* > > appreciated, it’s l

Re: [PATCH] D11280: [CUDA] Improve CUDA compilation pipeline creation.

echristo accepted this revision. echristo added a comment. This revision is now accepted and ready to land. This seems like a decent incremental improvement. I think we still need to separate out the pipeline a bit further and have the cuda compilation just be separate actions that don't need th

r245871 - Revert r245866.

Author: ahatanak Date: Mon Aug 24 15:30:57 2015 New Revision: 245871 URL: http://llvm.org/viewvc/llvm-project?rev=245871&view=rev Log: Revert r245866. This commit was causing buildbot failures. Removed: cfe/trunk/test/Driver/arm-float-abi.c Modified: cfe/trunk/lib/Driver/Tools.cpp Modif

Re: [PATCH] D11963: Create a __config_site file to capture configuration decisions.

EricWF commandeered this revision. EricWF edited reviewers, added: jroelofs; removed: EricWF. EricWF added a comment. Stealing this revision. I have a different plan. http://reviews.llvm.org/D11963 ___ cfe-commits mailing list cfe-commits@lists.llvm

Re: [PATCH] D11963: Create a __config_site file to capture configuration decisions.

EricWF updated this revision to Diff 32992. EricWF added a comment. This patch takes @jroelof's idea and runs with it. However this patch doesn't require a `__config_site` header. The patch works modifying `build-libcxx/include/c++/v1/__config` by prepending the contents of `__config_site`. If

Re: [PATCH] D12247: [libc++] remove possible trailing padding from aligned_storage

yiranwang added a comment. A test case is as following. It has to be build by GCC 4.9 -O3 (maybe or later), with latest libc++, and for AARCH64+ANDROID target. AARCH64 requires 128 bit alignment for aligned_storage and 64 bit pointers, while gcc 4.9 alias analysis will do field-sensitive points

Re: [PATCH] Have clang list the imported modules in the debug info

On Mon, Aug 24, 2015 at 1:23 PM, Adrian Prantl wrote: > > On Aug 19, 2015, at 1:20 PM, David Blaikie wrote: > > > > On Mon, Aug 10, 2015 at 5:00 PM, Adrian Prantl wrote: > >> >> On Jul 24, 2015, at 12:33 PM, David Blaikie wrote: >> >> *reads back through the thread* >> >> >> appreciated, it’s

[PATCH] D12294: Misc cleanups re-commandeered from D11963.

jroelofs created this revision. jroelofs added a reviewer: EricWF. jroelofs added a subscriber: cfe-commits. This separates out the bits of D11963 which were drive-by fixes, which aren't really relevant to the __config_site part of that change. http://reviews.llvm.org/D12294 Files: CMakeLists

Re: [PATCH] D11664: [CUDA] Implemented additional processing steps needed to link with CUDA libdevice bitcode.

tra added inline comments. Comment at: lib/CodeGen/CodeGenAction.cpp:166-170 @@ +165,7 @@ +std::vector ModuleFuncNames; +// We need to internalize contents of the linked module but it +// has to be done *after* the linking because internalized +// s

Re: [PATCH] D12294: Misc cleanups re-commandeered from D11963.

jroelofs updated this revision to Diff 32996. jroelofs added a comment. The diff uploaded strangely... trying again. http://reviews.llvm.org/D12294 Files: CMakeLists.txt test/libcxx/atomics/libcpp-has-no-threads.fail.cpp test/libcxx/atomics/libcpp-has-no-threads.pass.cpp test/std/atomic

Re: [PATCH] D12294: Misc cleanups re-commandeered from D11963.

EricWF added a comment. Can you move the `libcpp-has-no-threads` tests to `test/libcxx/atomics` instead of deleting them? (I imagine that's what you intended to do). Also `tested_elsewhere.pass.cpp` shouldn't really be called that because it contains the actual test. Could you rename it to some

Re: [PATCH] D12294: Misc cleanups re-commandeered from D11963.

EricWF accepted this revision. EricWF added a comment. This revision is now accepted and ready to land. LGTM after renaming `tested_elsewhere.pass.cpp` http://reviews.llvm.org/D12294 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lis

Re: [PATCH] D12294: Misc cleanups re-commandeered from D11963.

EricWF added a comment. Actually commit this as is and I'll handle the other cleanup. http://reviews.llvm.org/D12294 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D12294: Misc cleanups re-commandeered from D11963.

jroelofs closed this revision. jroelofs added a comment. r245876 http://reviews.llvm.org/D12294 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libcxx] r245876 - Misc drive-by cleanups. NFC

Author: jroelofs Date: Mon Aug 24 16:20:07 2015 New Revision: 245876 URL: http://llvm.org/viewvc/llvm-project?rev=245876&view=rev Log: Misc drive-by cleanups. NFC http://reviews.llvm.org/D12294 Added: libcxx/trunk/test/libcxx/atomics/ libcxx/trunk/test/libcxx/atomics/libcpp-has-no-thread

[libcxx] r245877 - Move test/std/utilities/date.time to proper stable name utilities/time/date.time

Author: ericwf Date: Mon Aug 24 16:27:25 2015 New Revision: 245877 URL: http://llvm.org/viewvc/llvm-project?rev=245877&view=rev Log: Move test/std/utilities/date.time to proper stable name utilities/time/date.time Added: libcxx/trunk/test/libcxx/utilities/time/ libcxx/trunk/test/libcxx/ut

Re: [PATCH] D11963: Create a __config_site file to capture configuration decisions.

EricWF updated the summary for this revision. EricWF updated this revision to Diff 32997. EricWF added a comment. Update diff to remove misc cleanup stuff. http://reviews.llvm.org/D11963 Files: CMakeLists.txt cmake/Modules/HandleLibcxxFlags.cmake include/CMakeLists.txt include/__config_

r245879 - [MS ABI] Don't emit stackrestore in cleanups

Author: majnemer Date: Mon Aug 24 16:34:21 2015 New Revision: 245879 URL: http://llvm.org/viewvc/llvm-project?rev=245879&view=rev Log: [MS ABI] Don't emit stackrestore in cleanups The stackrestore intrinsic isn't meaningful inside of a cleanup funclet. Modified: cfe/trunk/lib/CodeGen/CGCall.

Re: [PATCH] D11664: [CUDA] Implemented additional processing steps needed to link with CUDA libdevice bitcode.

echristo added inline comments. Comment at: lib/CodeGen/CodeGenAction.cpp:181-190 @@ -166,2 +180,12 @@ return; +if (LangOpts.CUDA && LangOpts.CUDAIsDevice && +LangOpts.CUDAUsesLibDevice) { + legacy::PassManager passes; + passes.add(

[PATCH] D12297: [libcxx] Fix most undefined behavior in

EricWF created this revision. EricWF added reviewers: mclow.lists, danalbert, rsmith. EricWF added subscribers: cfe-commits, awi, rsmith. This patch partially fixes the undefined behavior in ``. The undefined behavior is caused by casting a pointer to `__node_base __end_` to the derived type `__

r245881 - [modules] Remove unnecessary deserialization of fully-external HeaderFileInfos for all files we've seen in this compilation.

Author: rsmith Date: Mon Aug 24 16:59:32 2015 New Revision: 245881 URL: http://llvm.org/viewvc/llvm-project?rev=245881&view=rev Log: [modules] Remove unnecessary deserialization of fully-external HeaderFileInfos for all files we've seen in this compilation. Modified: cfe/trunk/include/clang/

[PATCH] D12299: [libcxx] ABI-Breaking Fix for ALL undefined behavior in .

EricWF created this revision. EricWF added reviewers: mclow.lists, danalbert, jroelofs. EricWF added a subscriber: cfe-commits. This patch is an ABI-breaking version of D12297. However unlike D12297, this patch does not have any undefined behavior when `element_type` has a stricter alignment req

Re: [PATCH] D10732: [OPENMP 4.0] Initial support for array sections.

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Thanks, LGTM! Comment at: lib/Sema/SemaExpr.cpp:4035 @@ +4034,3 @@ + // Build an unanalyzed expression if either operand is type-dependent. + if (Base->isTypeDependent() ||

[PATCH] D12301: [PATCH] New checker for UB in handler of a function-try-block

aaron.ballman created this revision. aaron.ballman added reviewers: alexfh, rsmith. aaron.ballman added a subscriber: cfe-commits. Per [except.handle]p10, the handler for a constructor or destructor function-try-block cannot refer to a non-static member of the object under construction. This pat

Re: [PATCH] Have clang list the imported modules in the debug info

+static void escape(llvm::raw_ostream &OS, StringRef String) { + for (auto C: String) { +if (strchr("\\\"", C)) + OS << '\\'; +OS << C; + } +} Do we not have an existing function somewhere to do that? Also, that strchr call is a very complex way of writing (C == '\\' || C == '"') O

Re: [PATCH] D12301: [PATCH] New checker for UB in handler of a function-try-block

On Mon, Aug 24, 2015 at 3:23 PM, Aaron Ballman wrote: > aaron.ballman created this revision. > aaron.ballman added reviewers: alexfh, rsmith. > aaron.ballman added a subscriber: cfe-commits. > > Per [except.handle]p10, the handler for a constructor or destructor > function-try-block cannot refer

Re: [PATCH] D12281: Tests no longer need the 'REQUIRES: SHELL' line.

alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG. Thank you! http://reviews.llvm.org/D12281 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listi

Re: [PATCH] Have clang list the imported modules in the debug info

> On Aug 24, 2015, at 2:01 PM, David Blaikie wrote: > > > > On Mon, Aug 24, 2015 at 1:23 PM, Adrian Prantl > wrote: > >> On Aug 19, 2015, at 1:20 PM, David Blaikie > > wrote: >> >> >> >> On Mon, Aug 10, 2015 at 5:00 PM, Adrian Prantl >

Re: [PATCH] D12301: [PATCH] New checker for UB in handler of a function-try-block

On Mon, Aug 24, 2015 at 6:29 PM, Richard Smith wrote: > On Mon, Aug 24, 2015 at 3:23 PM, Aaron Ballman > wrote: >> >> aaron.ballman created this revision. >> aaron.ballman added reviewers: alexfh, rsmith. >> aaron.ballman added a subscriber: cfe-commits. >> >> Per [except.handle]p10, the handler

[libclc] r245894 - Update mailing list reference.

Author: pcc Date: Mon Aug 24 17:43:24 2015 New Revision: 245894 URL: http://llvm.org/viewvc/llvm-project?rev=245894&view=rev Log: Update mailing list reference. Modified: libclc/trunk/www/index.html Modified: libclc/trunk/www/index.html URL: http://llvm.org/viewvc/llvm-project/libclc/trunk/

Re: [PATCH] D12287: Add replace-auto_ptr check.

alexfh added inline comments. Comment at: clang-tidy/modernize/ReplaceAutoPtrCheck.h:20 @@ +19,3 @@ + +class ReplaceAutoPtrCheck : public ClangTidyCheck { +public: Please add a class comment describing what the check does and why. http://reviews.llvm.org/D12287

Re: [PATCH] D11857: CFI: Introduce -fsanitize=cfi-icall flag.

kcc added inline comments. Comment at: docs/ControlFlowIntegrity.rst:149 @@ +148,3 @@ +shared library boundaries are handled as if the callee was not compiled with +``-fsanitize=cfi-icall``. + mention that LTO is require explicitly? http://reviews.llvm.org/D1185

Re: [PATCH] Have clang list the imported modules in the debug info

On Mon, Aug 24, 2015 at 3:34 PM, Adrian Prantl wrote: > > On Aug 24, 2015, at 2:01 PM, David Blaikie wrote: > > > > On Mon, Aug 24, 2015 at 1:23 PM, Adrian Prantl wrote: > >> >> On Aug 19, 2015, at 1:20 PM, David Blaikie wrote: >> >> >> >> On Mon, Aug 10, 2015 at 5:00 PM, Adrian Prantl wrote:

Re: [PATCH] D12215: [UBSan] Add the ability to print more precise error kind in summary line.

This revision was automatically updated to reflect the committed changes. Closed by commit rL245897: [UBSan] Add the ability to print more precise error kind in summary line. (authored by samsonov). Changed prior to commit: http://reviews.llvm.org/D12215?vs=32753&id=33019#toc Repository: rL

Re: [PATCH] D11737: Add -linker (and -linker=) alias for -fuse-ld=

echristo added inline comments. Comment at: include/clang/Driver/Options.td:1853-1854 @@ -1853,1 +1852,4 @@ +def fuse_ld_EQ : Joined<["-"], "fuse-ld=">, HelpText<"Use linker ">, Group; +def linker : Separate<["-"], "linker">, Alias, MetaVarName<"">; +def linker_EQ : Joined<["-"]

Re: [PATCH] D11857: CFI: Introduce -fsanitize=cfi-icall flag.

pcc updated this revision to Diff 33020. pcc added a comment. - Address comment http://reviews.llvm.org/D11857 Files: docs/ControlFlowIntegrity.rst include/clang/AST/Mangle.h include/clang/Basic/Sanitizers.def lib/AST/ItaniumMangle.cpp lib/AST/MicrosoftMangle.cpp lib/CodeGen/CGClass

Re: [PATCH] D11857: CFI: Introduce -fsanitize=cfi-icall flag.

pcc added inline comments. Comment at: docs/ControlFlowIntegrity.rst:149 @@ +148,3 @@ +shared library boundaries are handled as if the callee was not compiled with +``-fsanitize=cfi-icall``. + kcc wrote: > mention that LTO is require explicitly? Done above. http

r245901 - [ARM NEON] Add missing AArch64 vget tests.

Author: ab Date: Mon Aug 24 18:34:25 2015 New Revision: 245901 URL: http://llvm.org/viewvc/llvm-project?rev=245901&view=rev Log: [ARM NEON] Add missing AArch64 vget tests. Added: cfe/trunk/test/CodeGen/aarch64-neon-vget.c - copied, changed from r245893, cfe/trunk/test/CodeGen/arm64_vget

Re: [PATCH] D12038: CodeGen: Introduce CodeGenModule::CreateMetadataIdentifierForType.

pcc added a comment. @rsmith: Ping. http://reviews.llvm.org/D12038 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r245904 - [ARM NEON] Replace redundant code with a new GetFloatNeonType. NFC.

Author: ab Date: Mon Aug 24 18:41:31 2015 New Revision: 245904 URL: http://llvm.org/viewvc/llvm-project?rev=245904&view=rev Log: [ARM NEON] Replace redundant code with a new GetFloatNeonType. NFC. Modified: cfe/trunk/lib/CodeGen/CGBuiltin.cpp Modified: cfe/trunk/lib/CodeGen/CGBuiltin.cpp URL

Re: [PATCH] D12301: [PATCH] New checker for UB in handler of a function-try-block

On Mon, Aug 24, 2015 at 3:36 PM, Aaron Ballman wrote: > On Mon, Aug 24, 2015 at 6:29 PM, Richard Smith > wrote: > > On Mon, Aug 24, 2015 at 3:23 PM, Aaron Ballman > > wrote: > >> > >> aaron.ballman created this revision. > >> aaron.ballman added reviewers: alexfh, rsmith. > >> aaron.ballman add

r245906 - [ARM NEON] Use CGF cached Types instead of llvm::Type::get. NFC.

Author: ab Date: Mon Aug 24 18:47:29 2015 New Revision: 245906 URL: http://llvm.org/viewvc/llvm-project?rev=245906&view=rev Log: [ARM NEON] Use CGF cached Types instead of llvm::Type::get. NFC. Modified: cfe/trunk/lib/CodeGen/CGBuiltin.cpp Modified: cfe/trunk/lib/CodeGen/CGBuiltin.cpp URL:

r245908 - Use GetLinkerPath utility function to find linker for NaCl toolchain

Author: dschuff Date: Mon Aug 24 18:53:25 2015 New Revision: 245908 URL: http://llvm.org/viewvc/llvm-project?rev=245908&view=rev Log: Use GetLinkerPath utility function to find linker for NaCl toolchain Summary: This is more consistent with other targets and also makes the -fuse-ld flag work. Re

r245909 - broken test. uses system ld.gold

Author: dschuff Date: Mon Aug 24 18:53:28 2015 New Revision: 245909 URL: http://llvm.org/viewvc/llvm-project?rev=245909&view=rev Log: broken test. uses system ld.gold Added: cfe/trunk/test/Driver/Inputs/basic_nacl_tree/ cfe/trunk/test/Driver/Inputs/basic_nacl_tree/bin/ cfe/trunk/test/

r245910 - Revert "broken test. uses system ld.gold"

Author: dschuff Date: Mon Aug 24 18:53:30 2015 New Revision: 245910 URL: http://llvm.org/viewvc/llvm-project?rev=245910&view=rev Log: Revert "broken test. uses system ld.gold" We should check in a basic_nacl_tree that works and also fix --sysroot Removed: cfe/trunk/test/Driver/Inputs/basic_na

Re: [PATCH] Have clang list the imported modules in the debug info

> On Aug 24, 2015, at 4:17 PM, David Blaikie wrote: > > > > On Mon, Aug 24, 2015 at 3:34 PM, Adrian Prantl > wrote: > >> On Aug 24, 2015, at 2:01 PM, David Blaikie > > wrote: >> >> >> >> On Mon, Aug 24, 2015 at 1:23 PM, Adrian Prantl >

Re: [PATCH] D12265: [ZORG] Add support for libc++ to SphinxDocBuilder.py

EricWF updated this revision to Diff 33029. EricWF marked 2 inline comments as done. http://reviews.llvm.org/D12265 Files: buildbot/osuosl/master/config/builders.py zorg/buildbot/builders/SphinxDocsBuilder.py Index: zorg/buildbot/builders/SphinxDocsBuilder.py

Re: [PATCH] D12265: [ZORG] Add support for libc++ to SphinxDocBuilder.py

EricWF accepted this revision. EricWF added a reviewer: EricWF. EricWF added a comment. This revision is now accepted and ready to land. Accepting so I can commit. Comment at: buildbot/osuosl/master/config/builders.py:1166 @@ +1165,3 @@ + 'name':"libcxx-sphinx-docs

Re: [PATCH] D12265: [ZORG] Add support for libc++ to SphinxDocBuilder.py

EricWF closed this revision. EricWF added a comment. r245912. http://reviews.llvm.org/D12265 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D11737: Add -linker (and -linker=) alias for -fuse-ld=

filcab updated this revision to Diff 33030. filcab marked an inline comment as done. filcab added a comment. Addressed Eric's comment. http://reviews.llvm.org/D11737 Files: include/clang/Driver/Options.td test/Driver/fuse-ld.c Index: test/Driver/fuse-ld.c ==

Re: patch: clarify diagnostic when returned value doesn't match function return type

LGTM with a caveat. Changing diagnostic messages is scary because it can inadvertently break tools and IDE's. So there is a chance someone might want this reverted in the future. But for now it seems helpful :). vedant On Aug 24, 2015, at 1:19 PM, Nick Lewycky via cfe-commits wrote: > On 19

r245913 - [clang-cl] Only respect /Oy- for x86_32

Author: majnemer Date: Mon Aug 24 19:46:45 2015 New Revision: 245913 URL: http://llvm.org/viewvc/llvm-project?rev=245913&view=rev Log: [clang-cl] Only respect /Oy- for x86_32 The /Oy- flag should have no effect for 64-bit X86, it has reliable unwind tables. Modified: cfe/trunk/include/clang/

Re: [PATCH] D11737: Add -linker (and -linker=) alias for -fuse-ld=

echristo accepted this revision. echristo added a comment. This revision is now accepted and ready to land. One inline comment and then OK. -eric Comment at: test/Driver/fuse-ld.c:27-33 @@ -26,1 +26,9 @@ +// -linker= is an alias to fuse-ld. Don't need to retry all combination

Re: [PATCH] D11737: Add -linker (and -linker=) alias for -fuse-ld=

pcc added a subscriber: pcc. Comment at: test/Driver/fuse-ld.c:32-33 @@ +31,4 @@ +// RUN: -B%S/Inputs/basic_freebsd_tree/usr/bin 2>&1 \ +// RUN: | FileCheck %s -check-prefix=CHECK-FREEBSD-GOLD +// CHECK-FREEBSD-LINKER-GOLD: Inputs/basic_freebsd_tree/usr/bin{{/|\\+}}ld.gold +

Re: [PATCH] D11737: Add -linker (and -linker=) alias for -fuse-ld=

thakis added a subscriber: thakis. This revision is now accepted and ready to land. Comment at: include/clang/Driver/Options.td:1853 @@ -1853,1 +1852,3 @@ +def fuse_ld_EQ : Joined<["-"], "fuse-ld=">, HelpText<"Use linker ">, Group; +def linker_EQ : Joined<["-"], "linker=">, Alias

Re: [PATCH] D11737: Add -linker (and -linker=) alias for -fuse-ld=

echristo added inline comments. Comment at: include/clang/Driver/Options.td:1853 @@ -1853,1 +1852,3 @@ +def fuse_ld_EQ : Joined<["-"], "fuse-ld=">, HelpText<"Use linker ">, Group; +def linker_EQ : Joined<["-"], "linker=">, Alias, MetaVarName<"">; thakis wrote:

Re: patch: clarify diagnostic when returned value doesn't match function return type

On Mon, Aug 24, 2015 at 5:44 PM, Vedant Kumar via cfe-commits < cfe-commits@lists.llvm.org> wrote: > LGTM with a caveat. > > Changing diagnostic messages is scary because it can inadvertently break > tools and IDE's. So there is a chance someone might want this reverted in > the future. But for no

Re: patch: clarify diagnostic when returned value doesn't match function return type

I can't find any reference to a policy against changing diagnostics. We should be in the clear. vedant On Aug 24, 2015, at 5:49 PM, David Majnemer wrote: > On Mon, Aug 24, 2015 at 5:44 PM, Vedant Kumar via cfe-commits > wrote: > LGTM with a caveat. > > Changing diagnostic messages is scary b

Re: [PATCH] Have clang list the imported modules in the debug info

On Mon, Aug 24, 2015 at 5:33 PM, Adrian Prantl wrote: > > On Aug 24, 2015, at 4:17 PM, David Blaikie wrote: > > > > On Mon, Aug 24, 2015 at 3:34 PM, Adrian Prantl wrote: > >> >> On Aug 24, 2015, at 2:01 PM, David Blaikie wrote: >> >> >> >> On Mon, Aug 24, 2015 at 1:23 PM, Adrian Prantl wrote:

  1   2   >