[PATCH] D50229: +fp16fml feature for ARM and AArch64

2018-09-21 Thread Sjoerd Meijer via Phabricator via cfe-commits
SjoerdMeijer added a comment. Ah, and just for your info, the proposal was just sent to the dev list: http://lists.llvm.org/pipermail/llvm-dev/2018-September/126346.html Repository: rC Clang https://reviews.llvm.org/D50229 ___ cfe-commits mailin

[PATCH] D50229: +fp16fml feature for ARM and AArch64

2018-09-21 Thread Sjoerd Meijer via Phabricator via cfe-commits
SjoerdMeijer added a comment. (I am now picking this up, and will try to progress this patch and also https://reviews.llvm.org/D50179) > Do you expect that the regression tests will be affected by the TargetParser > fixes? No, and that's exactly the reason why it would be nice to get this in.

[PATCH] D50229: +fp16fml feature for ARM and AArch64

2018-09-04 Thread Eli Friedman via Phabricator via cfe-commits
efriedma added a comment. Do you expect that the regression tests will be affected by the TargetParser fixes? If not, I guess this is okay... but please add clear comments explaining which bits of code you expect to go away after the TargetParser rewrite. Repository: rC Clang https://revi

[PATCH] D50229: +fp16fml feature for ARM and AArch64

2018-09-04 Thread Bernard Ogden via Phabricator via cfe-commits
bogden added a reviewer: efriedma. bogden added a comment. Hello Eli -- Sjoerd pointed out that you had concerns about this sort of command-line manipulation in his patch https://reviews.llvm.org/D50179. I'm having to do something similar here to deal with +fp16fml being implied by +fp16 from v

[PATCH] D50229: +fp16fml feature for ARM and AArch64

2018-08-03 Thread Bernard Ogden via Phabricator via cfe-commits
bogden created this revision. bogden added a reviewer: SjoerdMeijer. Herald added a reviewer: javed.absar. Herald added subscribers: cfe-commits, chrib, kristof.beyls. v8.4-A adds a few fp16 instructions that can optionally be implemented in CPUs of v8.2-A and above. This patch adds a feature to