[PATCH] D79010: [AArch64][SVE] Remove unused FP reduction intrinsic definitions

Cullen Rhodes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 30 01:49:11 PDT 2020


c-rhodes added a comment.

In D79010#2010679 <https://reviews.llvm.org/D79010#2010679>, @efriedma wrote:

> LGTM
>
> It makes review harder when I have to read a patch and spend extra time to figure out whether something that looks like an NFC change at first glance is actually supposed to be one.


No worries, thanks for reviewing!


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

https://reviews.llvm.org/D79010





More information about the llvm-commits mailing list