Re: [PATCH] D12458: [clang] add __builtin_unpredictable and convert to metadata

2015-08-31 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. LGTM Comment at: lib/CodeGen/CodeGenFunction.cpp:1238 @@ -1217,3 +1237,3 @@ } - Builder.CreateCondBr(CondV, TrueBlock, FalseBlock, Weights); + Builder.CreateCondBr(CondV, TrueBlock, FalseBlock, Weights, Unpredictable); } spatel

Re: [PATCH] D12458: [clang] add __builtin_unpredictable and convert to metadata

2015-08-31 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added inline comments. Comment at: lib/CodeGen/CodeGenFunction.cpp:1238 @@ -1217,3 +1237,3 @@ } - Builder.CreateCondBr(CondV, TrueBlock, FalseBlock, Weights); + Builder.CreateCondBr(CondV, TrueBlock, FalseBlock, Weights, Unpredictable); } Is the

r254606 - Unxfail passing test: test/CodeGenCXX/crash.cpp

2015-12-03 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Thu Dec 3 06:03:56 2015 New Revision: 254606 URL: http://llvm.org/viewvc/llvm-project?rev=254606=rev Log: Unxfail passing test: test/CodeGenCXX/crash.cpp Modified: cfe/trunk/test/CodeGenCXX/crash.cpp Modified: cfe/trunk/test/CodeGenCXX/crash.cpp URL:

r255127 - [Hexagon] Use integrated assembler by default

2015-12-09 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Wed Dec 9 10:34:24 2015 New Revision: 255127 URL: http://llvm.org/viewvc/llvm-project?rev=255127=rev Log: [Hexagon] Use integrated assembler by default Modified: cfe/trunk/lib/Driver/ToolChains.h cfe/trunk/test/Driver/hexagon-toolchain-elf.c

r255667 - Unsupport test that should not be run on Hexagon

2015-12-15 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Tue Dec 15 13:14:24 2015 New Revision: 255667 URL: http://llvm.org/viewvc/llvm-project?rev=255667=rev Log: Unsupport test that should not be run on Hexagon Modified: cfe/trunk/test/CodeGen/ms_this.cpp Modified: cfe/trunk/test/CodeGen/ms_this.cpp URL:

r255510 - [Hexagon] Xfail two tests that fail due to over-aligning arrays

2015-12-14 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Mon Dec 14 11:17:20 2015 New Revision: 255510 URL: http://llvm.org/viewvc/llvm-project?rev=255510=rev Log: [Hexagon] Xfail two tests that fail due to over-aligning arrays Modified: cfe/trunk/test/CodeGen/c-strings.c

r256934 - [Hexagon] Use back_inserter as target iterator in std::copy

2016-01-06 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Wed Jan 6 08:13:11 2016 New Revision: 256934 URL: http://llvm.org/viewvc/llvm-project?rev=256934=rev Log: [Hexagon] Use back_inserter as target iterator in std::copy Modified: cfe/trunk/lib/Driver/ToolChains.cpp Modified: cfe/trunk/lib/Driver/ToolChains.cpp URL:

r256977 - [Hexagon] Treat -march and -mcpu as equivalent

2016-01-06 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Wed Jan 6 15:12:03 2016 New Revision: 256977 URL: http://llvm.org/viewvc/llvm-project?rev=256977=rev Log: [Hexagon] Treat -march and -mcpu as equivalent Modified: cfe/trunk/lib/Driver/ToolChains.cpp Modified: cfe/trunk/lib/Driver/ToolChains.cpp URL:

r256978 - [Hexagon] Expand -mvN to -mcpu=hexagonvN

2016-01-06 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Wed Jan 6 15:27:42 2016 New Revision: 256978 URL: http://llvm.org/viewvc/llvm-project?rev=256978=rev Log: [Hexagon] Expand -mvN to -mcpu=hexagonvN Modified: cfe/trunk/include/clang/Driver/Options.td Modified: cfe/trunk/include/clang/Driver/Options.td URL:

r257765 - [Hexagon] Change all builtins returning "bool" to return "int" instead

2016-01-14 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Thu Jan 14 08:26:36 2016 New Revision: 257765 URL: http://llvm.org/viewvc/llvm-project?rev=257765=rev Log: [Hexagon] Change all builtins returning "bool" to return "int" instead Modified: cfe/trunk/include/clang/Basic/BuiltinsHexagon.def Modified:

r260680 - [Hexagon] Specify vector alignment in DataLayout string

2016-02-12 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Fri Feb 12 08:48:34 2016 New Revision: 260680 URL: http://llvm.org/viewvc/llvm-project?rev=260680=rev Log: [Hexagon] Specify vector alignment in DataLayout string The DataLayout can calculate alignment of vectors based on the alignment of the element type and the number of

Re: r258128 - Add -Wexpansion-to-undefined: warn when using `defined` in a macro definition.

2016-01-19 Thread Krzysztof Parzyszek via cfe-commits
This generates hundreds of warnings when doing check-all. Here's the offending code: utils/unittest/googletest/include/gtest/internal/gtest-port.h // Cygwin 1.7 and below doesn't support ::std::wstring. // Solaris' libc++ doesn't support it either. Android has // no support for it at least

r261251 - Make test less prone to attribute changes

2016-02-18 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Thu Feb 18 14:02:03 2016 New Revision: 261251 URL: http://llvm.org/viewvc/llvm-project?rev=261251=rev Log: Make test less prone to attribute changes Modified: cfe/trunk/test/CodeGen/ms-intrinsics.c Modified: cfe/trunk/test/CodeGen/ms-intrinsics.c URL:

Re: [PATCH] D17784: Check if LLVM_PREFIX is defined before using it.

2016-03-03 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. This seemed fine to me, but at the second thought, the lines 2610-2613 in the original should be removed. http://reviews.llvm.org/D17784 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D17784: Check if LLVM_PREFIX is defined before using it.

2016-03-03 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. This seemed fine to me, but at the second thought, the lines 2610-2613 in the original should be removed. http://reviews.llvm.org/D17784 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

r269200 - [Hexagon] Avoid spurious failures in test/Driver/hexagon-toolchain-elf.c

2016-05-11 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Wed May 11 11:11:22 2016 New Revision: 269200 URL: http://llvm.org/viewvc/llvm-project?rev=269200=rev Log: [Hexagon] Avoid spurious failures in test/Driver/hexagon-toolchain-elf.c Modified: cfe/trunk/test/Driver/hexagon-toolchain-elf.c Modified:

r269934 - [Hexagon] Recognize "q" and "v" in inline-asm as register constraints

2016-05-18 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Wed May 18 09:56:14 2016 New Revision: 269934 URL: http://llvm.org/viewvc/llvm-project?rev=269934=rev Log: [Hexagon] Recognize "q" and "v" in inline-asm as register constraints Clang follow-up to r269933. Added: cfe/trunk/test/CodeGen/hexagon-inline-asm.c Modified:

r268637 - [Hexagon] Add a testcase for __builtin_HEXAGON_A2_tfrpi

2016-05-05 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Thu May 5 10:55:54 2016 New Revision: 268637 URL: http://llvm.org/viewvc/llvm-project?rev=268637=rev Log: [Hexagon] Add a testcase for __builtin_HEXAGON_A2_tfrpi Modified: cfe/trunk/test/CodeGen/builtins-hexagon.c Modified: cfe/trunk/test/CodeGen/builtins-hexagon.c

r266647 - [Hexagon] Define macros __HVX__ and __HVXDBL__ when appropriate

2016-04-18 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Mon Apr 18 13:38:11 2016 New Revision: 266647 URL: http://llvm.org/viewvc/llvm-project?rev=266647=rev Log: [Hexagon] Define macros __HVX__ and __HVXDBL__ when appropriate Added: cfe/trunk/test/Preprocessor/hexagon-predefines.c Modified:

r266989 - [Hexagon] Define architecture version macros for hexagonv55

2016-04-21 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Thu Apr 21 09:30:04 2016 New Revision: 266989 URL: http://llvm.org/viewvc/llvm-project?rev=266989=rev Log: [Hexagon] Define architecture version macros for hexagonv55 Modified: cfe/trunk/lib/Basic/Targets.cpp cfe/trunk/test/Preprocessor/hexagon-predefines.c

Re: r266976 - Correctly parse GCC-style asm line following MS-style asm line.

2016-04-21 Thread Krzysztof Parzyszek via cfe-commits
On 4/21/2016 7:49 AM, Renato Golin via cfe-commits wrote: MS-style inline assembly is not available: No available targets are compatible with this triple. http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15/builds/11538 Same thing on Hexagon:

Re: r266976 - Correctly parse GCC-style asm line following MS-style asm line.

2016-04-21 Thread Krzysztof Parzyszek via cfe-commits
On 4/21/2016 9:46 AM, Renato Golin wrote: On 21 April 2016 at 14:58, Krzysztof Parzyszek via cfe-commits <cfe-commits@lists.llvm.org> wrote: Same thing on Hexagon: http://lab.llvm.org:8011/builders/clang-hexagon-elf/builds/37972 Trying to fix in r266993. It worked! Thanks! -Krz

Re: [PATCH] D22766: Handle -mlong-calls on Hexagon

2016-07-28 Thread Krzysztof Parzyszek via cfe-commits
kparzysz updated this revision to Diff 65919. kparzysz marked 2 inline comments as done. kparzysz added a comment. Removed the cleanup part. Repository: rL LLVM https://reviews.llvm.org/D22766 Files: include/clang/Driver/Options.td lib/Basic/Targets.cpp lib/Driver/Tools.cpp

[PATCH] D22766: Handle -mlong-calls on Hexagon

2016-07-25 Thread Krzysztof Parzyszek via cfe-commits
kparzysz created this revision. kparzysz added reviewers: t.p.northover, echristo. kparzysz added a subscriber: cfe-commits. kparzysz set the repository for this revision to rL LLVM. Herald added subscribers: mehdi_amini, aemerson. Make -mlong-calls a general (i.e. non ARM-specific) option.

Re: [PATCH] D22766: Handle -mlong-calls on Hexagon

2016-07-27 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. It likely is sufficient. The code in Targets.cpp adds checking for "long-calls" feature in hasFeature, but it's not explicitly used anywhere. The rest of the changes are mostly to clean up the handling of feature strings. Repository: rL LLVM

Re: [PATCH] D22766: Handle -mlong-calls on Hexagon

2016-08-04 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. In https://reviews.llvm.org/D22766#505512, @echristo wrote: > You haven't removed the custom handling? The one from the driver? If I remove it, it won't be passed to the compiler (and the testcase will fail). I forgot to move the no-long-calls from the ARM group to

Re: [PATCH] D22766: Handle -mlong-calls on Hexagon

2016-08-03 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. Ping? Repository: rL LLVM https://reviews.llvm.org/D22766 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D22766: Handle -mlong-calls on Hexagon

2016-08-17 Thread Krzysztof Parzyszek via cfe-commits
kparzysz updated this revision to Diff 68416. kparzysz added a comment. Moving mno_long_calls to m_Group as well. I'm hoping this is all that's needed: what I understand as "custom handling" is still needed, since this option is only valid for ARM and Hexagon. The Hexagon way of handling

r280093 - [Hexagon] Use handleTargetFeaturesGroup to process target-specific features

2016-08-30 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Tue Aug 30 09:17:10 2016 New Revision: 280093 URL: http://llvm.org/viewvc/llvm-project?rev=280093=rev Log: [Hexagon] Use handleTargetFeaturesGroup to process target-specific features Modified: cfe/trunk/lib/Basic/Targets.cpp cfe/trunk/lib/Driver/Tools.cpp

r280089 - Handle -mlong-calls on Hexagon

2016-08-30 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Tue Aug 30 08:57:50 2016 New Revision: 280089 URL: http://llvm.org/viewvc/llvm-project?rev=280089=rev Log: Handle -mlong-calls on Hexagon Differential Revision:://reviews.llvm.org/D22766 Added: cfe/trunk/test/Driver/hexagon-long-calls.c Modified:

Re: [PATCH] D22766: Handle -mlong-calls on Hexagon

2016-08-23 Thread Krzysztof Parzyszek via cfe-commits
kparzysz marked an inline comment as done. Comment at: include/clang/Driver/Options.td:1380 @@ +1379,3 @@ +def mlong_calls : Flag<["-"], "mlong-calls">, Group, + HelpText<"ARM: Generate an indirect jump to enable jumps further than 64M, Hexagon: Generate constant-extended

Re: [PATCH] D22766: Handle -mlong-calls on Hexagon

2016-08-22 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. Ping. Repository: rL LLVM https://reviews.llvm.org/D22766 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D22766: Handle -mlong-calls on Hexagon

2016-08-29 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. Ping. Repository: rL LLVM https://reviews.llvm.org/D22766 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D25811: [libcxx] Fix toupper/tolower tests for UTF-8 locale

2016-10-19 Thread Krzysztof Parzyszek via cfe-commits
kparzysz created this revision. kparzysz added a reviewer: EricWF. kparzysz added a subscriber: cfe-commits. kparzysz set the repository for this revision to rL LLVM. The characters like '\xDA' or '\xFA' are not valid representations of anything in UTF-8, so toupper and tolower return the

[PATCH] D25811: [libcxx] Fix toupper/tolower tests for UTF-8 locale

2016-10-20 Thread Krzysztof Parzyszek via cfe-commits
kparzysz updated this revision to Diff 75292. kparzysz added a comment. Unxfail these tests on Linux. Repository: rL LLVM https://reviews.llvm.org/D25811 Files: test/std/localization/locale.categories/category.ctype/locale.ctype.byname/tolower_1.pass.cpp

[PATCH] D25597: Try to fix buildbot failure in VirtualFileSystem caused by r284129.

2016-10-14 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. Right on entry to the asserting function: (gdb) where #0 0x752b9870 in (anonymous namespace)::RedirectingFileSystem::lookupPath(llvm::sys::path::const_iterator, llvm::sys::path::const_iterator, (anonymous namespace)::Entry*) () from

[PATCH] D25597: Try to fix buildbot failure in VirtualFileSystem caused by r284129.

2016-10-14 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. dbgs showed that the path components were `.. target hexagon include`. The paths are constructed in lib/Driver/ToolChains.cpp, many are based on "getHexagonTargetDir". https://reviews.llvm.org/D25597 ___ cfe-commits

[PATCH] D25597: Try to fix buildbot failure in VirtualFileSystem caused by r284129.

2016-10-14 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. The `..:target:hexagon:include:` is debug code printing all the path components, separated by :. https://reviews.llvm.org/D25597 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D25811: [libcxx] Fix toupper/tolower tests for UTF-8 locale

2016-10-20 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. In https://reviews.llvm.org/D25811#575105, @EricWF wrote: > Seems like we should figure out why these pass on ToT OS X. All of them have // XFAIL: with_system_cxx_lib=x86_64-apple-darwin11 // XFAIL: with_system_cxx_lib=x86_64-apple-darwin12 Maybe that's it. I don't

[PATCH] D25811: [libcxx] Fix toupper/tolower tests for UTF-8 locale

2016-11-02 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. In https://reviews.llvm.org/D25811#586000, @arphaman wrote: > I've tested this patch on OS X and these 4 tests that you've changed now fail. > > The `XFAIL: with_system_cxx_lib` lines don't have any effect on my system - > these tests get invoked as expected by lit.

[PATCH] D25811: [libcxx] Fix toupper/tolower tests for UTF-8 locale

2016-11-02 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. And thanks for help. :) Repository: rL LLVM https://reviews.llvm.org/D25811 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D25597: Try to fix buildbot failure in VirtualFileSystem caused by r284129.

2016-10-17 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. This change bool FileManager::getStatValue(StringRef Path, FileData , bool isFile, std::unique_ptr *F) { // FIXME: FileSystemOpts shouldn't be passed in here, all paths should be // absolute! llvm::dbgs() <<

r284389 - Revert r284383, while I figure out how to reproduce the failures locally

2016-10-17 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Mon Oct 17 09:47:29 2016 New Revision: 284389 URL: http://llvm.org/viewvc/llvm-project?rev=284389=rev Log: Revert r284383, while I figure out how to reproduce the failures locally Modified: cfe/trunk/lib/Driver/ToolChains.cpp Modified:

[PATCH] D25597: Try to fix buildbot failure in VirtualFileSystem caused by r284129.

2016-10-17 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. In https://reviews.llvm.org/D25597#571532, @kparzysz wrote: > `FileSystemOpts.WorkingDir: ''` There is no space between the single quotes. https://reviews.llvm.org/D25597 ___ cfe-commits mailing list

[PATCH] D25597: Try to fix buildbot failure in VirtualFileSystem caused by r284129.

2016-10-17 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. Printing Path shows `/../target/hexagon/include` https://reviews.llvm.org/D25597 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r284383 - Return correct path from HexagonToolChain::getHexagonTargetDir

2016-10-17 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Mon Oct 17 08:23:41 2016 New Revision: 284383 URL: http://llvm.org/viewvc/llvm-project?rev=284383=rev Log: Return correct path from HexagonToolChain::getHexagonTargetDir This problem was exposed by r284129, causing clang-hexagon-elf to fail clang tests. Modified:

r284392 - Reapply r284383. The test failures were due to a missing dir in test/

2016-10-17 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Mon Oct 17 10:30:10 2016 New Revision: 284392 URL: http://llvm.org/viewvc/llvm-project?rev=284392=rev Log: Reapply r284383. The test failures were due to a missing dir in test/ Added: cfe/trunk/test/Driver/Inputs/hexagon_tree/Tools/bin/ Modified:

[PATCH] D25597: Try to fix buildbot failure in VirtualFileSystem caused by r284129.

2016-10-17 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. I committed https://reviews.llvm.org/rL284383 and the Hexagon bot is passing now. (The patch was reverted, but then it was recommitted.) https://reviews.llvm.org/D25597 ___ cfe-commits mailing list

r284402 - Add a dummy file in each subdirectory in test/Driver/Inputs/hexagon_tree

2016-10-17 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Mon Oct 17 13:04:05 2016 New Revision: 284402 URL: http://llvm.org/viewvc/llvm-project?rev=284402=rev Log: Add a dummy file in each subdirectory in test/Driver/Inputs/hexagon_tree Git does not store empty subdirectories (while SVN does). Git clone of the clang repository

Re: r284392 - Reapply r284383. The test failures were due to a missing dir in test/

2016-10-17 Thread Krzysztof Parzyszek via cfe-commits
ng is /Users/tim/llvm/llvm/tools/clang/test/Driver/Inputs/hexagon_tree/Tools/bin/hexagon/include, and there is no hexagon_tree directory in ToT. I think this might be something to do with git not actually recording empty directories. Could you have another look? Cheers. Tim. On 17 October 2016

r297778 - [Hexagon] Recognize hexagonv62 as a valid target CPU

2017-03-14 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Tue Mar 14 15:29:23 2017 New Revision: 297778 URL: http://llvm.org/viewvc/llvm-project?rev=297778=rev Log: [Hexagon] Recognize hexagonv62 as a valid target CPU Modified: cfe/trunk/include/clang/Driver/Options.td cfe/trunk/lib/Basic/Targets.cpp

r308035 - [Hexagon] Add intrinsics for data cache operations

2017-07-14 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Fri Jul 14 09:01:24 2017 New Revision: 308035 URL: http://llvm.org/viewvc/llvm-project?rev=308035=rev Log: [Hexagon] Add intrinsics for data cache operations This is the clang part, adding support for void __builtin_HEXAGON_Y2_dccleana(void*); void

r308763 - [Hexagon] Add inline-asm constraint 'a' for modifier register class

2017-07-21 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Fri Jul 21 11:07:15 2017 New Revision: 308763 URL: http://llvm.org/viewvc/llvm-project?rev=308763=rev Log: [Hexagon] Add inline-asm constraint 'a' for modifier register class For example asm ("memw(%0++%1) = %2" : : "r"(addr),"a"(mod),"r"(val) : "memory") Modified:

r308782 - [Hexagon] Attempt to fix selfhost bots

2017-07-21 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Fri Jul 21 14:23:05 2017 New Revision: 308782 URL: http://llvm.org/viewvc/llvm-project?rev=308782=rev Log: [Hexagon] Attempt to fix selfhost bots The codegen testcase was checking for specific value names. Make it ignore the names instead as they are irrelevant.

r301361 - [Hexagon] Set -ffp-contract=fast at -O3 unless explicitly specified

2017-04-25 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Tue Apr 25 15:51:51 2017 New Revision: 301361 URL: http://llvm.org/viewvc/llvm-project?rev=301361=rev Log: [Hexagon] Set -ffp-contract=fast at -O3 unless explicitly specified Modified: cfe/trunk/lib/Driver/ToolChains/Hexagon.cpp

r301364 - [Hexagon] Handle -O4 when checking optimization level

2017-04-25 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Tue Apr 25 16:31:55 2017 New Revision: 301364 URL: http://llvm.org/viewvc/llvm-project?rev=301364=rev Log: [Hexagon] Handle -O4 when checking optimization level Modified: cfe/trunk/lib/Driver/ToolChains/Hexagon.cpp Modified: cfe/trunk/lib/Driver/ToolChains/Hexagon.cpp

r302895 - [Hexagon] Make sure to pass empty struct arguments with nontrivial ctors

2017-05-12 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Fri May 12 08:18:07 2017 New Revision: 302895 URL: http://llvm.org/viewvc/llvm-project?rev=302895=rev Log: [Hexagon] Make sure to pass empty struct arguments with nontrivial ctors Thanks to Richard Smith for the suggested fix. This fixes llvm.org/PR33009 Modified:

Re: r302817 - PR22877: When constructing an array via a constructor with a default argument

2017-05-11 Thread Krzysztof Parzyszek via cfe-commits
Hexagon is still broken. :( http://lab.llvm.org:8011/builders/clang-hexagon-elf/builds/7942 -Krzysztof On 5/11/2017 1:58 PM, Richard Smith via cfe-commits wrote: Author: rsmith Date: Thu May 11 13:58:24 2017 New Revision: 302817 URL: http://llvm.org/viewvc/llvm-project?rev=302817=rev Log:

r303358 - [CodeGen] Propagate LValueBaseInfo instead of AlignmentSource

2017-05-18 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Thu May 18 12:07:11 2017 New Revision: 303358 URL: http://llvm.org/viewvc/llvm-project?rev=303358=rev Log: [CodeGen] Propagate LValueBaseInfo instead of AlignmentSource The functions creating LValues propagated information about alignment source. Extend the propagated data

r303851 - [CodeGen] Pessimize aliasing for member unions (and may-alias) objects

2017-05-25 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Thu May 25 07:55:47 2017 New Revision: 303851 URL: http://llvm.org/viewvc/llvm-project?rev=303851=rev Log: [CodeGen] Pessimize aliasing for member unions (and may-alias) objects Use the TBAA info of the omnipotent char for these objects. Differential Revision:

Re: [PATCH] D32199: [TBAASan] A TBAA Sanitizer (Clang)

2017-05-01 Thread Krzysztof Parzyszek via cfe-commits
On 5/1/2017 2:16 PM, Hal Finkel via cfe-commits wrote: On 05/01/2017 12:49 PM, Daniel Berlin wrote: On 04/21/2017 06:03 AM, Hal Finkel via Phabricator wrote: ... Our struct-path TBAA does the following: struct X { int a, b; }; X x { 50, 100 }; X *o = (X*) (((int*) ) +

Re: [PATCH] D32199: [TBAASan] A TBAA Sanitizer (Clang)

2017-05-02 Thread Krzysztof Parzyszek via cfe-commits
On 5/1/2017 6:17 PM, Hal Finkel wrote: However, the example can also be written as: struct X { int a, b; }; X x { 50, 100 }; X *o = (X*) int a_is_b = o->a; // This is UB (or so we say)? and then the pointer arithmetic considerations don't seem to apply. I know

r303419 - Restore and update documentation comment for EmitPointerWithAlignment

2017-05-19 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Fri May 19 07:03:34 2017 New Revision: 303419 URL: http://llvm.org/viewvc/llvm-project?rev=303419=rev Log: Restore and update documentation comment for EmitPointerWithAlignment Modified: cfe/trunk/lib/CodeGen/CodeGenFunction.h Modified:

Re: Patch bug 27628

2017-09-08 Thread Krzysztof Parzyszek via cfe-commits
This should to to cfe-commits. Redirecting. -Krzysztof On 9/8/2017 10:25 AM, Antoni Boucher via llvm-commits wrote: Hello. I've fixed the bug 27628: https://bugs.llvm.org/show_bug.cgi?id=27628 I attached the patch. Thanks. ___ llvm-commits

Re: Patch bug 27628

2017-09-08 Thread Krzysztof Parzyszek via cfe-commits
Aaand the patch itself... -K On 9/8/2017 10:32 AM, Krzysztof Parzyszek via cfe-commits wrote: This should to to cfe-commits. Redirecting. -Krzysztof On 9/8/2017 10:25 AM, Antoni Boucher via llvm-commits wrote: Hello. I've fixed the bug 27628: https://bugs.llvm.org/show_bug.cgi?id=27628 I

Re: [clang-tools-extra] r318840 - [FindAllSymbols] Cache regexes, creating them is expensive

2017-11-25 Thread Krzysztof Parzyszek via cfe-commits
After upgrading my FreeBSD to the latest -STABLE this no longer fails. -Krzysztof PS. And system clang was upgraded to 5.0.0. Nice! On 11/25/2017 11:20 AM, Dimitry Andric wrote: Yeah, in the past libc++ broke its own ABI with http://llvm.org/viewvc/llvm-project?view=revision=194536, which

Re: [clang-tools-extra] r318840 - [FindAllSymbols] Cache regexes, creating them is expensive

2017-11-23 Thread Krzysztof Parzyszek via cfe-commits
There has been some problem with std::pair on FreeBSD (due to some ABI compatibility issue), but I don't know much about what it was. Commenting the reserve doesn't help, unfortunately. The same problem is now flagged in emplace_back. -Krzysztof On 11/23/2017 11:47 AM, Benjamin Kramer

Re: [clang-tools-extra] r318840 - [FindAllSymbols] Cache regexes, creating them is expensive

2017-11-23 Thread Krzysztof Parzyszek via cfe-commits
Hi, This broke build on FreeBSD 11: [100%] Building CXX object tools/clang/tools/extra/include-fixer/find-all-symbols/CMakeFiles/findAllSymbols.dir/HeaderMapCollector.cpp.o cd /w/bld/org/tools/clang/tools/extra/include-fixer/find-all-symbols &&

Re: [clang-tools-extra] r318840 - [FindAllSymbols] Cache regexes, creating them is expensive

2017-11-23 Thread Krzysztof Parzyszek via cfe-commits
+Dimitry. On 11/23/2017 12:50 PM, Benjamin Kramer wrote: I'm afraid I can't really help you here. You can try twiddling the code a bit by creating the pair explicitly and moving it into the vector, just to avoid the brokenness in the standard library. Not sure if that will help though. On

r320579 - [Hexagon] Add front-end support for Hexagon V65

2017-12-13 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Wed Dec 13 05:48:07 2017 New Revision: 320579 URL: http://llvm.org/viewvc/llvm-project?rev=320579=rev Log: [Hexagon] Add front-end support for Hexagon V65 Modified: cfe/trunk/include/clang/Driver/Options.td cfe/trunk/lib/Basic/Targets/Hexagon.cpp

r320410 - [Hexagon] Remove unsupported vlut intrinsics

2017-12-11 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Mon Dec 11 11:29:56 2017 New Revision: 320410 URL: http://llvm.org/viewvc/llvm-project?rev=320410=rev Log: [Hexagon] Remove unsupported vlut intrinsics Modified: cfe/trunk/test/CodeGen/builtins-hexagon.c Modified: cfe/trunk/test/CodeGen/builtins-hexagon.c URL:

r332105 - [Hexagon] Implement checking arguments of builtin calls

2018-05-11 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Fri May 11 09:41:51 2018 New Revision: 332105 URL: http://llvm.org/viewvc/llvm-project?rev=332105=rev Log: [Hexagon] Implement checking arguments of builtin calls Added: cfe/trunk/test/CodeGen/hexagon-check-builtins.c Modified: cfe/trunk/include/clang/Sema/Sema.h

r332383 - [Hexagon] Add driver options for subtarget features

2018-05-15 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Tue May 15 11:15:59 2018 New Revision: 332383 URL: http://llvm.org/viewvc/llvm-project?rev=332383=rev Log: [Hexagon] Add driver options for subtarget features Added: cfe/trunk/test/Driver/hexagon-memops.c cfe/trunk/test/Driver/hexagon-nvj.c

r323360 - [Hexagon] Accept lowercase b in -hvx-length=64b and -hvx-length=128b

2018-01-24 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Wed Jan 24 10:42:19 2018 New Revision: 323360 URL: http://llvm.org/viewvc/llvm-project?rev=323360=rev Log: [Hexagon] Accept lowercase b in -hvx-length=64b and -hvx-length=128b Modified: cfe/trunk/lib/Driver/ToolChains/Hexagon.cpp cfe/trunk/test/Driver/hexagon-hvx.c

r340622 - [Hexagon] Remove unneeded strings from builtin definitions, NFC

2018-08-24 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Fri Aug 24 10:13:42 2018 New Revision: 340622 URL: http://llvm.org/viewvc/llvm-project?rev=340622=rev Log: [Hexagon] Remove unneeded strings from builtin definitions, NFC Modified: cfe/trunk/include/clang/Basic/BuiltinsHexagon.def Modified:

r336933 - [Hexagon] Diagnose intrinsics not supported by selected CPU/HVX

2018-07-12 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Thu Jul 12 11:54:04 2018 New Revision: 336933 URL: http://llvm.org/viewvc/llvm-project?rev=336933=rev Log: [Hexagon] Diagnose intrinsics not supported by selected CPU/HVX Added: cfe/trunk/test/Sema/builtins-hexagon-v55.c cfe/trunk/test/Sema/builtins-hexagon-v60.c

r337049 - [Hexagon] Fix hvx-length feature name in testcases

2018-07-13 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Fri Jul 13 14:32:33 2018 New Revision: 337049 URL: http://llvm.org/viewvc/llvm-project?rev=337049=rev Log: [Hexagon] Fix hvx-length feature name in testcases Modified: cfe/trunk/test/CodeGen/builtins-hvx128.c cfe/trunk/test/CodeGen/builtins-hvx64.c Modified:

r326366 - [Hexagon] Add -ffixed-r19 driver option and translate it to +reserved-r19

2018-02-28 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Wed Feb 28 12:31:55 2018 New Revision: 326366 URL: http://llvm.org/viewvc/llvm-project?rev=326366=rev Log: [Hexagon] Add -ffixed-r19 driver option and translate it to +reserved-r19 Modified: cfe/trunk/include/clang/Driver/Options.td

r327393 - [Hexagon] Clang side of r327302 in LLVM

2018-03-13 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Tue Mar 13 06:30:43 2018 New Revision: 327393 URL: http://llvm.org/viewvc/llvm-project?rev=327393=rev Log: [Hexagon] Clang side of r327302 in LLVM Add option -m[no-]packets to control generation of instruction packets (enabled by default). Added:

r329923 - [Hexagon] Enable auto-vectorization only when -fvectorize was given

2018-04-12 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Thu Apr 12 09:25:35 2018 New Revision: 329923 URL: http://llvm.org/viewvc/llvm-project?rev=329923=rev Log: [Hexagon] Enable auto-vectorization only when -fvectorize was given Added: cfe/trunk/test/Driver/hexagon-vectorize.c Modified:

r329394 - [Hexagon] Remove default values from lambda parameters

2018-04-06 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Fri Apr 6 06:51:48 2018 New Revision: 329394 URL: http://llvm.org/viewvc/llvm-project?rev=329394=rev Log: [Hexagon] Remove default values from lambda parameters Modified: cfe/trunk/lib/CodeGen/CGBuiltin.cpp Modified: cfe/trunk/lib/CodeGen/CGBuiltin.cpp URL:

r330150 - [Hexagon] Emit a warning when -fvectorize is given without -mhvx

2018-04-16 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Mon Apr 16 12:11:17 2018 New Revision: 330150 URL: http://llvm.org/viewvc/llvm-project?rev=330150=rev Log: [Hexagon] Emit a warning when -fvectorize is given without -mhvx Modified: cfe/trunk/include/clang/Basic/DiagnosticDriverKinds.td

r328641 - Update test after r328635 in LLVM

2018-03-27 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Tue Mar 27 10:17:39 2018 New Revision: 328641 URL: http://llvm.org/viewvc/llvm-project?rev=328641=rev Log: Update test after r328635 in LLVM Modified: cfe/trunk/test/CodeGen/alias.c Modified: cfe/trunk/test/CodeGen/alias.c URL:

r328725 - [Hexagon] Add support for "new" circular buffer intrinsics

2018-03-28 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Wed Mar 28 12:40:57 2018 New Revision: 328725 URL: http://llvm.org/viewvc/llvm-project?rev=328725=rev Log: [Hexagon] Add support for "new" circular buffer intrinsics These instructions have been around for a long time, but we haven't supported intrinsics for them. The

r329077 - [Hexagon] Remove -mhvx-double and the corresponding subtarget feature

2018-04-03 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Tue Apr 3 08:59:10 2018 New Revision: 329077 URL: http://llvm.org/viewvc/llvm-project?rev=329077=rev Log: [Hexagon] Remove -mhvx-double and the corresponding subtarget feature Specifying the HVX vector length should be done via the -mhvx-length option. Modified:

r328776 - [Hexagon] Aid bit-reverse load intrinsics lowering with bitcode

2018-03-29 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Thu Mar 29 06:54:31 2018 New Revision: 328776 URL: http://llvm.org/viewvc/llvm-project?rev=328776=rev Log: [Hexagon] Aid bit-reverse load intrinsics lowering with bitcode The conversion of operatios to bitcode helps to eliminate an additional store in certain cases. We

r344786 - [Hexagon] Remove support for V4

2018-10-19 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Fri Oct 19 08:36:45 2018 New Revision: 344786 URL: http://llvm.org/viewvc/llvm-project?rev=344786=rev Log: [Hexagon] Remove support for V4 Modified: cfe/trunk/docs/ClangCommandLineReference.rst cfe/trunk/include/clang/Driver/Options.td

r345170 - [Hexagon] Flip hexagon-autohvx to be true by default

2018-10-24 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Wed Oct 24 10:55:18 2018 New Revision: 345170 URL: http://llvm.org/viewvc/llvm-project?rev=345170=rev Log: [Hexagon] Flip hexagon-autohvx to be true by default This will allow other generators of LLVM IR to use the auto-vectorizer without having to change that flag. Note:

r348415 - [Hexagon] Add support for Hexagon V66

2018-12-05 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Wed Dec 5 13:38:35 2018 New Revision: 348415 URL: http://llvm.org/viewvc/llvm-project?rev=348415=rev Log: [Hexagon] Add support for Hexagon V66 Modified: cfe/trunk/include/clang/Driver/Options.td cfe/trunk/lib/Basic/Targets/Hexagon.cpp

r348214 - [Hexagon] Fix intrinsic test

2018-12-03 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Mon Dec 3 15:52:33 2018 New Revision: 348214 URL: http://llvm.org/viewvc/llvm-project?rev=348214=rev Log: [Hexagon] Fix intrinsic test Modified: cfe/trunk/test/CodeGen/builtins-hexagon.c Modified: cfe/trunk/test/CodeGen/builtins-hexagon.c URL:

r364648 - [Hexagon] driver uses out-of-date option name and binary name

2019-06-28 Thread Krzysztof Parzyszek via cfe-commits
Author: kparzysz Date: Fri Jun 28 08:08:03 2019 New Revision: 364648 URL: http://llvm.org/viewvc/llvm-project?rev=364648=rev Log: [Hexagon] driver uses out-of-date option name and binary name Patch by A. Skrobov (t.yomitch). Differential Revision: https://reviews.llvm.org/D62127 Modified:

[clang] 99f5196 - [Hexagon] Handle remaining registers in getRegisterByName()

2019-10-29 Thread Krzysztof Parzyszek via cfe-commits
Author: Krzysztof Parzyszek Date: 2019-10-29T08:56:01-05:00 New Revision: 99f51960fdb5559d6720281bff9a63041452bf9a URL: https://github.com/llvm/llvm-project/commit/99f51960fdb5559d6720281bff9a63041452bf9a DIFF:

[clang] 57148e0 - [Hexagon] Fix ABI info for returning HVX vectors

2020-02-11 Thread Krzysztof Parzyszek via cfe-commits
Author: Krzysztof Parzyszek Date: 2020-02-11T12:38:54-06:00 New Revision: 57148e0379d30ecabd2a338c5bb9abbb3a0e314f URL: https://github.com/llvm/llvm-project/commit/57148e0379d30ecabd2a338c5bb9abbb3a0e314f DIFF:

[clang] bc413da - [Hexagon] Fix alignment info for __builtin_circ_lduh

2020-01-16 Thread Krzysztof Parzyszek via cfe-commits
Author: Krzysztof Parzyszek Date: 2020-01-16T10:54:45-06:00 New Revision: bc413da0865d6c0dba1c39f185298806c60890e3 URL: https://github.com/llvm/llvm-project/commit/bc413da0865d6c0dba1c39f185298806c60890e3 DIFF:

[clang] 237fd94 - [Hexagon] Remove unnecessary case in StringSwitch, NFC

2020-01-16 Thread Krzysztof Parzyszek via cfe-commits
Author: Krzysztof Parzyszek Date: 2020-01-16T10:00:57-06:00 New Revision: 237fd94312526c7aef55f929b51696bba451dab8 URL: https://github.com/llvm/llvm-project/commit/237fd94312526c7aef55f929b51696bba451dab8 DIFF:

[clang] 7f5f6ff - [Hexagon] Add preprocessor test for hexagonv66

2020-01-16 Thread Krzysztof Parzyszek via cfe-commits
Author: Krzysztof Parzyszek Date: 2020-01-16T10:18:58-06:00 New Revision: 7f5f6ff5476b6dafb5be2f9c102d9b324a313717 URL: https://github.com/llvm/llvm-project/commit/7f5f6ff5476b6dafb5be2f9c102d9b324a313717 DIFF:

[clang] 202446c - Add BuiltinsHexagonDep.def to clang module map

2020-01-16 Thread Krzysztof Parzyszek via cfe-commits
Author: Krzysztof Parzyszek Date: 2020-01-16T15:54:24-06:00 New Revision: 202446c639fdd27a54c3be268154a7c66af4f36d URL: https://github.com/llvm/llvm-project/commit/202446c639fdd27a54c3be268154a7c66af4f36d DIFF:

[clang] d0ca104 - [Hexagon] Refactor handling of circular load/store builtins, NFC

2020-03-09 Thread Krzysztof Parzyszek via cfe-commits
Author: Krzysztof Parzyszek Date: 2020-03-09T14:40:08-05:00 New Revision: d0ca1041ba0d2d6b8adc7f04d017502e44a118e3 URL: https://github.com/llvm/llvm-project/commit/d0ca1041ba0d2d6b8adc7f04d017502e44a118e3 DIFF:

[clang] 7406eb4 - [Hexagon] Avoid creating an empty target feature

2020-08-10 Thread Krzysztof Parzyszek via cfe-commits
Author: Krzysztof Parzyszek Date: 2020-08-10T10:37:24-05:00 New Revision: 7406eb4f6afd8df9bd4dbb918f5e7005ba71d58c URL: https://github.com/llvm/llvm-project/commit/7406eb4f6afd8df9bd4dbb918f5e7005ba71d58c DIFF:

[clang] a83475d - [Hexagon] Add -mv68 option to driver

2021-02-04 Thread Krzysztof Parzyszek via cfe-commits
Author: Krzysztof Parzyszek Date: 2021-02-04T10:29:34-06:00 New Revision: a83475d34b4550ff5bd40430d6537e630eb761f1 URL: https://github.com/llvm/llvm-project/commit/a83475d34b4550ff5bd40430d6537e630eb761f1 DIFF:

  1   2   3   >