Add a FIXME comment.
authorDan Gohman <gohman@apple.com>
Thu, 11 Nov 2010 18:08:43 +0000 (18:08 +0000)
committerDan Gohman <gohman@apple.com>
Thu, 11 Nov 2010 18:08:43 +0000 (18:08 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@118803 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/MachineLICM.cpp

index 811c86cfd8fee79520bc43b0e20b121fb7a96e5e..e3b21c4ca577bc0e989b26a585351540b7db2622 100644 (file)
@@ -944,6 +944,8 @@ bool MachineLICM::IsProfitableToHoist(MachineInstr &MI) {
     // In low register pressure situation, we can be more aggressive about 
     // hoisting. Also, favors hoisting long latency instructions even in
     // moderately high pressure situation.
+    // FIXME: If there are long latency loop-invariant instructions inside the
+    // loop at this point, why didn't the optimizer's LICM hoist them?
     DenseMap<unsigned, int> Cost;
     for (unsigned i = 0, e = MI.getDesc().getNumOperands(); i != e; ++i) {
       const MachineOperand &MO = MI.getOperand(i);