[PATCH] D35013: [inline asm] repeating constraint matches the same output

2017-07-26 Thread Ziv Izhar via Phabricator via cfe-commits
zizhar abandoned this revision. zizhar added a comment. I agree, My bad, apparently gcc doesn't support that, I see no reason to add that. Closing the review. Repository: rL LLVM https://reviews.llvm.org/D35013 ___ cfe-commits mailing list cfe-co

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

2017-05-10 Thread Ziv Izhar via Phabricator via cfe-commits
zizhar added a comment. Ping https://reviews.llvm.org/D31383 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

2017-05-03 Thread Ziv Izhar via Phabricator via cfe-commits
zizhar updated this revision to Diff 97602. https://reviews.llvm.org/D31383 Files: lib/Basic/TargetInfo.cpp test/Sema/asm.c Index: test/Sema/asm.c === --- test/Sema/asm.c +++ test/Sema/asm.c @@ -160,6 +160,41 @@ return ret;

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

2017-05-03 Thread Ziv Izhar via Phabricator via cfe-commits
zizhar added inline comments. Comment at: lib/Basic/TargetInfo.cpp:506 case '!': // Disparage severely. case '*': // Ignore for choosing register preferences. +case 'i': // Ignore i as output constraint (match from the other chars) ahatanak wrote: >