[PATCH] D74904: [CallSiteInfo] Handle bundles when updating call site info

David Stenberg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 25 08:37:53 PST 2020


dstenb added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineFunction.cpp:886
+  if (MI->isBundle()) {
+    auto BundleIt = MI->getIterator();
+    do {
----------------
As the {erase,copy,move} functions seems to handle bundles in the same way, perhaps it is worth breaking this out to a common helper that either returns MI if it's not a bundle, or finds the call instruction in the bundle?


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

https://reviews.llvm.org/D74904





More information about the llvm-commits mailing list