[PATCH] D140192: [InstCombine] Preserve instruction name in replaceInstUsesWith()

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 16 04:45:45 PST 2022


spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM

Nice! The trade-off of using the Builder vs. raw create APIs always causes friction. 
If this means we can use the Builder all the time, that would be great.


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

https://reviews.llvm.org/D140192



More information about the llvm-commits mailing list