Re: [PATCH] D12594: [X86-64] Allow additional register names in inline assembler.

2015-09-03 Thread Alexey Bataev via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL246835: [X86-64] Allow additional register names in inline assembler. (authored by ABataev). Changed prior to commit: http://reviews.llvm.org/D12594?vs=33926&id=34011#toc Repository: rL LLVM http://

Re: [PATCH] D12594: [X86-64] Allow additional register names in inline assembler.

2015-09-03 Thread Alexey Bataev via cfe-commits
ABataev added a comment. Eric, thanks for the review! Will do! http://reviews.llvm.org/D12594 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D12594: [X86-64] Allow additional register names in inline assembler.

2015-09-03 Thread Eric Christopher via cfe-commits
echristo accepted this revision. echristo added a comment. This revision is now accepted and ready to land. If you could interleave the test and the check I'd really appreciate it, e.g.: __asm__ ("mov r8w, 100;"); // CHECK: call void asm sideeffect "mov r8w, 100;" Otherwise LGTM. Thanks! -er

[PATCH] D12594: [X86-64] Allow additional register names in inline assembler.

2015-09-03 Thread Alexey Bataev via cfe-commits
ABataev created this revision. ABataev added reviewers: rjmccall, echristo. ABataev added a subscriber: cfe-commits. Patch allows to recognize additional registers x8d, x8b, x8w - x15d, x15b, x15w in inline assembler, already recognized by backend http://reviews.llvm.org/D12594 Files: lib/Bas