Re: [libunwind] r364217 - Merging r360861:

2019-07-08 Thread Dimitry Andric via cfe-commits
On 24 Jun 2019, at 20:40, Tom Stellard via cfe-commits wrote: > > Author: tstellar > Date: Mon Jun 24 11:40:58 2019 > New Revision: 364217 > > URL: http://llvm.org/viewvc/llvm-project?rev=364217&view=rev > Log: > Merging r360861: > >

[libunwind] r365539 - Revert merge of r360861:

2019-07-09 Thread Dimitry Andric via cfe-commits
Author: dim Date: Tue Jul 9 12:14:43 2019 New Revision: 365539 URL: http://llvm.org/viewvc/llvm-project?rev=365539&view=rev Log: Revert merge of r360861: r360861 | mstorsjo | 2019-05-15 23:49:13 -0700 (Wed, 15 May 2019) | 1

[libunwind] r365542 - Merging r360861, with an additional change to also add the PPC64_OPD1

2019-07-09 Thread Dimitry Andric via cfe-commits
Author: dim Date: Tue Jul 9 12:22:59 2019 New Revision: 365542 URL: http://llvm.org/viewvc/llvm-project?rev=365542&view=rev Log: Merging r360861, with an additional change to also add the PPC64_OPD1 and PPC64_OPD2 lines to the DEFINE_LIBUNWIND_PRIVATE_FUNCTION() macro, which was removed in r35764

Re: [libunwind] r364217 - Merging r360861:

2019-07-09 Thread Dimitry Andric via cfe-commits
On 9 Jul 2019, at 04:54, Tom Stellard wrote: > > On 07/08/2019 11:51 AM, Dimitry Andric wrote: >> On 24 Jun 2019, at 20:40, Tom Stellard via cfe-commits >> wrote: >>> >>> Author: tstellar >>> Date: Mon Jun 24 11:40:58 2019 >>> New Revision: 364217 >>> >>> URL: http://llvm.org/viewvc/llvm-proj

[clang] 02cfa75 - Bump the default target CPU for i386-freebsd to i686

2020-07-12 Thread Dimitry Andric via cfe-commits
Author: Dimitry Andric Date: 2020-07-12T23:45:22+02:00 New Revision: 02cfa7530d9e7cfd8ea940dab4173afb7938b831 URL: https://github.com/llvm/llvm-project/commit/02cfa7530d9e7cfd8ea940dab4173afb7938b831 DIFF: https://github.com/llvm/llvm-project/commit/02cfa7530d9e7cfd8ea940dab4173afb7938b831.diff

[clang] ee31adb - Populate CUDA flags on FreeBSD too, as many other toolchains do.

2019-11-18 Thread Dimitry Andric via cfe-commits
Author: Dimitry Andric Date: 2019-11-18T21:54:25+01:00 New Revision: ee31adb7fa42f5b601d9612f23755b4604f83cac URL: https://github.com/llvm/llvm-project/commit/ee31adb7fa42f5b601d9612f23755b4604f83cac DIFF: https://github.com/llvm/llvm-project/commit/ee31adb7fa42f5b601d9612f23755b4604f83cac.diff

[clang] 62654ca - Restore functionality of --sysroot on FreeBSD after b18cb9c47

2020-02-15 Thread Dimitry Andric via cfe-commits
Author: Dimitry Andric Date: 2020-02-16T01:50:06+01:00 New Revision: 62654cab7e654384ba503d6b62b1054cde19a553 URL: https://github.com/llvm/llvm-project/commit/62654cab7e654384ba503d6b62b1054cde19a553 DIFF: https://github.com/llvm/llvm-project/commit/62654cab7e654384ba503d6b62b1054cde19a553.diff

[clang-tools-extra] 58592f6 - Include for std::abort() in clangd

2020-01-24 Thread Dimitry Andric via cfe-commits
Author: Dimitry Andric Date: 2020-01-24T20:52:37+01:00 New Revision: 58592f6c49249293f79698cfcb31dba532e12603 URL: https://github.com/llvm/llvm-project/commit/58592f6c49249293f79698cfcb31dba532e12603 DIFF: https://github.com/llvm/llvm-project/commit/58592f6c49249293f79698cfcb31dba532e12603.diff

Re: r313784 - Remove offset size check in nullptr arithmetic handling

2018-04-24 Thread Dimitry Andric via cfe-commits
On 20 Sep 2017, at 20:06, Andrew Kaylor via cfe-commits wrote: > > Author: akaylor > Date: Wed Sep 20 11:06:44 2017 > New Revision: 313784 > > URL: http://llvm.org/viewvc/llvm-project?rev=313784&view=rev > Log: > Remove offset size check in nullptr arithmetic handling > > Differential Revision

r320755 - Don't trigger -Wuser-defined-literals for system headers

2017-12-14 Thread Dimitry Andric via cfe-commits
Author: dim Date: Thu Dec 14 14:32:24 2017 New Revision: 320755 URL: http://llvm.org/viewvc/llvm-project?rev=320755&view=rev Log: Don't trigger -Wuser-defined-literals for system headers Summary: In D41064, I proposed adding `#pragma clang diagnostic ignored "-Wuser-defined-literals"` to some of

[clang-tools-extra] r321016 - Fix more inconsistent line endings. NFC.

2017-12-18 Thread Dimitry Andric via cfe-commits
Author: dim Date: Mon Dec 18 11:46:56 2017 New Revision: 321016 URL: http://llvm.org/viewvc/llvm-project?rev=321016&view=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: clang-tools

r321016 - Fix more inconsistent line endings. NFC.

2017-12-18 Thread Dimitry Andric via cfe-commits
Author: dim Date: Mon Dec 18 11:46:56 2017 New Revision: 321016 URL: http://llvm.org/viewvc/llvm-project?rev=321016&view=rev Log: Fix more inconsistent line endings. NFC. Modified: cfe/trunk/include/clang/AST/OperationKinds.def cfe/trunk/lib/Parse/ParseTemplate.cpp cfe/trunk/www/cxx_d

Re: [libcxx] r319736 - Land D28253 which fixes PR28929 (which we mistakenly marked as fixed before)

2018-01-06 Thread Dimitry Andric via cfe-commits
On 5 Dec 2017, at 05:09, Marshall Clow via cfe-commits wrote: > > Author: marshall > Date: Mon Dec 4 20:09:49 2017 > New Revision: 319736 > > URL: http://llvm.org/viewvc/llvm-project?rev=319736&view=rev > Log: > Land D28253 which fixes PR28929 (which we mistakenly marked as fixed before) ... >

[libcxx] r321963 - Add pre-C++11 is_constructible wrappers for 3 arguments

2018-01-07 Thread Dimitry Andric via cfe-commits
Author: dim Date: Sun Jan 7 08:45:11 2018 New Revision: 321963 URL: http://llvm.org/viewvc/llvm-project?rev=321963&view=rev Log: Add pre-C++11 is_constructible wrappers for 3 arguments Summary: After rL319736 for D28253 (which fixes PR28929), gcc cannot compile `` anymore in pre-C+11 modes, com

[libcxx] r324855 - Add default C++ ABI libname and include paths for FreeBSD

2018-02-11 Thread Dimitry Andric via cfe-commits
Author: dim Date: Sun Feb 11 14:31:05 2018 New Revision: 324855 URL: http://llvm.org/viewvc/llvm-project?rev=324855&view=rev Log: Add default C++ ABI libname and include paths for FreeBSD Summary: As noted in a discussion about testing the LLVM 6.0.0 release candidates (with libc++) for FreeBSD,

[libcxx] r325027 - Put type attributes after class keyword

2018-02-13 Thread Dimitry Andric via cfe-commits
Author: dim Date: Tue Feb 13 09:40:59 2018 New Revision: 325027 URL: http://llvm.org/viewvc/llvm-project?rev=325027&view=rev Log: Put type attributes after class keyword Summary: Compiling `` in C++17 or higher mode results in: ``` functional:2500:1: warning: attribute '__visibility__' is ignore

[libcxx] r325028 - Make the ctype_byname::widen test cases pass on FreeBSD.

2018-02-13 Thread Dimitry Andric via cfe-commits
Author: dim Date: Tue Feb 13 09:43:24 2018 New Revision: 325028 URL: http://llvm.org/viewvc/llvm-project?rev=325028&view=rev Log: Make the ctype_byname::widen test cases pass on FreeBSD. Modified: libcxx/trunk/test/std/localization/locale.categories/category.ctype/locale.ctype.byname/widen_1

Re: [PATCH] D43394: [X86] Add 'sahf' CPU feature, and emit __LAHFSAHF__ for it

2018-02-16 Thread Dimitry Andric via cfe-commits
On 17 Feb 2018, at 00:12, Craig Topper via Phabricator wrote: > > craig.topper added inline comments. > > > > Comment at: lib/Basic/Targets/X86.cpp:1049 > > + if (HasLAHFSAHF) > +Builder.defineMacro("__LAHFSAHF__"); > > dim wrote: >> craig.topper wrote:

r325446 - [X86] Add 'sahf' CPU feature to frontend

2018-02-17 Thread Dimitry Andric via cfe-commits
Author: dim Date: Sat Feb 17 13:04:35 2018 New Revision: 325446 URL: http://llvm.org/viewvc/llvm-project?rev=325446&view=rev Log: [X86] Add 'sahf' CPU feature to frontend Summary: Make clang accept `-msahf` (and `-mno-sahf`) flags to activate the `+sahf` feature for the backend, for bug 36028 (In

r336008 - Request init/fini array on FreeBSD 12 and later

2018-06-29 Thread Dimitry Andric via cfe-commits
Author: dim Date: Fri Jun 29 12:18:17 2018 New Revision: 336008 URL: http://llvm.org/viewvc/llvm-project?rev=336008&view=rev Log: Request init/fini array on FreeBSD 12 and later Summary: It seems a bad idea to change the default in the middle of a release branch due to possible changes in global

[clang-tools-extra] r323040 - Assume the shared library path variable is LD_LIBRARY_PATH on systems

2018-01-20 Thread Dimitry Andric via cfe-commits
Author: dim Date: Sat Jan 20 06:34:33 2018 New Revision: 323040 URL: http://llvm.org/viewvc/llvm-project?rev=323040&view=rev Log: Assume the shared library path variable is LD_LIBRARY_PATH on systems except Darwin and Windows. This prevents inserting an environment variable with an empty name (wh

r329827 - Document -std= values for different languages

2018-04-11 Thread Dimitry Andric via cfe-commits
Author: dim Date: Wed Apr 11 10:21:52 2018 New Revision: 329827 URL: http://llvm.org/viewvc/llvm-project?rev=329827&view=rev Log: Document -std= values for different languages Summary: After a remark on a FreeBSD mailing list that the clang man page did not have any list of possible values for th

r372261 - On PowerPC, Secure-PLT by default for FreeBSD 13 and higher

2019-09-18 Thread Dimitry Andric via cfe-commits
Author: dim Date: Wed Sep 18 13:58:03 2019 New Revision: 372261 URL: http://llvm.org/viewvc/llvm-project?rev=372261&view=rev Log: On PowerPC, Secure-PLT by default for FreeBSD 13 and higher Summary: In https://svnweb.freebsd.org/changeset/base/349351, FreeBSD 13 and higher transitioned to Secure-

[PATCH] D15166: Fix C++ support on recent DragonFly BSD releases

2015-12-02 Thread Dimitry Andric via cfe-commits
dim created this revision. dim added reviewers: joerg, rsmith. dim added a subscriber: cfe-commits. [ Copied from https://llvm.org/bugs/show_bug.cgi?id=25597 ] Clang support for DragonFly BSD is lagging a bit, resulting in poor support for c++. DragonFlyBSD is unique in that it has two base comp

r256461 - For FreeBSD on mips, pass -G options to the linker

2015-12-26 Thread Dimitry Andric via cfe-commits
Author: dim Date: Sun Dec 27 00:47:09 2015 New Revision: 256461 URL: http://llvm.org/viewvc/llvm-project?rev=256461&view=rev Log: For FreeBSD on mips, pass -G options to the linker Summary: On {mips,mipsel,mips64,mips64el}-freebsd, we need to pass any -G option to the linker. See also: https:/

Re: [PATCH] D10137: On {mips, mipsel, mips64, mips64el}-freebsd, we need to pass any -G option to the assembler.

2015-12-27 Thread Dimitry Andric via cfe-commits
dim updated this revision to Diff 43652. dim added a comment. Reapplied to the latest trunk. http://reviews.llvm.org/D10137 Files: lib/Driver/Tools.cpp Index: lib/Driver/Tools.cpp === --- lib/Driver/Tools.cpp +++ lib/Driver/Tool

Re: [PATCH] D10137: On {mips, mipsel, mips64, mips64el}-freebsd, we need to pass any -G option to the assembler.

2015-12-27 Thread Dimitry Andric via cfe-commits
dim commandeered this revision. dim added a reviewer: brooks. dim added a comment. Picking this up, since it needs a minor change to apply. http://reviews.llvm.org/D10137 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/

Re: [PATCH] D15166: Fix C++ support on recent DragonFly BSD releases

2015-12-27 Thread Dimitry Andric via cfe-commits
dim added a comment. Tried this with trunk r256465, but I get the following test failure: FAIL: Clang :: Driver/dragonfly.c (3997 of 30174) TEST 'Clang :: Driver/dragonfly.c' FAILED Script: -- /home/dim/obj/llvm-256465M-trun

Re: [PATCH] D15166: Fix C++ support on recent DragonFly BSD releases

2015-12-27 Thread Dimitry Andric via cfe-commits
dim updated this revision to Diff 43654. dim added a comment. Fix the test to check for --enable-new-dtags. http://reviews.llvm.org/D15166 Files: lib/Driver/ToolChains.cpp lib/Driver/Tools.cpp lib/Frontend/InitHeaderSearch.cpp test/Driver/dragonfly.c Index: test/Driver/dragonfly.c

r256467 - Fix C++ support on recent DragonFly BSD releases

2015-12-27 Thread Dimitry Andric via cfe-commits
Author: dim Date: Sun Dec 27 04:01:44 2015 New Revision: 256467 URL: http://llvm.org/viewvc/llvm-project?rev=256467&view=rev Log: Fix C++ support on recent DragonFly BSD releases Summary: [ Copied from https://llvm.org/bugs/show_bug.cgi?id=25597 ] Clang support for DragonFly BSD is lagging a bit

Re: [PATCH] D10137: On {mips, mipsel, mips64, mips64el}-freebsd, we need to pass any -G option to the assembler.

2015-12-27 Thread Dimitry Andric via cfe-commits
dim updated this revision to Diff 43655. dim added a comment. Add a test for passing -G0 to the assembler. http://reviews.llvm.org/D10137 Files: lib/Driver/Tools.cpp test/Driver/freebsd-mips-as.c Index: test/Driver/freebsd-mips-as.c =

r256468 - On {mips, mipsel, mips64, mips64el}-freebsd, we need to pass any -G option to the assembler.

2015-12-27 Thread Dimitry Andric via cfe-commits
Author: dim Date: Sun Dec 27 04:36:44 2015 New Revision: 256468 URL: http://llvm.org/viewvc/llvm-project?rev=256468&view=rev Log: On {mips,mipsel,mips64,mips64el}-freebsd, we need to pass any -G option to the assembler. Summary: See also: https://sourceware.org/binutils/docs/as/MIPS-Options.html

Re: [PATCH] D15791: Disable generating movt on FreeBSD

2015-12-27 Thread Dimitry Andric via cfe-commits
dim added a comment. In http://reviews.llvm.org/D15791#317175, @emaste wrote: > > @davide, any idea whether lld will be able to handle movt correctly? If so, > > we might want to make this dependent on -fuse-ld=bfd or -fuse-ld=lld ? > > > I'm sure lld will handle everything we need eventually (i

Re: [PATCH] D7164: Recognize "CC" and "clang-CC" as C++ compiler aliases on FreeBSD

2015-12-30 Thread Dimitry Andric via cfe-commits
dim updated this revision to Diff 43784. dim added a comment. Updating this diff for the moving of FindDriverSuffix() to lib/Driver/ToolChain.cpp. http://reviews.llvm.org/D7164 Files: lib/Driver/ToolChain.cpp Index: lib/Driver/ToolChain.cpp ==

Re: [PATCH] D7164: Recognize "CC" and "clang-CC" as C++ compiler aliases on FreeBSD

2015-12-30 Thread Dimitry Andric via cfe-commits
dim abandoned this revision. dim added a comment. Abandoning this in favor of a wrapper shell script, which will call `/usr/bin/c++` instead. http://reviews.llvm.org/D7164 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.or

[PATCH] D15849: Fix several accidental DOS line endings in source files

2016-01-03 Thread Dimitry Andric via cfe-commits
dim created this revision. dim added a reviewer: joerg. dim added a subscriber: cfe-commits. There are a number of files in the tree which have been accidentally checked in with DOS line endings. Convert these to native line endings. There are also a few files which have DOS line endings on purp

r256704 - Fix several accidental DOS line endings in source files

2016-01-03 Thread Dimitry Andric via cfe-commits
Author: dim Date: Sun Jan 3 09:55:40 2016 New Revision: 256704 URL: http://llvm.org/viewvc/llvm-project?rev=256704&view=rev Log: Fix several accidental DOS line endings in source files Summary: There are a number of files in the tree which have been accidentally checked in with DOS line endings

r256742 - Convert test/CXX/lex/lex.literal/lex.string/p4.cpp back to DOS line

2016-01-04 Thread Dimitry Andric via cfe-commits
Author: dim Date: Mon Jan 4 04:17:48 2016 New Revision: 256742 URL: http://llvm.org/viewvc/llvm-project?rev=256742&view=rev Log: Convert test/CXX/lex/lex.literal/lex.string/p4.cpp back to DOS line endings, since the file is supposed to have them, according to its comments. Also set its svn:eol-s

[PATCH] D15899: Add -fno-movt frontend option, to disable movt/movw on ARM

2016-01-05 Thread Dimitry Andric via cfe-commits
dim created this revision. dim added reviewers: davide, echristo, dexonsmith. dim added subscribers: emaste, ahatanak, cfe-commits, davide. Herald added subscribers: rengolin, aemerson. In rL256641, @davide turned off movt generation by default for FreeBSD. This was because our ld is very old, and

Re: [PATCH] D15899: Add -fno-movt frontend option, to disable movt/movw on ARM

2016-01-05 Thread Dimitry Andric via cfe-commits
dim added a comment. In http://reviews.llvm.org/D15899#319902, @davide wrote: > I'm not opposed to this but ... this will likely it only 11 (and maybe 10.3 > if backported) -- what about people running <= 10.2 ? Isn't this a problem > for them? People running ports-provided or hand-built clan

Re: [PATCH] D15899: Add -fno-movt frontend option, to disable movt/movw on ARM

2016-01-05 Thread Dimitry Andric via cfe-commits
dim updated this revision to Diff 44059. dim added a comment. Rename `-fno-movt` to `-mno-movt`. http://reviews.llvm.org/D15899 Files: include/clang/Driver/Options.td lib/Driver/Tools.cpp test/Driver/arm-no-movt.c Index: test/Driver/arm-no-movt.c =

r256920 - Add -fno-movt frontend option, to disable movt/movw on ARM

2016-01-05 Thread Dimitry Andric via cfe-commits
Author: dim Date: Wed Jan 6 01:42:18 2016 New Revision: 256920 URL: http://llvm.org/viewvc/llvm-project?rev=256920&view=rev Log: Add -fno-movt frontend option, to disable movt/movw on ARM Summary: In rL256641, @davide turned off movt generation by default for FreeBSD. This was because our ld is

[PATCH] D13051: Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available.

2015-09-22 Thread Dimitry Andric via cfe-commits
dim created this revision. dim added reviewers: EricWF, mclow.lists. dim added subscribers: majnemer, jroelofs, cfe-commits. Herald added a subscriber: aemerson. In rL241532, atomic_support.h was added, which provides handling of atomic operations for libc++. When atomic builtins are not availabl

Re: [PATCH] D13051: Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available.

2015-09-22 Thread Dimitry Andric via cfe-commits
dim added a comment. In http://reviews.llvm.org/D13051#250836, @EricWF wrote: > Does anything actually need this? This interface was never really meant to be > complete, just want was needed. I need it at least for FreeBSD/arm, since it defaults to armv4, and it does not yet have a full set o

[libcxx] r248313 - Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available.

2015-09-22 Thread Dimitry Andric via cfe-commits
Author: dim Date: Tue Sep 22 13:55:37 2015 New Revision: 248313 URL: http://llvm.org/viewvc/llvm-project?rev=248313&view=rev Log: Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available. Summary: In rL241532, atomic_support.h was added, which provides handling of atom

Re: r244063 - Add missing atomic libcall support.

2015-09-22 Thread Dimitry Andric via cfe-commits
On 05 Aug 2015, at 18:57, James Y Knight wrote: > Author: jyknight > Date: Wed Aug 5 11:57:36 2015 > New Revision: 244063 > > URL: http://llvm.org/viewvc/llvm-project?rev=244063&view=rev > Log: > Add missing atomic libcall support. > > Support for emitting libcalls for __atomic_fetch_nand and >

Re: [PATCH] D13051: Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available.

2015-09-22 Thread Dimitry Andric via cfe-commits
dim added a comment. In http://reviews.llvm.org/D13051#250921, @jroelofs wrote: > In http://reviews.llvm.org/D13051#250910, @EricWF wrote: > > > @jroelofs @dim, could we fallback to the __sync_* builtins on arm? > The actual implementations of these __sync builtins should still come from somew

Re: [libcxx] r292276 - Fix std::string assignment ambiguity from braced initializer lists.

2017-01-23 Thread Dimitry Andric via cfe-commits
> On 17 Jan 2017, at 23:10, Eric Fiselier via cfe-commits > wrote: > > Author: ericwf > Date: Tue Jan 17 16:10:32 2017 > New Revision: 292276 > > URL: http://llvm.org/viewvc/llvm-project?rev=292276&view=rev > Log: > Fix std::string assignment ambiguity from braced initializer lists. > > When

[libcxx] r293197 - Disable thread safety analysis for some functions in __thread_support

2017-01-26 Thread Dimitry Andric via cfe-commits
Author: dim Date: Thu Jan 26 12:37:18 2017 New Revision: 293197 URL: http://llvm.org/viewvc/llvm-project?rev=293197&view=rev Log: Disable thread safety analysis for some functions in __thread_support Many thread-related libc++ test cases fail on FreeBSD, due to the following -Werror warnings:

Re: [libcxx] r293197 - Disable thread safety analysis for some functions in __thread_support

2017-01-29 Thread Dimitry Andric via cfe-commits
I'd like to merge this to the release_40 branch. Eric, you OK with that? -Dimitry > On 26 Jan 2017, at 19:37, Dimitry Andric via cfe-commits > wrote: > > Author: dim > Date: Thu Jan 26 12:37:18 2017 > New Revision: 293197 > > URL: http://llvm.org/viewvc/llv

[libcxx] r293443 - Merging r293197:

2017-01-29 Thread Dimitry Andric via cfe-commits
Author: dim Date: Sun Jan 29 14:30:51 2017 New Revision: 293443 URL: http://llvm.org/viewvc/llvm-project?rev=293443&view=rev Log: Merging r293197: r293197 | dim | 2017-01-26 19:37:18 +0100 (Thu, 26 Jan 2017) | 43 lines Disab

Re: [libcxx] r293197 - Disable thread safety analysis for some functions in __thread_support

2017-01-29 Thread Dimitry Andric via cfe-commits
Eric, you OK with that? > > -Dimitry > > > On 26 Jan 2017, at 19:37, Dimitry Andric via cfe-commits > > mailto:cfe-commits@lists.llvm.org>> wrote: > > > > Author: dim > > Date: Thu Jan 26 12:37:18 2017 > > New Revision: 293197 > > > > URL: h

Re: r290539 - [inline-asm]No error for conflict between inputs\outputs and clobber list

2017-01-31 Thread Dimitry Andric via cfe-commits
On 26 Dec 2016, at 13:23, Marina Yatsina via cfe-commits wrote: > > Author: myatsina > Date: Mon Dec 26 06:23:42 2016 > New Revision: 290539 > > URL: http://llvm.org/viewvc/llvm-project?rev=290539&view=rev > Log: > [inline-asm]No error for conflict between inputs\outputs and clobber list > > A

Re: [libcxxabi] r308470 - Drop 'svn' suffix from version number.

2017-09-09 Thread Dimitry Andric via cfe-commits
On 19 Jul 2017, at 16:04, Hans Wennborg via cfe-commits wrote: > > Author: hans > Date: Wed Jul 19 07:04:19 2017 > New Revision: 308470 > > URL: http://llvm.org/viewvc/llvm-project?rev=308470&view=rev > Log: > Drop 'svn' suffix from version number. [Replying to this commit, since I don't have

[libcxx] r302362 - Ensure showbase does not overflow do_put buffers

2017-05-06 Thread Dimitry Andric via cfe-commits
Author: dim Date: Sat May 6 15:58:50 2017 New Revision: 302362 URL: http://llvm.org/viewvc/llvm-project?rev=302362&view=rev Log: Ensure showbase does not overflow do_put buffers Summary: In https://bugs.freebsd.org/207918, Daniel McRobb describes how using std::showbase with ostreams can cause t

r304836 - Print registered targets in clang's version information

2017-06-06 Thread Dimitry Andric via cfe-commits
Author: dim Date: Tue Jun 6 16:54:21 2017 New Revision: 304836 URL: http://llvm.org/viewvc/llvm-project?rev=304836&view=rev Log: Print registered targets in clang's version information Summary: Other llvm tools display their registered targets when showing version information, but for some reaso

[clang-tools-extra] r304837 - Adjust SetVersionPrinter call for D33899

2017-06-06 Thread Dimitry Andric via cfe-commits
Author: dim Date: Tue Jun 6 16:54:45 2017 New Revision: 304837 URL: http://llvm.org/viewvc/llvm-project?rev=304837&view=rev Log: Adjust SetVersionPrinter call for D33899 Summary: In D33899, I'm adding a `raw_ostream &` parameter to the function objects passed to `cl::SetVersionPrinter`. Adjust

r304899 - Only print registered targets for `--version`

2017-06-07 Thread Dimitry Andric via cfe-commits
Author: dim Date: Wed Jun 7 07:05:41 2017 New Revision: 304899 URL: http://llvm.org/viewvc/llvm-project?rev=304899&view=rev Log: Only print registered targets for `--version` Summary: In D33900, I added printing of the registered targets in clang's `PrintVersion` function, which is not only used

Re: [PATCH] D21329: Rename and rework `_LIBCPP_TRIVIAL_PAIR_COPY_CTOR`. Move FreeBSD configuration in-tree.

2016-07-04 Thread Dimitry Andric via cfe-commits
dim accepted this revision. dim added a comment. This revision is now accepted and ready to land. LGTM. Eric, apologies that this took some time to review. I finally dug out my old test cases for the original problem report in FreeBSD, and re-tested them against this patch. (These test cases

Re: [PATCH] D21329: Rename and rework `_LIBCPP_TRIVIAL_PAIR_COPY_CTOR`. Move FreeBSD configuration in-tree.

2016-07-05 Thread Dimitry Andric via cfe-commits
dim added a comment. @theraven, do you have any other feedback? :) http://reviews.llvm.org/D21329 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D22010: Add more gcc compatibility names to clang's cpuid.h

2016-07-05 Thread Dimitry Andric via cfe-commits
dim created this revision. dim added reviewers: joerg, rsmith. dim added a subscriber: cfe-commits. Some cpuid bit defines are named slightly different from how gcc's cpuid.h calls them. Define a few more compatibility names to appease software built for gcc: * `bit_PCLMUL` alias of `bit_PC

Re: [PATCH] D21329: Rename and rework `_LIBCPP_TRIVIAL_PAIR_COPY_CTOR`. Move FreeBSD configuration in-tree.

2016-07-07 Thread Dimitry Andric via cfe-commits
dim added a comment. In http://reviews.llvm.org/D21329#475105, @theraven wrote: > Looks fine to me, though I wonder if we want to move to the new ABI for > FreeBSD11 and use the old one for <=10. For 11 it is certainly too late now, since it is ABI/API frozen now, and we would still need to f

Re: r286129 - [OPENMP] Fixed codegen for __real/__imag expressions in atomic

2016-11-07 Thread Dimitry Andric via cfe-commits
Hi Alexey, Does this look like a good candidate for 3.9.1? -Dimitry > On 07 Nov 2016, at 19:15, Alexey Bataev via cfe-commits > wrote: > > Author: abataev > Date: Mon Nov 7 12:15:02 2016 > New Revision: 286129 > > URL: http://llvm.org/viewvc/llvm-project?rev=286129&view=rev > Log: > [OPENMP

Re: [PATCH] D20037: Make __FreeBSD_cc_version predefined macro configurable at build time

2016-05-16 Thread Dimitry Andric via cfe-commits
dim added a subscriber: cfe-commits. dim added a comment. Belatedly adding cfe-commits. http://reviews.llvm.org/D20037 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r270240 - Make __FreeBSD_cc_version predefined macro configurable at build time

2016-05-20 Thread Dimitry Andric via cfe-commits
Author: dim Date: Fri May 20 12:27:22 2016 New Revision: 270240 URL: http://llvm.org/viewvc/llvm-project?rev=270240&view=rev Log: Make __FreeBSD_cc_version predefined macro configurable at build time The `FreeBSDTargetInfo` class has always set the `__FreeBSD_cc_version` predefined macro to a rat

Re: [PATCH] D20037: Make __FreeBSD_cc_version predefined macro configurable at build time

2016-05-20 Thread Dimitry Andric via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL270240: Make __FreeBSD_cc_version predefined macro configurable at build time (authored by dim). Changed prior to commit: http://reviews.llvm.org/D20037?vs=56476&id=57952#toc Repository: rL LLVM htt

Re: r266775 - Pass dwarf-version to cc1as.

2016-08-16 Thread Dimitry Andric via cfe-commits
Hi Hans, Can we please merge this one to release_39 too? I forgot all about it, sorry. Doug, can you please approve? -Dimitry > On 19 Apr 2016, at 19:43, Douglas Katzman via cfe-commits > wrote: > > Author: dougk > Date: Tue Apr 19 12:43:54 2016 > New Revision: 266775 > > URL: http://llvm.

Re: r266775 - Pass dwarf-version to cc1as.

2016-08-16 Thread Dimitry Andric via cfe-commits
Ah, I missed Doug's follow-up commit r266797, which moved the code again. I noticed that the RenderDebugEnablingArgs() call was missing in the spot indicated below, but it was only moved. -Dimitry > On 17 Aug 2016, at 01:17, Hans Wennborg wrote: > > Looks like it was committed back in April,

Re: [libcxx] r278904 - Support allocators with explicit conversion constructors. Fixes bug #29000

2016-08-17 Thread Dimitry Andric via cfe-commits
Marshall, this is maybe a good candidate for merging to release_39? -Dimitry > On 17 Aug 2016, at 07:58, Marshall Clow via cfe-commits > wrote: > > Author: marshall > Date: Wed Aug 17 00:58:40 2016 > New Revision: 278904 > > URL: http://llvm.org/viewvc/llvm-project?rev=278904&view=rev > Log:

[PATCH] D23960: Avoid narrowing warnings in __bitset constructor

2016-08-27 Thread Dimitry Andric via cfe-commits
dim created this revision. dim added reviewers: mclow.lists, EricWF. dim added subscribers: emaste, cfe-commits. When I compile is compiled with warnings enabled, I get the following error (which is interesting in itself, it should only be a warning): /usr/include/c++/v1/bitset:265:16: error

[PATCH] D23961: Avoid embedded preprocessor directives in __tree

2016-08-27 Thread Dimitry Andric via cfe-commits
dim created this revision. dim added reviewers: mclow.lists, EricWF. dim added subscribers: emaste, cfe-commits. When I compile as part of clang, with -pedantic enabled, I get the following warnings: In file included from /usr/include/c++/v1/map:442: /usr/include/c++/v1/__tree:874:2: war

[libcxx] r279926 - Avoid embedded preprocessor directives in __tree

2016-08-27 Thread Dimitry Andric via cfe-commits
Author: dim Date: Sat Aug 27 14:32:03 2016 New Revision: 279926 URL: http://llvm.org/viewvc/llvm-project?rev=279926&view=rev Log: Avoid embedded preprocessor directives in __tree Similar to rL242623, move C++ version checks outside of _NOEXCEPT_() macro invocation argument lists, to avoid "embedd

Re: [PATCH] D23961: Avoid embedded preprocessor directives in __tree

2016-08-27 Thread Dimitry Andric via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL279926: Avoid embedded preprocessor directives in __tree (authored by dim). Changed prior to commit: https://reviews.llvm.org/D23961?vs=69492&id=69496#toc Repository: rL LLVM https://reviews.llvm.or

Re: [PATCH] D23961: Avoid embedded preprocessor directives in __tree

2016-08-28 Thread Dimitry Andric via cfe-commits
dim added a comment. In https://reviews.llvm.org/D23961#527284, @vitalybuka wrote: > PPC bots are broken by this patch > http://lab.llvm.org:8011/builders/sanitizer-ppc64be-linux/builds/3547 > http://lab.llvm.org:8011/builders/sanitizer-ppc64le-linux/builds/2353 I saw some mails about it, bu

Re: [PATCH] D23921: Remove va_start diagnostic false positive with enumerations

2016-08-28 Thread Dimitry Andric via cfe-commits
dim added a subscriber: dim. dim added a comment. This works for me. I had two test cases from the FreeBSD source tree which resulted in warnings, e.g.: cddl/contrib/opensolaris/lib/libnvpair/libnvpair.c:388:15: error: passing an object that undergoes default argument promotion to 'va_start'

Re: [PATCH] D23960: Avoid narrowing warnings in __bitset constructor

2016-09-01 Thread Dimitry Andric via cfe-commits
dim added a comment. Ping... https://reviews.llvm.org/D23960 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D23960: Avoid narrowing warnings in __bitset constructor

2016-09-01 Thread Dimitry Andric via cfe-commits
dim added inline comments. Comment at: include/bitset:262 @@ -261,3 +261,3 @@ #elif __SIZEOF_SIZE_T__ == 4 -: __first_{__v, __v >> __bits_per_word} +: __first_{static_cast<__storage_type>(__v), static_cast<__storage_type>(__v >> __bits_per_word)} #else

Re: [PATCH] D23960: Avoid narrowing warnings in __bitset constructor

2016-09-02 Thread Dimitry Andric via cfe-commits
dim added a comment. @mclow.lists, I tested code generation (on i386-freebsd) with this small sample program: #include void f(const std::bitset<53>&); void g(void) { std::bitset<53> bs(0x1234567890abcdef); f(bs); } The assembly doesn't change at all after this fix: g

[libcxx] r280543 - Avoid narrowing warnings in __bitset constructor

2016-09-02 Thread Dimitry Andric via cfe-commits
Author: dim Date: Fri Sep 2 16:02:11 2016 New Revision: 280543 URL: http://llvm.org/viewvc/llvm-project?rev=280543&view=rev Log: Avoid narrowing warnings in __bitset constructor When is compiled with warnings enabled, on a platform where size_t is 4 bytes, it results in errors similar to:

Re: [PATCH] D23960: Avoid narrowing warnings in __bitset constructor

2016-09-02 Thread Dimitry Andric via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL280543: Avoid narrowing warnings in __bitset constructor (authored by dim). Changed prior to commit: https://reviews.llvm.org/D23960?vs=69489&id=70231#toc Repository: rL LLVM https://reviews.llvm.or

[libunwind] r280669 - Add missing _US_ACTION_MASK constant to unwind.h

2016-09-05 Thread Dimitry Andric via cfe-commits
Author: dim Date: Mon Sep 5 13:01:13 2016 New Revision: 280669 URL: http://llvm.org/viewvc/llvm-project?rev=280669&view=rev Log: Add missing _US_ACTION_MASK constant to unwind.h Summary: During building of recent compiler-rt sources on FreeBSD for arm, I noticed that our unwind.h (which originat

Re: [PATCH] D16264: For FreeBSD, use _p variants of libraries for linking C++ programs

2016-02-13 Thread Dimitry Andric via cfe-commits
dim updated this revision to Diff 47913. dim added a comment. Add checks for -lm and -lm_p, when profiling is on. http://reviews.llvm.org/D16264 Files: lib/Driver/ToolChains.cpp lib/Driver/ToolChains.h test/Driver/freebsd.cpp Index: test/Driver/freebsd.cpp ===

r260851 - As reported in https://llvm.org/bugs/show_bug.cgi?id=25496, on FreeBSD,

2016-02-14 Thread Dimitry Andric via cfe-commits
Author: dim Date: Sun Feb 14 10:08:20 2016 New Revision: 260851 URL: http://llvm.org/viewvc/llvm-project?rev=260851&view=rev Log: As reported in https://llvm.org/bugs/show_bug.cgi?id=25496, on FreeBSD, C++ programs compiled for profiling (using `-pg`) should be linked with `-lc++_p` (or `-lstdc++_

Re: r260851 - As reported in https://llvm.org/bugs/show_bug.cgi?id=25496, on FreeBSD,

2016-02-14 Thread Dimitry Andric via cfe-commits
On 14 Feb 2016, at 17:08, Dimitry Andric via cfe-commits wrote: > > Author: dim > Date: Sun Feb 14 10:08:20 2016 > New Revision: 260851 > > URL: http://llvm.org/viewvc/llvm-project?rev=260851&view=rev > Log: > As reported in https://llvm.org/bugs/show_bug.c

Re: [PATCH] D16264: For FreeBSD, use _p variants of libraries for linking C++ programs

2016-02-14 Thread Dimitry Andric via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL260851: As reported in https://llvm.org/bugs/show_bug.cgi?id=25496, on FreeBSD, (authored by dim). Changed prior to commit: http://reviews.llvm.org/D16264?vs=47913&id=47930#toc Repository: rL LLVM h

Re: r260851 - As reported in https://llvm.org/bugs/show_bug.cgi?id=25496, on FreeBSD,

2016-02-16 Thread Dimitry Andric via cfe-commits
On 16 Feb 2016, at 18:51, Hans Wennborg wrote: > > On Sun, Feb 14, 2016 at 8:14 AM, Dimitry Andric wrote: >> On 14 Feb 2016, at 17:08, Dimitry Andric via cfe-commits >> wrote: >>> >>> Author: dim >>> Date: Sun Feb 14 10:08:20 2016 >>> New

Re: r261422 - Fix handling of vaargs on PPC32 when going from regsave to overflow.

2016-02-20 Thread Dimitry Andric via cfe-commits
On 20 Feb 2016, at 09:31, Roman Divacky via cfe-commits wrote: > > Author: rdivacky > Date: Sat Feb 20 02:31:24 2016 > New Revision: 261422 > > URL: http://llvm.org/viewvc/llvm-project?rev=261422&view=rev > Log: > Fix handling of vaargs on PPC32 when going from regsave to overflow. > > It can

Re: [PATCH] D17286: Make FreeBSD and NetBSD use CLANG_DEFAULT_CXX_STDLIB

2016-02-29 Thread Dimitry Andric via cfe-commits
dim added subscribers: joerg, dim. dim added a comment. LGTM, at least. @emaste, @joerg, OK with you too? http://reviews.llvm.org/D17286 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi

Re: [Diffusion] rL244063: Add missing atomic libcall support.

2016-03-08 Thread Dimitry Andric via cfe-commits
dim accepted this commit. Users: jyknight (Author, Auditor) 3.7-release (Auditor) cfe-commits (Auditor) tstellarAMD (Auditor) compnerd (Auditor) majnemer (Auditor) rsmith (Auditor) dim (Auditor) http://reviews.llvm.org/rL244063 ___ cf

Re: [Diffusion] rL244063: Add missing atomic libcall support.

2016-03-08 Thread Dimitry Andric via cfe-commits
dim resigned from this audit. Users: jyknight (Author, Auditor) 3.7-release (Auditor) cfe-commits (Auditor) tstellarAMD (Auditor) compnerd (Auditor) majnemer (Auditor) rsmith (Auditor) dim (Auditor) http://reviews.llvm.org/rL244063 __

[PATCH] D15967: Turn off lldb debug tuning by default for FreeBSD

2016-01-07 Thread Dimitry Andric via cfe-commits
dim created this revision. dim added reviewers: emaste, probinson. dim added a subscriber: cfe-commits. Herald added a subscriber: emaste. This is the clang part of D15966. In rL256104, debugger tuning was added to the clang driver, and again the default for FreeBSD was set to lldb. The default

r257104 - Turn off lldb debug tuning by default for FreeBSD

2016-01-07 Thread Dimitry Andric via cfe-commits
Author: dim Date: Thu Jan 7 16:09:47 2016 New Revision: 257104 URL: http://llvm.org/viewvc/llvm-project?rev=257104&view=rev Log: Turn off lldb debug tuning by default for FreeBSD Summary: This is the clang part of D15966. In rL256104, debugger tuning was added to the clang driver, and again the

r257509 - Add a clang test for r257376 (Ensure -mcpu=xscale works for arm targets).

2016-01-12 Thread Dimitry Andric via cfe-commits
Author: dim Date: Tue Jan 12 13:40:55 2016 New Revision: 257509 URL: http://llvm.org/viewvc/llvm-project?rev=257509&view=rev Log: Add a clang test for r257376 (Ensure -mcpu=xscale works for arm targets). Added: cfe/trunk/test/Driver/arm-xscale.c Added: cfe/trunk/test/Driver/arm-xscale.c URL:

Re: r256468 - On {mips, mipsel, mips64, mips64el}-freebsd, we need to pass any -G option to the assembler.

2016-01-14 Thread Dimitry Andric via cfe-commits
>> Of Joerg Sonnenberger via cfe-commits >> Sent: 27 December 2015 11:59 >> To: cfe-commits@lists.llvm.org >> Subject: Re: r256468 - On {mips, mipsel, mips64, mips64el}-freebsd, we need >> to pass any -G option to the assembler. >> >> On Sun, Dec 27, 2015 at 10:

Re: [PATCH] D16102: Bug 25496 - clang++ -pg links with -lc++ instead of -lc++_p

2016-01-15 Thread Dimitry Andric via cfe-commits
dim added a subscriber: dim. dim added a comment. In http://reviews.llvm.org/D16102#324395, @davide wrote: > I'm fine with this solution but Saleem pointed out there are some changes > that need to be made before getting this in. @compnerd, what were these changes to be made? http://reviews.

[PATCH] D16264: For FreeBSD, use _p variants of libraries for linking C++ programs

2016-01-16 Thread Dimitry Andric via cfe-commits
dim created this revision. dim added reviewers: compnerd, davide, dws, emaste. dim added subscribers: rpokala, cfe-commits. Herald added a subscriber: emaste. As reported in https://llvm.org/bugs/show_bug.cgi?id=25496, on FreeBSD, C++ programs compiled for profiling (using `-pg`) should be linked

Re: [PATCH] D16102: Bug 25496 - clang++ -pg links with -lc++ instead of -lc++_p

2016-01-16 Thread Dimitry Andric via cfe-commits
dim commandeered this revision. dim added a reviewer: dws. dim added a comment. Going to abandon this review in favor of http://reviews.llvm.org/D16264. http://reviews.llvm.org/D16102 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://li

Re: [libcxx] r258107 - Fix PR#26175. Thanks to Josh Petrie for the report and the patch. Reviewed as http://reviews.llvm.org/D16262

2016-01-19 Thread Dimitry Andric via cfe-commits
On 19 Jan 2016, at 01:50, Marshall Clow via cfe-commits wrote: > > Author: marshall > Date: Mon Jan 18 18:50:37 2016 > New Revision: 258107 > > URL: http://llvm.org/viewvc/llvm-project?rev=258107&view=rev > Log: > Fix PR#26175. Thanks to Josh Petrie for the report and the patch. Reviewed as >

Re: r258110 - Fix PR26134: When substituting into default template arguments, keep CurContext unchanged.

2016-01-19 Thread Dimitry Andric via cfe-commits
Hi Richard, I am unsure if you are specifically the code owner of Sema, but can you please approve this change for the 3.8 branch? -Dimitry > On 19 Jan 2016, at 04:58, Faisal Vali via cfe-commits > wrote: > > Author: faisalv > Date: Mon Jan 18 21:58:55 2016 > New Revision: 258110 > > URL: h

[clang-tools-extra] r258327 - Ensure the lit.site.cfg and Unit/lit.site.cfg make targets do not use

2016-01-20 Thread Dimitry Andric via cfe-commits
Author: dim Date: Wed Jan 20 12:53:44 2016 New Revision: 258327 URL: http://llvm.org/viewvc/llvm-project?rev=258327&view=rev Log: Ensure the lit.site.cfg and Unit/lit.site.cfg make targets do not use the same temporary file, otherwise there is a race condition when using parallel make jobs. Modif

  1   2   >