[PATCH] D94687: [AArch64] Make target intrinsics DefaultAttrIntrinsics.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 18 09:40:00 PST 2021


fhahn added a comment.

In D94687#2501147 <https://reviews.llvm.org/D94687#2501147>, @SjoerdMeijer wrote:

> Looks like we get a lot of happiness from using these attributes. ;-)
> I had a look at it before and the general direction also looked okay to me too, which I was happy to see confirmed. Therefore I am happy to accept this, even though I haven't checked all intrinsics because this is a lot of changes. So perhaps wait a day with committing in case anyone wants to have one more look.

Thanks! The changes will at least ensure that there shouldn't  be regression after D94106 <https://reviews.llvm.org/D94106>.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94687



More information about the llvm-commits mailing list