[PATCH] D75885: [AArch64] Allow logical immediates to have all-1 in top bits

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 17 09:38:15 PDT 2020


MaskRay added a comment.

In D75885#1926549 <https://reviews.llvm.org/D75885#1926549>, @samparker wrote:

> I think you should try to figure what the issue is with SVE... It's not good practise to scratch your particular itch and then not fix the bug it exposes too.


May I say that this suggests a quality of implementation issue of the SVE patches themselves? GNU as has the same problem... The organization of code makes is sorta clear that if we intend to fix SVE, it will be a separate patch.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75885/new/

https://reviews.llvm.org/D75885





More information about the llvm-commits mailing list