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

llvm-svn: 97081

1 files changed, 4 insertions, 0 deletions

diff --git a/llvm/utils/TableGen/DAGISelMatcherGen.cpp b/llvm/utils/TableGen/DAGISelMatcherGen.cpp
index 9727c5f0dd1..c070e442dbd 100644
--- a/llvm/utils/TableGen/DAGISelMatcherGen.cpp
+++ b/llvm/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()));