r306301 - [inline asm] dot operator while using imm generates wrong ir + asm - clang part

2017-06-26 Thread Marina Yatsina via cfe-commits
Author: myatsina Date: Mon Jun 26 09:09:55 2017 New Revision: 306301 URL: http://llvm.org/viewvc/llvm-project?rev=306301=rev Log: [inline asm] dot operator while using imm generates wrong ir + asm - clang part Inline asm dot operator while using imm generates wrong ir and asm This is the test

r306297 - [inline asm][gcc-compatiblity] "=i" output constraint support

2017-06-26 Thread Marina Yatsina via cfe-commits
Author: myatsina Date: Mon Jun 26 08:55:51 2017 New Revision: 306297 URL: http://llvm.org/viewvc/llvm-project?rev=306297=rev Log: [inline asm][gcc-compatiblity] "=i" output constraint support Ignore ‘i’,’n’,’E’,’F’ as output constraints in inline assembly (gcc compatibility) Differential

r290541 - Fix build error caused by r290539.

2016-12-26 Thread Marina Yatsina via cfe-commits
Author: myatsina Date: Mon Dec 26 07:16:40 2016 New Revision: 290541 URL: http://llvm.org/viewvc/llvm-project?rev=290541=rev Log: Fix build error caused by r290539. Modified: cfe/trunk/lib/Sema/SemaStmtAsm.cpp Modified: cfe/trunk/lib/Sema/SemaStmtAsm.cpp URL:

r290539 - [inline-asm]No error for conflict between inputs\outputs and clobber list

2016-12-26 Thread Marina Yatsina via cfe-commits
Author: myatsina Date: Mon Dec 26 06:23:42 2016 New Revision: 290539 URL: http://llvm.org/viewvc/llvm-project?rev=290539=rev Log: [inline-asm]No error for conflict between inputs\outputs and clobber list According to extended asm syntax, a case where the clobber list includes a variable from

[PATCH] D26587: [AVX512][inline-asm] Fix AVX512 inline assembly instruction resolution when the size qualifier of a memory operand is not specified explicitly.

2016-11-23 Thread Marina Yatsina via cfe-commits
myatsina accepted this revision. myatsina added a comment. This revision is now accepted and ready to land. LGTM Please emphasize in you commit message that this the test case for the llvm commit (and even better if you add the commit number of the llvm commit), Repository: rL LLVM

Re: [PATCH] D21959: [X86] Add xgetbv xsetbv intrinsics

2016-08-16 Thread Marina Yatsina via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL278783: [X86] Add xgetbv/x[X86] Add xgetbv xsetbv intrinsics to non-windows platforms (authored by myatsina). Changed prior to commit: https://reviews.llvm.org/D21959?vs=65676=68146#toc Repository:

r278783 - [X86] Add xgetbv/x[X86] Add xgetbv xsetbv intrinsics to non-windows platforms

2016-08-16 Thread Marina Yatsina via cfe-commits
Author: myatsina Date: Tue Aug 16 03:13:36 2016 New Revision: 278783 URL: http://llvm.org/viewvc/llvm-project?rev=278783=rev Log: [X86] Add xgetbv/x[X86] Add xgetbv xsetbv intrinsics to non-windows platforms commit on behalf of guyblank Differential Revision: https://reviews.llvm.org/D21959

Re: [PATCH] D18123: Fix implicit copy ctor and copy assignment operator warnings when -Wdeprecated passed.

2016-03-16 Thread Marina Yatsina via cfe-commits
myatsina added a comment. I've committed the fix in revision 263630 http://reviews.llvm.org/D18123 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D18175: Avoid using LookupResult's implicit copy ctor and assignment operator to avoid warnings

2016-03-16 Thread Marina Yatsina via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL263630: Avoid using LookupResult's implicit copy ctor and assignment operator to… (authored by myatsina). Changed prior to commit: http://reviews.llvm.org/D18175?vs=50780=50811#toc Repository: rL

r263630 - Avoid using LookupResult's implicit copy ctor and assignment operator to avoid warnings

2016-03-16 Thread Marina Yatsina via cfe-commits
Author: myatsina Date: Wed Mar 16 04:56:58 2016 New Revision: 263630 URL: http://llvm.org/viewvc/llvm-project?rev=263630=rev Log: Avoid using LookupResult's implicit copy ctor and assignment operator to avoid warnings The purpose of this patch is to keep the same functionality without using

Re: [PATCH] D18175: Avoid using LookupResult's implicit copy ctor and assignment operator to avoid warnings

2016-03-15 Thread Marina Yatsina via cfe-commits
myatsina updated this revision to Diff 50780. myatsina added a comment. Adding requested test case + changing the code accordingly Repository: rL LLVM http://reviews.llvm.org/D18175 Files: lib/Sema/SemaStmtAsm.cpp test/CodeGen/ms-inline-asm-errors.cpp Index: lib/Sema/SemaStmtAsm.cpp

Re: [PATCH] D18175: Avoid using LookupResult's implicit copy ctor and assignment operator to avoid warnings

2016-03-15 Thread Marina Yatsina via cfe-commits
myatsina added a comment. How can I create a test case where isSingleResult() returns false? I don't see existent tests in the commit that added this method's functionality. Repository: rL LLVM http://reviews.llvm.org/D18175 ___ cfe-commits

[PATCH] D18175: Avoid using LookupResult's implicit copy ctor and assignment operator to avoid warnings

2016-03-15 Thread Marina Yatsina via cfe-commits
myatsina created this revision. myatsina added reviewers: rnk, hintonda, rjmccall, dblaikie. myatsina added a subscriber: cfe-commits. myatsina set the repository for this revision to rL LLVM. The purpose of this patch is to keep the same functionality without using LookupResult's implicit copy

Re: [PATCH] D18123: Fix implicit copy ctor and copy assignment operator warnings when -Wdeprecated passed.

2016-03-14 Thread Marina Yatsina via cfe-commits
myatsina added a comment. In http://reviews.llvm.org/D18123#374170, @hintonda wrote: > LookupResult's copy ctor and assignment operator are used in > Sema::LookupInlineAsmField(). Looks like these were added back in December. > > I'll remove the defaults, from this patch, but not sure how to

Re: [PATCH] D17766: [ms-inline-asm][AVX512] Add ability to use k registers in MS inline asm + fix bag with curly braces

2016-03-07 Thread Marina Yatsina via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL262842: [ms-inline-asm][AVX512] Add ability to use k registers in MS inline asm + fix… (authored by myatsina). Changed prior to commit: http://reviews.llvm.org/D17766?vs=49945=49978#toc Repository:

r262842 - [ms-inline-asm][AVX512] Add ability to use k registers in MS inline asm + fix bag with curly braces

2016-03-07 Thread Marina Yatsina via cfe-commits
Author: myatsina Date: Mon Mar 7 12:10:25 2016 New Revision: 262842 URL: http://llvm.org/viewvc/llvm-project?rev=262842=rev Log: [ms-inline-asm][AVX512] Add ability to use k registers in MS inline asm + fix bag with curly braces Until now curly braces could only be used in MS inline assembly

Re: [PATCH] D17766: [ms-inline-asm][AVX512] Add ability to use k registers in MS inline asm + fix bag with curly braces

2016-03-07 Thread Marina Yatsina via cfe-commits
myatsina added a subscriber: cfe-commits. myatsina updated this revision to Diff 49945. myatsina marked an inline comment as done. myatsina added a comment. Updated test to reflect changes in the llvm part of the review (http://reviews.llvm.org/D17767) Repository: rL LLVM

r261618 - [ms-inline-asm] Fixing bug in single asm statement support

2016-02-23 Thread Marina Yatsina via cfe-commits
Author: myatsina Date: Tue Feb 23 02:53:45 2016 New Revision: 261618 URL: http://llvm.org/viewvc/llvm-project?rev=261618=rev Log: [ms-inline-asm] Fixing bug in single asm statement support Fixing a crash caused by trying to merge a single-line asm statement with an asm block that follows it,

r259639 - -inline-asm][X86] Add ability to use AVX512 in MS inline asm

2016-02-03 Thread Marina Yatsina via cfe-commits
Author: myatsina Date: Wed Feb 3 05:32:08 2016 New Revision: 259639 URL: http://llvm.org/viewvc/llvm-project?rev=259639=rev Log: -inline-asm][X86] Add ability to use AVX512 in MS inline asm Defined the new AVX512 registers in clang inline asm. Fixed a bug in the MC subtarget info creation

r256545 - [ms inline asm] Add support for label names with '$' chars

2015-12-29 Thread Marina Yatsina via cfe-commits
Author: myatsina Date: Tue Dec 29 02:49:34 2015 New Revision: 256545 URL: http://llvm.org/viewvc/llvm-project?rev=256545=rev Log: [ms inline asm] Add support for label names with '$' chars In MS inline asm syntax a label with '$' char produces an error, while in AT it does not. In AT inline asm

[PATCH] D15795: [ms inline asm] Add support for label names with '$' chars

2015-12-28 Thread Marina Yatsina via cfe-commits
myatsina created this revision. myatsina added reviewers: ehsan, rnk. myatsina added subscribers: llvm-commits, cfe-commits. myatsina set the repository for this revision to rL LLVM. In MS inline asm syntax a label with '$' char produces an error, while in AT it does not. In AT inline asm syntax

r256382 - [X86][ms-inline asm] Test case for adding support for memory operands that include structs

2015-12-24 Thread Marina Yatsina via cfe-commits
Author: myatsina Date: Thu Dec 24 06:11:40 2015 New Revision: 256382 URL: http://llvm.org/viewvc/llvm-project?rev=256382=rev Log: [X86][ms-inline asm] Test case for adding support for memory operands that include structs Test case for commit 256381 Differential Revision:

Re: [PATCH] D15749: [X86][ms-inline asm] Test case for adding support for memory operands that include structs

2015-12-24 Thread Marina Yatsina via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL256382: [X86][ms-inline asm] Test case for adding support for memory operands that… (authored by myatsina). Changed prior to commit: http://reviews.llvm.org/D15749?vs=43531=43593#toc Repository: rL

[PATCH] D15749: [X86][ms-inline asm] Test case for adding support for memory operands that include structs

2015-12-23 Thread Marina Yatsina via cfe-commits
myatsina created this revision. myatsina added reviewers: rnk, mcrosier. myatsina added subscribers: llvm-commits, cfe-commits. myatsina set the repository for this revision to rL LLVM. Test case for review: http://reviews.llvm.org/D15748 Repository: rL LLVM http://reviews.llvm.org/D15749

r255890 - [ms-inline-asm] Add support for composite structs in MS inline asm

2015-12-17 Thread Marina Yatsina via cfe-commits
Author: myatsina Date: Thu Dec 17 06:51:51 2015 New Revision: 255890 URL: http://llvm.org/viewvc/llvm-project?rev=255890=rev Log: [ms-inline-asm] Add support for composite structs in MS inline asm Add MS inline asm support for structs that contain fields that are also structs. Differential