On Fri, Jul 17, 2015 at 5:52 AM, Alexey Frolov <alexfrolov1...@yandex.ru> wrote:
> alexfrol added a comment. > > In http://reviews.llvm.org/D10536#203421, @compnerd wrote: > > > LGTM with the small change to rename isValidAsmValue at least. > > > All issues are fixed. I'll update the patch soon. > > In http://reviews.llvm.org/D10536#203820, @echristo wrote: > > > This will need documentation updates yes? Also backend support? > > > > -eric > > > What documentation updates do you mean? > Additions/modifications to: http://llvm.org/docs/LangRef.html#inline-assembler-expressions -- Sean Silva > Backend support is necessary for ‘v’, ‘k’, and ‘Yk’ constraints. They are > AVX-512 specific and as far as I know work for AVX512 backend support is > still ongoing. > > In http://reviews.llvm.org/D10536#203821, @echristo wrote: > > > I don't see a test for the 'k' support or a few other things either. Can > you split this out a bit into the various parts for checking the existing > constraints, and adding individual ones? > > > > -eric > > > Sure, will do. > > > Repository: > rL LLVM > > http://reviews.llvm.org/D10536 > > > > > _______________________________________________ > cfe-commits mailing list > cfe-commits@cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list cfe-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
