[PATCH] D71556: [AArch64][SVE] Implement intrinsic for non-faulting loads

Kerry McLaughlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 19 04:21:40 PST 2019


kmclaughlin planned changes to this revision.
kmclaughlin added a comment.

Thanks for the feedback on this patch, @efriedma & @sdesmalen!
I think there is still value in adding a NonFaulting flag to MachineMemOperand so that we can benefit from legalisation, but as this is not a requirement for the ACLE I have created a new patch which implements the non-faulting load intrinsic explicitly: https://reviews.llvm.org/D71698
I will leave this patch in the 'plan changes' state so that it can be referred to in future discussions on the mailing list.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71556





More information about the llvm-commits mailing list