[PATCH] D117328: GlobalISel: Add bitcast to tryFoldUnmergeCast

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 29 12:46:16 PDT 2022


arsenm added a comment.

I'm not sure what to do about this. It feels right to treat bitcast as a legalization artifact, but I also can't convince myself it really is. You can always implement bitcast as a stack store and reload, so the fundamental property the artifact combiner is needed for isn't there.


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

https://reviews.llvm.org/D117328



More information about the llvm-commits mailing list