Author: hiraditya
Date: Sun Oct 2 21:36:33 2016
New Revision: 283085
URL: http://llvm.org/viewvc/llvm-project?rev=283085&view=rev
Log:
Keep the test only for Itanium abi
Modified:
cfe/trunk/test/CodeGenCXX/alias-available-externally.cpp
Modified: cfe/trunk/test/CodeGenCXX/alias-available-ex
echristo accepted this revision.
echristo added a comment.
This revision is now accepted and ready to land.
Going to accept this pending the backend patch, but when that one is applied I
wanted you to feel OK to add this. A couple of inline nitpick comments and some
agreement that we should do s
rsmmr created this revision.
rsmmr added reviewers: djasper, lodato.
rsmmr added a subscriber: cfe-commits.
Herald added a subscriber: klimek.
This patch adds two new options, feedback welcome.
SpacesAroundConditions (bool)
If true, spaces will be inserted around if/for/while conditions.
Spac
vleschuk marked 3 inline comments as done.
vleschuk added inline comments.
> vleschuk wrote in CGDebugInfo.cpp:608
> Will check if this works in all cases. I think it's worth putting this
> snippet into helper function within anon namespace.
That is correct for types, for particular decls we st
vleschuk updated this revision to Diff 73228.
vleschuk added a comment.
- Move alignment detection into helper functions in anon namespace
- Use updated LLVM DIBuilder API (from https://reviews.llvm.org/D24425)
https://reviews.llvm.org/D24426
Files:
lib/CodeGen/CGDebugInfo.cpp
lib/CodeGen/C
elsteveogrande updated this revision to Diff 73224.
elsteveogrande added a comment.
Updated to actually use the include token's text, not a hardcoded `#include`.
Updated unit test to expect the right word here.
https://reviews.llvm.org/D25153
Files:
include/clang/Driver/Options.td
include
jlebar created this revision.
jlebar added a reviewer: tra.
jlebar added a subscriber: cfe-commits.
This prevents clang from emitting 'invoke's and catch statements.
Things previously mostly worked thanks to TryToMarkNoThrow() in
CodeGenFunction. But this is not a proper IPO, and it doesn't prop
mgorny added a comment.
Thanks for the review. Proceeding with the commit now.
https://reviews.llvm.org/D25161
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: mgorny
Date: Sun Oct 2 14:28:57 2016
New Revision: 283075
URL: http://llvm.org/viewvc/llvm-project?rev=283075&view=rev
Log:
[cmake] Install 'clang-cpp' symlink
Install the 'clang-cpp' symlink used to spawn the preprocessor. The code
handling this suffix is already included in Driver. Fre
Author: ctopper
Date: Sun Oct 2 12:43:00 2016
New Revision: 283073
URL: http://llvm.org/viewvc/llvm-project?rev=283073&view=rev
Log:
[AVX-512] Use native IR for masked 512-bit add/sub/mul/div ps/pd intrinsics
when rounding mode isn't used.
Modified:
cfe/trunk/lib/Headers/avx512fintrin.h
joerg accepted this revision.
joerg added a reviewer: joerg.
joerg added a comment.
This revision is now accepted and ready to land.
LGTM, have been doing that in NetBSD for ages as well.
Side note, someone from Apple please make sure that xcode gets a similar link.
https://reviews.llvm.org/D25
Anastasia accepted this revision.
Anastasia added a comment.
This revision is now accepted and ready to land.
LGTM! Thanks!
https://reviews.llvm.org/D25123
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman
The mangling is different when targeting MSVC ABI, see failure at
http://bb.pgr.jp/builders/ninja-x64-msvc-RA-centos6/builds/29889/steps/test_all/logs/Clang%20%3A%3A%20CodeGenCXX__alias-available-externally.cpp
maybe -triple x86_64-pc-linux ?
2016-10-02 6:06 GMT+03:00 Aditya Kumar via cfe-com
mgorny requested changes to this revision.
mgorny added a reviewer: mgorny.
mgorny added a comment.
This revision now requires changes to proceed.
Few minor nits. However, I'd like to say that I like the idea in general and
see forward to deploy it in Gentoo.
One use case which doesn't seem to h
aaron.ballman added inline comments.
> UseTransparentFunctorsCheck.cpp:26
> + unless(hasAnyTemplateArgument(refersToType(voidType(,
> + hasAnyName("::std::plus", "::std::minus", "::std::multiplies",
> + "::std::divides", "::std::modulus", "::std::negate",
Should we
elsteveogrande added a comment.
Added a couple of notes
> PrintPreprocessedOutput.cpp:241
>}
> -
>return false;
Re-reading this diff, I saw hat my editor trimmed EOL whitespace automatically
and I let these minor changes become part of the patch. Sorry for the noise :)
> PrintPrep
Author: jlebar
Date: Sun Oct 2 10:24:50 2016
New Revision: 283068
URL: http://llvm.org/viewvc/llvm-project?rev=283068&view=rev
Log:
[CUDA] Allow extern __shared__ on empty-length arrays.
"extern __shared__ int x[]" is OK.
Modified:
cfe/trunk/lib/Sema/SemaDeclAttr.cpp
cfe/trunk/test/Sema
Author: omtcyfz
Date: Sun Oct 2 09:51:33 2016
New Revision: 283067
URL: http://llvm.org/viewvc/llvm-project?rev=283067&view=rev
Log:
[clang-rename] Overhaul clang-rename.el.
* Use lexical binding, as recommended for new libraries.
* Fix customization variable (set correct group and type).
* Crea
mgorny planned changes to this revision.
mgorny added a comment.
I will be reworking this to use CMake's PRIVATE and PUBLIC linked libs
consistently across the build, linker scripts and possibly tests.
@EricWF, would you prefer if I killed the `add_library_flags*` macros or
extended them to sup
ioeric created this revision.
ioeric added a reviewer: djasper.
ioeric added a subscriber: cfe-commits.
Herald added a subscriber: klimek.
https://reviews.llvm.org/D25162
Files:
lib/Format/Format.cpp
Index: lib/Format/Format.cpp
mgorny created this revision.
mgorny added reviewers: beanz, doug.gregor.
mgorny added a subscriber: cfe-commits.
Herald added a subscriber: emaste.
Install the 'clang-cpp' symlink used to spawn the preprocessor. The code
handling this suffix is already included in Driver. FreeBSD is already crea
21 matches
Mail list logo
