Author: dpolukhin
Date: Tue May 24 01:37:14 2016
New Revision: 270535
URL: http://llvm.org/viewvc/llvm-project?rev=270535&view=rev
Log:
[MSVC2015] dllexport for defaulted special class members
Clang doesn't dllexport defaulted special member function defaulted
inside class but does it if they def
This revision was automatically updated to reflect the committed changes.
Closed by commit rL270535: [MSVC2015] dllexport for defaulted special class
members (authored by dpolukhin).
Changed prior to commit:
http://reviews.llvm.org/D20422?vs=58088&id=58197#toc
Repository:
rL LLVM
http://rev
madsravn added a comment.
@alexfh I don't know how I could miss that. But I got my commit access and
committed the code myself. Thanks though.
@prazek Yes I reverted. The code made the build server (as seen on IRC) fail.
So I quickly reverted. I'm gonna fix the code tonight - I had to "make cle
@Nico, Yes, I will. I'm sorry about that. I had mistakenly read that it
would take the title and commit message from phabricator if I linked to
that in my svn commit message.
@Piotr, A test failed to build on the build server (as shown on IRC), so I
quickly reverted the commit. I will remember bet
craig.topper added inline comments.
Comment at: lib/Target/X86/X86IntrinsicsInfo.h:279
@@ -278,3 +278,1 @@
X86_INTRINSIC_DATA(avx512_scattersiv8_si, SCATTER, X86::VPSCATTERDDZ256mr,
0),
- X86_INTRINSIC_DATA(avx512_storent_pd_512, STOREANT, ISD::DELETED_NODE, 0),
- X86_INTRIN
craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.
LGTM
http://reviews.llvm.org/D20358
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/lis
Author: compnerd
Date: Mon May 23 22:21:01 2016
New Revision: 270528
URL: http://llvm.org/viewvc/llvm-project?rev=270528&view=rev
Log:
CodeGen: indicate to the backend the exception model
Thread through -fsjlj-exceptions to the backend via the TargetOptions. This is
in preparation for supporting
Revision r211132 was supposed to disable -Warc-repeated-use-of-weak for
Objective-C properties marked with the IBOutlet attribute. Those properties are
supposed to be weak but they are only accessed from the main thread so there is
no risk of asynchronous updates setting them to nil. That combin
flx added a comment.
In http://reviews.llvm.org/D20010#427510, @alexfh wrote:
> How many more (in relative numbers) results does this check generate now?
147% more :)
http://reviews.llvm.org/D20010
___
cfe-commits mailing list
cfe-commits@lists.l
Author: mps
Date: Mon May 23 20:47:41 2016
New Revision: 270521
URL: http://llvm.org/viewvc/llvm-project?rev=270521&view=rev
Log:
Visualize ellipses in TemplateTypeParm and TemplateTypeParmDecl
Now a TemplateTypeParm will be visualized as typename ...T if it is a pack
Modified:
cfe/trunk/uti
flx removed rL LLVM as the repository for this revision.
flx updated this revision to Diff 58181.
flx marked an inline comment as done.
http://reviews.llvm.org/D20010
Files:
clang-tidy/performance/UnnecessaryCopyInitialization.cpp
clang-tidy/performance/UnnecessaryCopyInitialization.h
test/
Author: compnerd
Date: Mon May 23 20:23:24 2016
New Revision: 270520
URL: http://llvm.org/viewvc/llvm-project?rev=270520&view=rev
Log:
clang-c: de-anonymize structure declaration
The statement constructed an anonymous structure which was typedefed. The
anonymous structure has internal linkage, a
rsmith added inline comments.
Comment at: include/clang/Sema/Sema.h:1534
@@ +1533,3 @@
+ /// type name has failed in a dependent context. In these situations, we
+ /// automatically form DependentTypeName that will retry lookup in a related
+ /// scope during instantiation.
---
Author: rsmith
Date: Mon May 23 19:01:49 2016
New Revision: 270514
URL: http://llvm.org/viewvc/llvm-project?rev=270514&view=rev
Log:
Properly track the found declaration (possibly a using-declaration) when
handling an explicit member specialization.
Modified:
cfe/trunk/lib/Sema/SemaTemplate.c
hintonda added a comment.
Actually, this will never work correctly -- in fact, raw lexing will always be
problematic. Consider:
void foo()
#if !__has_feature(cxx_noexcept)
throw(std::bad_alloc)
#endif
{}
In this case we *could* figure out if __has_feature(cxx_noexcept) evaluated to
true o
rnk updated this revision to Diff 58169.
rnk marked an inline comment as done.
rnk added a comment.
- Share the MSVC compatibility hack between the inside a method case and the
default type template argument case
http://reviews.llvm.org/D20500
Files:
include/clang/Sema/Sema.h
lib/Parse/Par
Hello everyone,
Below are some buildbot numbers for the last week of 5/15/2016 - 5/21/2016.
Thanks
Galina
"Status change ratio" by active builder (percent of builds that changed the
builder status from greed to red or from red to green):
buildername
Ping.
On Tue, Apr 5, 2016 at 11:46 AM NAKAMURA Takumi
wrote:
> Tom, may I commit it into release_38?
> It has been preventing "lit --use-processes" on windows.
>
> 2016-02-16 17:13 GMT+09:00 NAKAMURA Takumi via cfe-commits
> :
> > Author: chapuni
> > Date: Tue Feb 16 02:13:36 2016
> > New Revisi
Prazek added a comment.
Did you revert the commit? I see that it is commieted, but after it I see
revert.
Also please stick to convention of commit messages
http://llvm.org/docs/DeveloperPolicy.html#commit-messages
Commit message like "[clang-tidy] modernize-pass-by-value bugfix" would be much
alexfh added a comment.
In http://reviews.llvm.org/D20365#436335, @madsravn wrote:
> Just curious, as I'm sort of new to this. How long will it take before its
> merged in?
I was waiting for an answer to the "Do you need me to submit the patch for
you?" question. Apparently, the answer is "no
Author: mps
Date: Mon May 23 17:27:44 2016
New Revision: 270505
URL: http://llvm.org/viewvc/llvm-project?rev=270505&view=rev
Log:
Visualizer for Pack template arguments
Modified:
cfe/trunk/utils/ClangVisualizers/clang.natvis
Modified: cfe/trunk/utils/ClangVisualizers/clang.natvis
URL:
http
alexfh added a comment.
Richard, ping.
http://reviews.llvm.org/D20428
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
This revision was automatically updated to reflect the committed changes.
Closed by commit rL270499: [X86][SSE] Replace lossless i32/f32 to f64
conversion intrinsics with generic IR (authored by RKSimon).
Changed prior to commit:
http://reviews.llvm.org/D20528?vs=58146&id=58160#toc
Repository:
Author: rksimon
Date: Mon May 23 17:13:02 2016
New Revision: 270499
URL: http://llvm.org/viewvc/llvm-project?rev=270499&view=rev
Log:
[X86][SSE] Replace lossless i32/f32 to f64 conversion intrinsics with generic IR
Both the (V)CVTDQ2PD(Y) (i32 to f64) and (V)CVTPS2PD(Y) (f32 to f64) conversion
i
mkuper accepted this revision.
mkuper added a comment.
This revision is now accepted and ready to land.
> The existing llvm\test\CodeGen\X86\vec_fpext.ll and
> llvm\test\CodeGen\X86\vec_int_to_fp.ll already demonstrate the correct
> optimized DAG codegen using the same IR as output in the
> cla
RKSimon added a comment.
In http://reviews.llvm.org/D20528#437165, @mkuper wrote:
> Presumably, the fast-isel lowering of the IR pattern is already correct, and
> in any case, it isn't affected by this patch.
> I just want to make sure we don't regress the optimized DAG codegen - that
> is, it
thakis added a comment.
Hm, the ASTReader code this works around is over 6 years old (r100866). Maybe
we could try enabling the access time check instead?
http://reviews.llvm.org/D20243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http:/
rmaprath added a comment.
In http://reviews.llvm.org/D20119#436849, @jroelofs wrote:
> In http://reviews.llvm.org/D20119#431997, @rmaprath wrote:
>
> > Addressing review comments from @jroelofs:
> >
> > - Moved the assertion in `libunwind.cpp` back to `UnwindCursor.cpp` where
> > it really belog
hintonda added a comment.
In http://reviews.llvm.org/D18575#435388, @alexfh wrote:
> Let's wait for http://reviews.llvm.org/D20428
No worries.
http://reviews.llvm.org/D18575
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llv
hintonda updated this revision to Diff 58151.
hintonda added a comment.
Fixed matcher -- added 'unless(isImplicit())'. Thanks to Aaron Ballman for the
suggestion.
http://reviews.llvm.org/D18575
Files:
clang-tidy/modernize/CMakeLists.txt
clang-tidy/modernize/ModernizeTidyModule.cpp
clang
Ping on the testcase :-)
On Tue, May 17, 2016 at 9:36 AM, Nico Weber via cfe-commits
wrote:
> Is it possible to write a test for this?
>
> On Tue, May 17, 2016 at 10:34 AM, Cameron Desrochers via cfe-commits
> wrote:
>>
>> Author: cameron314
>> Date: Tue May 17 09:34:53 2016
>> New Revision: 269
ariccio added a comment.
I should elaborate. The principle of operation of this latest patch is that the
`FunctionDecl` in `IsCMemFunction` should never return a `nullptr`
`IdentifierInfo*` from `getIdentifier` (is that a valid assumption?)... Thus,
when`! isWindowsMSVCEnvironment`, I leave the
mkuper added a comment.
Presumably, the fast-isel lowering of the IR pattern is already correct, and in
any case, it isn't affected by this patch.
I just want to make sure we don't regress the optimized DAG codegen - that is,
it still produces the instruction we'd expect from the intrinsic (or s
BTW why did you revert this change? And why the commit message doesn't have
"revert" in name?
2016-05-23 20:51 GMT+02:00 Nico Weber via cfe-commits <
cfe-commits@lists.llvm.org>:
> Next time, please use real commit messages: Describe what the change does,
> and why it's being done. Include a link
Prazek updated this revision to Diff 58148.
Prazek added a comment.
+Fixed bug with operators
+ added fixup for function return type
I will post changes on clang tomorrow
http://reviews.llvm.org/D18821
Files:
clang-tidy/CMakeLists.txt
clang-tidy/bugprone/BoolToIntegerConversionCheck.cpp
bruno added a subscriber: bruno.
bruno added a comment.
Hi Pierre,
Comment at: test/PCH/pragma-once-timestamp.cpp:17
@@ +16,3 @@
+
+// REQUIRES: system-windows
+
Can you move this to the beginning of the file? It makes it easier to spot that
this is windows onl
RKSimon added a comment.
In http://reviews.llvm.org/D20528#437117, @mkuper wrote:
> Sorry, I didn't intend to imply the rest of the llvm work is necessary for
> this to go in. Just that I'd be happier with this patch knowing that we have
> a regression test for doing the (shuffle + fpext, say)
bruno added a subscriber: bruno.
bruno added a comment.
Hi Cameron,
Can you add a testcase?
Repository:
rL LLVM
http://reviews.llvm.org/D20338
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo
rsmith added a comment.
It sounds like we're failing to properly inherit default template arguments
onto redeclarations generated by template instantiation, resulting in a
violation of our AST invariants. This patch will hide the problem in some
cases, but the right thing to do is to fix the ro
ahatanak updated this revision to Diff 58144.
ahatanak added a comment.
Rewrote the patch based on John's review comment.
Remove typedefs and _Atomic from the return and parameter types of getters and
setters of objective-c properties.
http://reviews.llvm.org/D20407
Files:
include/clang/AST
RKSimon updated this revision to Diff 58146.
Repository:
rL LLVM
http://reviews.llvm.org/D20528
Files:
include/clang/Basic/BuiltinsX86.def
lib/Headers/avxintrin.h
lib/Headers/emmintrin.h
test/CodeGen/avx-builtins.c
test/CodeGen/builtins-x86.c
test/CodeGen/sse2-builtins.c
test/Cod
This revision was automatically updated to reflect the committed changes.
Closed by commit rL270484: [CUDA] Add -fcuda-approx-transcendentals flag.
(authored by jlebar).
Changed prior to commit:
http://reviews.llvm.org/D20493?vs=58123&id=58145#toc
Repository:
rL LLVM
http://reviews.llvm.org
Author: jlebar
Date: Mon May 23 15:19:56 2016
New Revision: 270484
URL: http://llvm.org/viewvc/llvm-project?rev=270484&view=rev
Log:
[CUDA] Add -fcuda-approx-transcendentals flag.
Summary:
This lets us emit e.g. sin.approx.f32. See
http://docs.nvidia.com/cuda/parallel-thread-execution/#floating-
mkuper added a comment.
In http://reviews.llvm.org/D20528#437090, @RKSimon wrote:
> In http://reviews.llvm.org/D20528#436893, @mkuper wrote:
>
> > This looks right, but we may lose some end-to-end tests, since right now we
> > have a clang-level test that checks the builtin is lowered to the
>
erik.pilkington added a comment.
Ping!!
http://reviews.llvm.org/D20192
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: rsmith
Date: Mon May 23 15:03:04 2016
New Revision: 270482
URL: http://llvm.org/viewvc/llvm-project?rev=270482&view=rev
Log:
Fix filtering of prior declarations when checking for a tag redeclaration to
map to the redecl context for both decls, not just one of them, and to properly
check th
RKSimon added a comment.
In http://reviews.llvm.org/D20528#436893, @mkuper wrote:
> This looks right, but we may lose some end-to-end tests, since right now we
> have a clang-level test that checks the builtin is lowered to the intrinsic,
> and (hopefully) a CG-level test that the intrinsic is
vmiklos created this revision.
vmiklos added reviewers: klimek, cfe-commits.
The second check failed, FOO(C::X) wasn't renamed to FOO(C::Y).
http://reviews.llvm.org/D20537
Files:
clang-rename/USRLocFinder.cpp
test/clang-rename/DeclRefExpr.cpp
Index: test/clang-rename/DeclRefExpr.cpp
===
rsmith added inline comments.
Comment at: include/clang/Sema/Sema.h:1536-1537
@@ -1535,3 +1535,4 @@
/// other template arguments.
ParsedType ActOnDelayedDefaultTemplateArg(const IdentifierInfo &II,
SourceLocation NameLoc);
+ Pars
jvesely retitled this revision from "[OpenCL] cl_khr_msaa_sharing is OpenCL1.2
extension" to "[OpenCL] Fixup extension list".
jvesely added a subscriber: cfe-commits.
jvesely updated this revision to Diff 58130.
jvesely added a comment.
I went through the specs and fixed up all I could find.
adde
bruno added a comment.
Ping!
http://reviews.llvm.org/D20451
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Next time, please use real commit messages: Describe what the change does,
and why it's being done. Include a link to the review link at the end of
the commit message. If every change just had a phab link as commit message,
people bisecting changes would have to click through for every change in
`s
bruno added a comment.
Ping!
http://reviews.llvm.org/D20404
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tra added a comment.
LGTM.
http://reviews.llvm.org/D20493
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: madsravn
Date: Mon May 23 13:27:05 2016
New Revision: 270473
URL: http://llvm.org/viewvc/llvm-project?rev=270473&view=rev
Log:
Commiting for http://reviews.llvm.org/D20365
Modified:
clang-tools-extra/trunk/clang-tidy/modernize/PassByValueCheck.cpp
clang-tools-extra/trunk/test/clan
jlebar updated this revision to Diff 58123.
jlebar added a comment.
More tightly scope the __USE_FAST_MATH__ macro.
tra pointed out that device_functions.hpp uses __USE_FAST_MATH__ for its own
purposes. For this CL, we only want to define __USE_FAST_MATH__ around
math_functions.hpp.
http://rev
Author: madsravn
Date: Mon May 23 13:15:40 2016
New Revision: 270472
URL: http://llvm.org/viewvc/llvm-project?rev=270472&view=rev
Log:
Commiting for http://reviews.llvm.org/D20365
Modified:
clang-tools-extra/trunk/clang-tidy/modernize/PassByValueCheck.cpp
clang-tools-extra/trunk/test/clan
madsravn closed this revision.
madsravn added a comment.
Code committed.
http://reviews.llvm.org/D20365
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
benlangmuir added a subscriber: doug.gregor.
benlangmuir added a comment.
I'd like to see Doug and/or Richard review this. It seems reasonable to me to
first blush, but I assume there was a good reason we weren't doing this
already...
http://reviews.llvm.org/D20383
Author: madsravn
Date: Mon May 23 13:06:29 2016
New Revision: 270470
URL: http://llvm.org/viewvc/llvm-project?rev=270470&view=rev
Log:
Commiting for http://reviews.llvm.org/D20365
Modified:
clang-tools-extra/trunk/clang-tidy/misc/MacroParenthesesCheck.cpp
clang-tools-extra/trunk/test/clan
rnk updated this revision to Diff 58119.
rnk added a comment.
- Recover in ParseImplicitInt instead
http://reviews.llvm.org/D20500
Files:
include/clang/Sema/Sema.h
lib/Parse/ParseDecl.cpp
lib/Sema/SemaDecl.cpp
test/SemaTemplate/ms-delayed-default-template-args.cpp
test/SemaTemplate/ms
rnk marked an inline comment as done.
Comment at: lib/Parse/ParseDecl.cpp:2282
@@ -2281,3 +2281,3 @@
// Otherwise, if we don't consume this token, we are going to emit an
// error anyway. Try to recover from various common problems. Check
rsmith wrote:
>
On Mon, May 23, 2016 at 1:36 PM, David Majnemer
wrote:
>
>
> On Mon, May 23, 2016 at 10:28 AM, Aaron Ballman
> wrote:
>>
>> On Mon, May 23, 2016 at 1:16 PM, David Majnemer via cfe-commits
>> wrote:
>> > Author: majnemer
>> > Date: Mon May 23 12:16:12 2016
>> > New Revision: 270457
>> >
>> > URL:
Author: majnemer
Date: Mon May 23 12:32:35 2016
New Revision: 270461
URL: http://llvm.org/viewvc/llvm-project?rev=270461&view=rev
Log:
Address post-commit review feedback to r270457
Add two tests which show our error handling behavior for invalid
parameters in the layout_version and empty_bases a
On Mon, May 23, 2016 at 10:28 AM, Aaron Ballman
wrote:
> On Mon, May 23, 2016 at 1:16 PM, David Majnemer via cfe-commits
> wrote:
> > Author: majnemer
> > Date: Mon May 23 12:16:12 2016
> > New Revision: 270457
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=270457&view=rev
> > Log:
> > [MS
This revision was automatically updated to reflect the committed changes.
Closed by commit rL270458: Clang support for __is_assignable intrinsic
(authored by majnemer).
Changed prior to commit:
http://reviews.llvm.org/D20492?vs=58014&id=58112#toc
Repository:
rL LLVM
http://reviews.llvm.org/
Author: majnemer
Date: Mon May 23 12:21:55 2016
New Revision: 270458
URL: http://llvm.org/viewvc/llvm-project?rev=270458&view=rev
Log:
Clang support for __is_assignable intrinsic
MSVC now supports the __is_assignable type trait intrinsic,
to enable easier and more efficient implementation of the
On Mon, May 23, 2016 at 1:16 PM, David Majnemer via cfe-commits
wrote:
> Author: majnemer
> Date: Mon May 23 12:16:12 2016
> New Revision: 270457
>
> URL: http://llvm.org/viewvc/llvm-project?rev=270457&view=rev
> Log:
> [MS ABI] Implement __declspec(empty_bases) and __declspec(layout_version)
>
>
mkuper added a comment.
Thanks, Simon!
This looks right, but we may lose some end-to-end tests, since right now we
have a clang-level test that checks the builtin is lowered to the intrinsic,
and (hopefully) a CG-level test that the intrinsic is lowered to the correct
instruction.
Do you know i
Author: majnemer
Date: Mon May 23 12:16:12 2016
New Revision: 270457
URL: http://llvm.org/viewvc/llvm-project?rev=270457&view=rev
Log:
[MS ABI] Implement __declspec(empty_bases) and __declspec(layout_version)
The layout_version attribute is pretty straightforward: use the layout
rules from versio
hokein added inline comments.
Comment at: clang-tidy/llvm/HeaderGuardCheck.h:19
@@ -18,3 +18,3 @@
/// Finds and fixes header guards that do not adhere to LLVM style.
class LLVMHeaderGuardCheck : public utils::HeaderGuardCheck {
madsravn wrote:
> hokein wrote:
Anastasia accepted this revision.
Anastasia added a comment.
This revision is now accepted and ready to land.
Do we have any test for this? Would be nice to add one if possible.
http://reviews.llvm.org/D20373
___
cfe-commits mailing list
cfe-commits
Anastasia added inline comments.
Comment at: lib/CodeGen/CGExprScalar.cpp:3403
@@ +3402,3 @@
+static Value *ConvertVec3AndVec4(CGBuilderTy &Builder, CodeGenFunction &CGF,
+ Value *Src, unsigned numElementsDst) {
+ llvm::Value *UnV = llvm::UndefValue::get(Src->getType());
---
jroelofs added a comment.
In http://reviews.llvm.org/D20119#431997, @rmaprath wrote:
> Addressing review comments from @jroelofs:
>
> - Moved the assertion in `libunwind.cpp` back to `UnwindCursor.cpp` where it
> really belogs.
>
> @jroelofs: I just realized that, with this new native-only bui
rmaprath added a comment.
@jroelofs: OK to commit?
Thanks.
/ Asiri
http://reviews.llvm.org/D20119
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rmaprath added a comment.
@mclow.lists, @EricWF: Any comments on this? If it helps, I could split off the
library source changes and test setup into two separate patches. I thought to
keep the two together as the test setup gives an idea of the overall objective.
Thanks.
/ Asiri
http://revie
RKSimon created this revision.
RKSimon added reviewers: qcolombet, craig.topper, mkuper, andreadb, spatel.
RKSimon added a subscriber: cfe-commits.
RKSimon set the repository for this revision to rL LLVM.
Both the (V)CVTDQ2PD(Y) (i32 to f64) and (V)CVTPS2PD(Y) (f32 to f64) conversion
instructions
madsravn added inline comments.
Comment at: clang-tidy/llvm/HeaderGuardCheck.h:19
@@ -18,3 +18,3 @@
/// Finds and fixes header guards that do not adhere to LLVM style.
class LLVMHeaderGuardCheck : public utils::HeaderGuardCheck {
hokein wrote:
> You should add
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
http://reviews.llvm.org/D18953
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
Thanks!
http://reviews.llvm.org/D20422
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe
kzhuravl added a comment.
Ping
http://reviews.llvm.org/D20336
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sbenza added inline comments.
Comment at: include/clang/AST/ASTContext.h:824
@@ -823,1 +823,3 @@
unsigned overridden_methods_size(const CXXMethodDecl *Method) const;
+ typedef llvm::iterator_range
+ overridden_method_range;
Sure. Sorry about that.
The mai
Author: mzuckerm
Date: Mon May 23 10:04:39 2016
New Revision: 270441
URL: http://llvm.org/viewvc/llvm-project?rev=270441&view=rev
Log:
[clang][AVX512][Builtin] adding missing intrinsics for
vpmultishiftqb{128|256|512} instruction set .
Differential Revision: http://reviews.llvm.org/D20521
Modi
This revision was automatically updated to reflect the committed changes.
Closed by commit rL270441: [clang][AVX512][Builtin] adding missing intrinsics
for… (authored by mzuckerm).
Changed prior to commit:
http://reviews.llvm.org/D20521?vs=58086&id=58098#toc
Repository:
rL LLVM
http://revie
flx added a comment.
In http://reviews.llvm.org/D20277#436717, @Prazek wrote:
> Cool check! Did you think about sugesting std::move for rvalue references if
> they are used once?
Thanks! I'm not sure this fits with what a user would expect from a check named
"unnecessary-value-param" since in
Prazek added a subscriber: Prazek.
Prazek added a comment.
Cool check! Did you think about sugesting std::move for rvalue references if
they are used once?
http://reviews.llvm.org/D20277
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http:
m_zuckerman created this revision.
m_zuckerman added reviewers: AsafBadouh, igorb, delena.
m_zuckerman added a subscriber: cfe-commits.
http://reviews.llvm.org/D20523
Files:
lib/Headers/avx512fintrin.h
test/CodeGen/avx512f-builtins.c
Index: test/CodeGen/avx512f-builtins.c
===
hokein added inline comments.
Comment at: include-fixer/find-stl-symbols/STLSymbolsFinder.h:25
@@ +24,3 @@
+///
+/// To decide which symbols can be #include'd from each headear, we run
+/// FindSTLSymbolsAction on each header to retrieve all symbols in the
s/heade
hokein added a subscriber: hokein.
Comment at: clang-tidy/llvm/HeaderGuardCheck.h:19
@@ -18,3 +18,3 @@
/// Finds and fixes header guards that do not adhere to LLVM style.
class LLVMHeaderGuardCheck : public utils::HeaderGuardCheck {
You should add a document f
bkramer added inline comments.
Comment at: include-fixer/find-all-symbols/FindAllSymbols.h:47
@@ +46,3 @@
+protected:
+ // The way SymbolInfo of a decl is reported can be overrided.
+ virtual void reportDecl(const SourceManager &SM, const clang::NamedDecl *ND,
o
flx updated this revision to Diff 58093.
flx added a comment.
Upated documentation as well.
http://reviews.llvm.org/D20277
Files:
clang-tidy/performance/UnnecessaryValueParamCheck.cpp
clang-tidy/performance/UnnecessaryValueParamCheck.h
clang-tidy/utils/CMakeLists.txt
clang-tidy/utils/De
flx added inline comments.
Comment at: clang-tidy/utils/TypeTraits.cpp:131
@@ +130,3 @@
+ return Record->hasNonTrivialMoveConstructor();
+}
+
Done. Shortened this now to just check whether the type has a non-trivial move
constructor or assignment operator. For t
flx updated this revision to Diff 58090.
http://reviews.llvm.org/D20277
Files:
clang-tidy/performance/UnnecessaryValueParamCheck.cpp
clang-tidy/performance/UnnecessaryValueParamCheck.h
clang-tidy/utils/CMakeLists.txt
clang-tidy/utils/DeclRefExprUtils.cpp
clang-tidy/utils/DeclRefExprUtil
hokein added inline comments.
Comment at: clang-tidy/misc/UnusedUsingDeclsCheck.cpp:47
@@ +46,3 @@
+ return;
+// Ignores using-declarations defined in function definitions to avoid
+// arguement-dependent lookup.
alexfh wrote:
> I don't think using de
DmitryPolukhin added a comment.
PTAL
Comment at: lib/Sema/SemaDeclCXX.cpp:13113-13114
@@ -13065,28 +13112,4 @@
-if (MD->isInvalidDecl())
- return;
-
-switch (Member) {
-case CXXDefaultConstructor:
- DefineImplicitDefaultConstructor(DefaultLoc,
-
DmitryPolukhin updated this revision to Diff 58088.
DmitryPolukhin added a comment.
- drop dllexport for trivial defaulted x-tors for compatibility with MSVC2015
- move checks to CheckCompletedCXXClass because completed class definition is
required to detect if x-tor is trivial
http://reviews.l
AsafBadouh accepted this revision.
AsafBadouh added a comment.
This revision is now accepted and ready to land.
LGTM
http://reviews.llvm.org/D20521
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
Author: jacob_hansen
Date: Mon May 23 08:12:29 2016
New Revision: 270435
URL: http://llvm.org/viewvc/llvm-project?rev=270435&view=rev
Log:
Test commit
Modified:
cfe/trunk/lib/Driver/Tools.cpp
Modified: cfe/trunk/lib/Driver/Tools.cpp
URL:
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Dri
m_zuckerman created this revision.
m_zuckerman added reviewers: AsafBadouh, igorb, delena.
m_zuckerman added a subscriber: cfe-commits.
http://reviews.llvm.org/D20521
Files:
include/clang/Basic/BuiltinsX86.def
lib/Headers/avx512vbmiintrin.h
lib/Headers/avx512vbmivlintrin.h
test/CodeGen/av
alexfh added inline comments.
Comment at: clang-tidy/misc/UnusedUsingDeclsCheck.cpp:47
@@ +46,3 @@
+ return;
+// Ignores using-declarations defined in function definitions to avoid
+// arguement-dependent lookup.
I don't think using declarations in fu
1 - 100 of 116 matches
Mail list logo
