[PATCH] D125546: [RISCV] Use tail agnostic if inserting subvector/element at the end of the vector.

Zakk Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 24 08:55:20 PDT 2022


khchen abandoned this revision.
khchen added a comment.

In D125546#3525173 <https://reviews.llvm.org/D125546#3525173>, @reames wrote:

> Skimming through the code, is there any value in having a policy arg on slideup?  If so, I might suggest adding it to both in an NFC change, and then doing a separate change to show the opt quality improvement.

@reames thanks, I agree with you!
This patch changes policy from TU to TA at four places. It's difficult to review the improvement.
I will split them into smaller patches.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125546



More information about the llvm-commits mailing list