Re: [PATCH] D13375: [MSVC] Fix for http://llvm.org/PR24132: __declspec(property): double invocations of foo() when compiling foo()->propertyName

2015-10-13 Thread Alexey Bataev via cfe-commits
ABataev added a comment. I see, no problems, thanks! Best regards, Alexey Bataev = Software Engineer Intel Compiler Team Intel Corp. http://reviews.llvm.org/D13375 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D13375: [MSVC] Fix for http://llvm.org/PR24132: __declspec(property): double invocations of foo() when compiling foo()->propertyName

2015-10-13 Thread Reid Kleckner via cfe-commits
rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land. lgtm Thanks, sorry for the delay, EH was keeping me busy. =/ http://reviews.llvm.org/D13375 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D13375: [MSVC] Fix for http://llvm.org/PR24132: __declspec(property): double invocations of foo() when compiling foo()->propertyName

2015-10-01 Thread Alexey Bataev via cfe-commits
ABataev created this revision. ABataev added a reviewer: rnk. ABataev added a subscriber: cfe-commits. Removes extra codegen for base expression of MS property call http://reviews.llvm.org/D13375 Files: lib/Sema/SemaPseudoObject.cpp test/CodeGenCXX/ms-property.cpp Index: