add a fixme for an experiment that defeated me for the time being.

[oota-llvm.git] / utils / TableGen / DAGISelMatcherGen.cpp

diff --git a/utils/TableGen/DAGISelMatcherGen.cpp b/utils/TableGen/DAGISelMatcherGen.cpp
index 9727c5f0dd1806e877df053932f7f8fee934dc75..c070e442dbd9b54d1885ae2658b095e4cb0ea8df 100644 (file)
--- a/utils/TableGen/DAGISelMatcherGen.cpp
+++ b/utils/TableGen/DAGISelMatcherGen.cpp
@@ -490,6 +490,10 @@ void MatcherGen::EmitMatchCode(const TreePatternNode *N,
 void MatcherGen::EmitMatcherCode() {
   // If the pattern has a predicate on it (e.g. only enabled when a subtarget
   // feature is around, do the check).
+  // FIXME: This should get emitted after the match code below to encourage
+  // sharing.  This can't happen until we get an X86ISD::AddrMode node made by
+  // dag combine, eliminating the horrible side-effect-full stuff from 
+  // X86's MatchAddress.
   if (!Pattern.getPredicateCheck().empty())
     AddMatcherNode(new 
                  CheckPatternPredicateMatcherNode(Pattern.getPredicateCheck()));