[PATCH] D91512: [AArch64][Isel] Avoid implicit zext for SIGN_EXTEND_INREG (TRUNCATE)

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 14 11:34:45 PDT 2021


efriedma added a comment.

In D91512#2999435 <https://reviews.llvm.org/D91512#2999435>, @jaykang10 wrote:

> I wonder we can extend this check to other basic blocks.
>
> For example, if the `trunc` is in other basic block, there could be `CopyFromReg` instead of `trunc` in current basic block. Can we detect the `trunc` in this case please?

Ideally, I think we should kill off isDef32, and run a pass after isel to clean up redundant extension operations.  That would have fewer bugs, and we'd get cross-block optimization for free.


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

https://reviews.llvm.org/D91512



More information about the llvm-commits mailing list