[PATCH] D31383: [inline asm] "=i" output constraint support - gcc compatiblity

This revision was automatically updated to reflect the committed changes. Closed by commit rL306297: [inline asm][gcc-compatiblity] "=i" output constraint support (authored by myatsina). Changed prior to commit: https://reviews.llvm.org/D31383?vs=97602&id=103967#toc Repository: rL LLVM http

[PATCH] D15075: No error for conflict between inputs\outputs and clobber list

myatsina added a comment. In https://reviews.llvm.org/D15075#940711, @dolson wrote: > Hello, > > In the process of upgrading from clang 3.6.1 to a newer version, I ran into > this new error and thus imported the new intrinsics from intrin.h for rep > movsb and friends. I see several discussion

[PATCH] D15075: No error for conflict between inputs\outputs and clobber list

This revision was automatically updated to reflect the committed changes. Closed by commit rL290539: [inline-asm]No error for conflict between inputs\outputs and clobber list (authored by myatsina). Changed prior to commit: https://reviews.llvm.org/D15075?vs=74500&id=82495#toc Repository: rL

[PATCH] D15075: No error for conflict between inputs\outputs and clobber list

myatsina added a comment. In https://reviews.llvm.org/D15075#631207, @vitalybuka wrote: > These patches break asan tests: > http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/528/steps/check-asan%20in%20gcc%20build/logs/stdio Vitaly , This patch is a gcc compatibility issue and i

[PATCH] D15075: No error for conflict between inputs\outputs and clobber list

myatsina added a comment. In https://reviews.llvm.org/D15075#631316, @ahatanak wrote: > In https://reviews.llvm.org/D15075#631237, @myatsina wrote: > > > In https://reviews.llvm.org/D15075#631207, @vitalybuka wrote: > > > > > These patches break asan tests: > > > http://lab.llvm.org:8011/builder

[PATCH] D15075: No error for conflict between inputs\outputs and clobber list

myatsina added a comment. Vitaly, Thanks for fixing the test! Test was fixed by in https://reviews.llvm.org/rL290621: [asan] Fix test broken by r290540 Review: https://reviews.llvm.org/D28128 Repository: rL LLVM https://reviews.llvm.org/D15075 ___

[PATCH] D29599: Clang Changes for alloc_align

myatsina added inline comments. Comment at: test/CodeGen/alloc-align-attr.c:19 +} +// Condition where test2 param needs casting. +__INT32_TYPE__ test2(__SIZE_TYPE__ a) { Where exactly do we see this test2 param casting? I think you have a missing check before the

[PATCH] D28989: [X86][MS]Adjacent comments within multi-line inline assembly statement

myatsina accepted this revision. myatsina added a comment. This revision is now accepted and ready to land. LGTM Repository: rL LLVM https://reviews.llvm.org/D28989 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi