[PATCH] [GVN] Set proper debug locations for some instructions created by GVN.

Alexey Samsonov vonosmas at gmail.com
Tue Jun 9 17:34:31 PDT 2015


In http://reviews.llvm.org/D10351#185934, @echristo wrote:

> The GVN ones seem reasonable since the instructions they're replacing are deleted, are the ones in PHITransAddr the same?


You could say so. Instructions created in PHITransAddr would eventually be used here:

  // Okay, we can eliminate this load by inserting a reload in the predecessor
  // and using PHI construction to get the value in the other predecessors, do
  // it.
  DEBUG(dbgs() << "GVN REMOVING PRE LOAD: " << *LI << '\n');
  DEBUG(if (!NewInsts.empty())
          dbgs() << "INSERTED " << NewInsts.size() << " INSTS: "
                 << *NewInsts.back() << '\n');


http://reviews.llvm.org/D10351

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list