[PATCH] D155212: [nfc] small maintainability IndirectCallPromotion changes

Rong Xu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 20 14:06:39 PDT 2023


xur added a comment.

How about we get rid of the module field? The function passed in all associating with the module so we can get the module from there.  And there seems only one use of M (when calling annotateValueSite).  
Other changes look good to me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155212



More information about the llvm-commits mailing list