[clang] c49866a - [clang] stop baremetal driver to append .a to lib

2020-02-13 Thread Christof Douma via cfe-commits
Author: Christof Douma Date: 2020-02-13T11:08:46Z New Revision: c49866acceb1ffbcc0f723993648f0678e92a91c URL: https://github.com/llvm/llvm-project/commit/c49866acceb1ffbcc0f723993648f0678e92a91c DIFF: https://github.com/llvm/llvm-project/commit/c49866acceb1ffbcc0f723993648f0678e92a91c.diff

Re: [libcxx] r342073 - Implement the infrastructure for feature-test macros. Very few actual feature test macros, though. Reviewed as: https://reviews.llvm.org/D51955

2018-10-22 Thread Christof Douma via cfe-commits
Hi Marshall and others. After this discussion and some internal discussions I agree with the idea that files without extension are treated as C++ header files and non-header files should either move out of the C++ code base or have a proper extension. The VERSION case is just the most visible

r306384 - Revert "Revert "[NFC] Refactor DiagnosticRenderer to use FullSourceLoc""

2017-06-27 Thread Christof Douma via cfe-commits
Author: christof Date: Tue Jun 27 02:50:38 2017 New Revision: 306384 URL: http://llvm.org/viewvc/llvm-project?rev=306384=rev Log: Revert "Revert "[NFC] Refactor DiagnosticRenderer to use FullSourceLoc"" This reverts commit r305688 meaning it reintroduces r305684. To repeat: [NFC] Refactor

r305688 - Revert "[NFC] Refactor DiagnosticRenderer to use FullSourceLoc"

2017-06-19 Thread Christof Douma via cfe-commits
Author: christof Date: Mon Jun 19 07:41:22 2017 New Revision: 305688 URL: http://llvm.org/viewvc/llvm-project?rev=305688=rev Log: Revert "[NFC] Refactor DiagnosticRenderer to use FullSourceLoc" This reverts commit 305684. This patch breaks extra/tools/clang-tidy Modified:

r305684 - [NFC] Refactor DiagnosticRenderer to use FullSourceLoc

2017-06-19 Thread Christof Douma via cfe-commits
Author: christof Date: Mon Jun 19 07:05:58 2017 New Revision: 305684 URL: http://llvm.org/viewvc/llvm-project?rev=305684=rev Log: [NFC] Refactor DiagnosticRenderer to use FullSourceLoc Move the DiagnosticRenderer and its dependents to using FullSourceLocs instead of a SourceLocation and

r296454 - [ARM] Don't pass -arm-execute-only to cc1as

2017-02-28 Thread Christof Douma via cfe-commits
Author: christof Date: Tue Feb 28 03:09:53 2017 New Revision: 296454 URL: http://llvm.org/viewvc/llvm-project?rev=296454=rev Log: [ARM] Don't pass -arm-execute-only to cc1as The option -mexecute-only is translated into the backend option -arm-execute-only. But this option only makes sense for

Re: [PATCH] D21117: Specify target in lifetime-asan test.

2016-06-08 Thread Christof Douma via cfe-commits
christof accepted this revision. christof added a reviewer: christof. christof added a comment. I've tested it and it removes the issue I saw. The following error is gone: clang-3.9: error: unsupported option '-fsanitize=address' for target 'aarch64-arm-none-eabi’ Thanks

Re: [PATCH] D21082: Do not assume that -fsanitize=address is valid option in clang tests

2016-06-08 Thread Christof Douma via cfe-commits
christof added a comment. In http://reviews.llvm.org/D21082#451393, @kubabrecka wrote: > This doesn’t make sense to me, Clang is able to produce ASanified code even > when the compiler itself isn’t ASanified (that’s what LLVM_USE_SANITIZER > does). Where exactly is this test failing? Right.

[PATCH] D21082: Do not assume that -fsanitize=address is valid option in clang tests

2016-06-07 Thread Christof Douma via cfe-commits
christof created this revision. christof added reviewers: vitalybuka, chapuni. christof added a subscriber: cfe-commits. christof set the repository for this revision to rL LLVM. The asan tests should only run on builds that have LLVM_USE_SANITIZER set. The feature 'asan' reflects this and is

Re: [PATCH] D15664: Teaches clang about Exynos-M1

2015-12-21 Thread Christof Douma via cfe-commits
christof added a comment. LGTM. But since this is my first review, I would like somebody else to sign off on this as well. http://reviews.llvm.org/D15664 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D15142: Teaches clang about Cortex-A35.

2015-12-02 Thread Christof Douma via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL254505: Teaches clang about Cortex-A35. (authored by christof). Changed prior to commit: http://reviews.llvm.org/D15142?vs=41600=41609#toc Repository: rL LLVM http://reviews.llvm.org/D15142 Files:

r254505 - Teaches clang about Cortex-A35.

2015-12-02 Thread Christof Douma via cfe-commits
Author: christof Date: Wed Dec 2 06:03:42 2015 New Revision: 254505 URL: http://llvm.org/viewvc/llvm-project?rev=254505=rev Log: Teaches clang about Cortex-A35. Adds support for the new Cortex-A35 ARMv8-A core. Differential Revision: http://reviews.llvm.org/D15142 Modified: