r330571 - Use special new Clang flag 'FrontendTimesIsEnabled' instead of 'llvm::TimePassesIsEnabled' inside -ftime-report feature.

2018-04-23 Thread Andrew V. Tischenko via cfe-commits
Author: avt77 Date: Mon Apr 23 02:22:30 2018 New Revision: 330571 URL: http://llvm.org/viewvc/llvm-project?rev=330571=rev Log: Use special new Clang flag 'FrontendTimesIsEnabled' instead of 'llvm::TimePassesIsEnabled' inside -ftime-report feature. Differential Revision:

r329714 - I removed the failed test.

2018-04-10 Thread Andrew V. Tischenko via cfe-commits
Author: avt77 Date: Tue Apr 10 08:45:43 2018 New Revision: 329714 URL: http://llvm.org/viewvc/llvm-project?rev=329714=rev Log: I removed the failed test. Removed: cfe/trunk/test/Frontend/ftime-report-template-decl.cpp Removed: cfe/trunk/test/Frontend/ftime-report-template-decl.cpp URL:

r329693 - The test was fixed.

2018-04-10 Thread Andrew V. Tischenko via cfe-commits
Author: avt77 Date: Tue Apr 10 05:17:01 2018 New Revision: 329693 URL: http://llvm.org/viewvc/llvm-project?rev=329693=rev Log: The test was fixed. Modified: cfe/trunk/test/Frontend/ftime-report-template-decl.cpp Modified: cfe/trunk/test/Frontend/ftime-report-template-decl.cpp URL:

r329684 - -ftime-report switch support in Clang.

2018-04-10 Thread Andrew V. Tischenko via cfe-commits
Author: avt77 Date: Tue Apr 10 03:34:13 2018 New Revision: 329684 URL: http://llvm.org/viewvc/llvm-project?rev=329684=rev Log: -ftime-report switch support in Clang. The current support of the feature produces only 2 lines in report: -Some general Code Generation Time; -Total time of Backend

r327618 - More warnings when double truncation to float: compound assignment is supported now.

2018-03-15 Thread Andrew V. Tischenko via cfe-commits
Author: avt77 Date: Thu Mar 15 03:03:35 2018 New Revision: 327618 URL: http://llvm.org/viewvc/llvm-project?rev=327618=rev Log: More warnings when double truncation to float: compound assignment is supported now. Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td

r325011 - An updated test to show the current warnings produced for implicit conversions from 'double' to 'float'.

2018-02-13 Thread Andrew V. Tischenko via cfe-commits
Author: avt77 Date: Tue Feb 13 07:20:29 2018 New Revision: 325011 URL: http://llvm.org/viewvc/llvm-project?rev=325011=rev Log: An updated test to show the current warnings produced for implicit conversions from 'double' to 'float'. Modified: cfe/trunk/test/Sema/conversion.c Modified:

r324721 - Fif for an issue when Clang permits assignment to vector/extvector elements in a const method.

2018-02-09 Thread Andrew V. Tischenko via cfe-commits
Author: avt77 Date: Fri Feb 9 01:30:42 2018 New Revision: 324721 URL: http://llvm.org/viewvc/llvm-project?rev=324721=rev Log: Fif for an issue when Clang permits assignment to vector/extvector elements in a const method. Modified: cfe/trunk/lib/Sema/SemaExpr.cpp

Re: [PATCH] D18953: [ms][dll] #26935 Defining a dllimport function should cause it to be exported

2016-05-24 Thread Andrew V. Tischenko via cfe-commits
avt77 updated this revision to Diff 58233. avt77 added a comment. I built the project from scratch and found an issue in Clang build system. There was a problem with undefined DiagGroup in DiagnosticSemaKinds.td: the error message was generated but all executables were created that's why I was

Re: [PATCH] D18953: [ms][dll] #26935 Defining a dllimport function should cause it to be exported

2016-05-23 Thread Andrew V. Tischenko via cfe-commits
avt77 added a comment. OK, as I see all issues were resolved, right? Could I commit the patch? http://reviews.llvm.org/D18953 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D19479: 26748 - clang-cl fails to compile atlctrlw.h header from WTL

2016-05-20 Thread Andrew V. Tischenko via cfe-commits
avt77 added a comment. It seems I gave up with this issue :-( We have 2 possible situations here: 1. A type from instantiated type used inside the template (like in the test I'm working on) 2. A feature from instantiated type used inside the template (like below) struct A { int bar (); }

Re: [PATCH] D19156: [ms][dll] #27212: Generating of implicit special members should take into account MSVC compatibility version

2016-05-12 Thread Andrew V. Tischenko via cfe-commits
avt77 marked 2 inline comments as done. Comment at: lib/Sema/SemaDeclCXX.cpp:4813 @@ +4812,3 @@ +// and move constructor, so don't attempt to import/export them if +// we have a definition. +auto *CXXC = dyn_cast(MD); rnk wrote: > Oh, so

Re: [PATCH] D19156: [ms][dll] #27212: Generating of implicit special members should take into account MSVC compatibility version

2016-04-29 Thread Andrew V. Tischenko via cfe-commits
avt77 updated this revision to Diff 55567. avt77 added a comment. Now it's really a micro-patch: please review. http://reviews.llvm.org/D19156 Files: lib/Sema/SemaDeclCXX.cpp test/CodeGenCXX/dllexport.cpp Index: lib/Sema/SemaDeclCXX.cpp

Re: [PATCH] D18953: [ms][dll] #26935 Defining a dllimport function should cause it to be exported

2016-04-28 Thread Andrew V. Tischenko via cfe-commits
avt77 added inline comments. Comment at: test/SemaCXX/dllimport.cpp:179 @@ -140,1 +178,3 @@ +template +int ExternVarTmplDeclInit = 1; majnemer wrote: > avt77 wrote: > > rnk wrote: > > > Can you check with MSVC 2015 update 2 actually does with definitions of >

Re: [PATCH] D18953: [ms][dll] #26935 Defining a dllimport function should cause it to be exported

2016-04-26 Thread Andrew V. Tischenko via cfe-commits
avt77 added a comment. It seems the latest MSVC changed the support of several features covering in this patch: more investigations needed. Comment at: test/SemaCXX/dllimport.cpp:179 @@ -140,1 +178,3 @@ +template +int ExternVarTmplDeclInit = 1; rnk wrote: >

Re: [PATCH] D18953: [ms][dll] #26935 Defining a dllimport function should cause it to be exported

2016-04-26 Thread Andrew V. Tischenko via cfe-commits
avt77 added a comment. In fact you introduced the plan of actions to fix the issue. Tnx. http://reviews.llvm.org/D18953 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D19156: [ms][dll] #27212: Generating of implicit special members should take into account MSVC compatibility version

2016-04-26 Thread Andrew V. Tischenko via cfe-commits
avt77 added a comment. It seems it will be even shorter if we do it via Sema::checkClassLevelDLLAttribute. Tnx. http://reviews.llvm.org/D19156 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D19479: 26748 - clang-cl fails to compile atlctrlw.h header from WTL

2016-04-26 Thread Andrew V. Tischenko via cfe-commits
avt77 added a comment. OK, I'll try to implement something like a DependentScopeDeclRefExpr but because of the latest changes in Intel I'm afraid it will take some more time from me. http://reviews.llvm.org/D19479 ___ cfe-commits mailing list

[PATCH] D19479: 26748 - clang-cl fails to compile atlctrlw.h header from WTL

2016-04-25 Thread Andrew V. Tischenko via cfe-commits
avt77 created this revision. avt77 added a reviewer: rnk. avt77 added a subscriber: cfe-commits. This is the first patch to fix clang-cl incompatibility prohibited to compile the header file. The patch covers the following case: class CCommandBarCtrlBase { public: typedef int CMsgHookMap; };

Re: [PATCH] D18953: [ms][dll] #26935 Defining a dllimport function should cause it to be exported

2016-04-15 Thread Andrew V. Tischenko via cfe-commits
avt77 changed the visibility of this Differential Revision from "All Users" to "Public (No Login Required)". avt77 updated this revision to Diff 53870. avt77 added a comment. I fixed all issues discovered by Richard and Reid. As result the tests were updated as well. Please, review again.

[PATCH] D19156: [ms][dll] #27212: Generating of implicit special members should take into account MSVC compatibility version

2016-04-15 Thread Andrew V. Tischenko via cfe-commits
avt77 created this revision. avt77 added a reviewer: rnk. avt77 added a subscriber: cfe-commits. Clang creates implicit move constructor/assign operator in all cases if there is std=c++11. But MSVC supports such generation starting from version 1900 only. As result we have some binary

Re: [PATCH] D18953: [ms][dll] #26935 Defining a dllimport function should cause it to be exported

2016-04-12 Thread Andrew V. Tischenko via cfe-commits
avt77 marked an inline comment as done. Comment at: lib/Sema/SemaDecl.cpp:5570 @@ -5565,3 +5569,3 @@ // and qualified friend declarations. - // NB: MSVC converts such a declaration to dllexport. + // NB: MSVC converts such a declaration to dllexport that's why we do it

[PATCH] D18953: [ms][dll] #26935 Defining a dllimport function should cause it to be exported

2016-04-11 Thread Andrew V. Tischenko via cfe-commits
avt77 created this revision. avt77 added a reviewer: rnk. avt77 added a subscriber: cfe-commits. avt77 changed the visibility of this Differential Revision from "Public (No Login Required)" to "All Users". If we have some function with dllimport attribute and then we have the function