[PATCH] D77875: [ARM] Armv8.6-a Matrix Mul cmd line support

2020-04-24 Thread Luke Geeson via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG740a1dd050ee: [ARM] Armv8.6-a Matrix Mul cmd line support (authored by LukeGeeson). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77875/new/ https://reviews

[PATCH] D77875: [ARM] Armv8.6-a Matrix Mul cmd line support

2020-04-14 Thread Luke Geeson via Phabricator via cfe-commits
LukeGeeson updated this revision to Diff 257362. LukeGeeson added a comment. - updated commit message to reflect sve=>f32/64 implication - added comment in AArch64.cpp to that effect CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77875/new/ https://reviews.llvm.org/D77875 Files: clang

[PATCH] D77875: [ARM] Armv8.6-a Matrix Mul cmd line support

2020-04-14 Thread Luke Geeson via Phabricator via cfe-commits
LukeGeeson added a comment. In D77875#1979980 , @DavidSpickett wrote: > > Note: +f32mm and +f64mm are optional and so have to be enabled by default > > I think I know what you mean, but "and so are not enabled by default" would > be clearer. > > Also to

[PATCH] D77875: [ARM] Armv8.6-a Matrix Mul cmd line support

2020-04-14 Thread David Spickett via Phabricator via cfe-commits
DavidSpickett added a comment. Or in other words, does amrv8.6-a+sve imply +f32mm *and* +f64mm? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77875/new/ https://reviews.llvm.org/D77875 ___ cfe-commits

[PATCH] D77875: [ARM] Armv8.6-a Matrix Mul cmd line support

2020-04-14 Thread David Spickett via Phabricator via cfe-commits
DavidSpickett added a reviewer: DavidSpickett. DavidSpickett added a comment. > Note: +f32mm and +f64mm are optional and so have to be enabled by default I think I know what you mean, but "and so are not enabled by default" would be clearer. Also to double check, does f64mm imply f32mm? (not su

[PATCH] D77875: [ARM] Armv8.6-a Matrix Mul cmd line support

2020-04-10 Thread Luke Geeson via Phabricator via cfe-commits
LukeGeeson added a comment. harbour master builds, unit tests passing, failures down to linting Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77875/new/ https://reviews.llvm.org/D77875 ___ cfe-commits

[PATCH] D77875: [ARM] Armv8.6-a Matrix Mul cmd line support

2020-04-10 Thread Luke Geeson via Phabricator via cfe-commits
LukeGeeson created this revision. LukeGeeson added a reviewer: t.p.northover. Herald added subscribers: cfe-commits, danielkiss, kristof.beyls. Herald added a project: clang. LukeGeeson added a parent revision: D77874: [AArch32] Armv8.6a Matrix Mul Assembly. This patch upstreams support for the