[PATCH] D140993: [DAGCombine] fold (sext (sext_inreg x)) -> (sext (trunc x))

Amaury SECHET via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 4 11:48:03 PST 2023


deadalnix added a comment.

Yes, it shouldn't in itself, but throw another transform in the loop, and it may well be.

Is there another approach you are thinking of? I thought about doing just `(sext (sext_inreg (trunc x))) -> (sext (trunc x))` but that seemed too topical.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140993



More information about the llvm-commits mailing list