From: Evan Cheng Date: Fri, 27 Jan 2006 22:54:32 +0000 (+0000) Subject: A bit of wisdom from Chris on the last entry. X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=d41e9e5e7a5243fdda88587a96eadaac37093eea;p=oota-llvm.git A bit of wisdom from Chris on the last entry. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@25715 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Target/X86/README.txt b/lib/Target/X86/README.txt index 9410a9acf8d..23087616446 100644 --- a/lib/Target/X86/README.txt +++ b/lib/Target/X86/README.txt @@ -187,4 +187,13 @@ fn1: ret The problem is the store's chain operand is not the load X but rather -a TokenFactor of the load X and load Y. This prevents the folding. +a TokenFactor of the load X and load Y, which prevents the folding. + +There are two ways to fix this: + +1. The dag combiner can start using alias analysis to realize that y/x + don't alias, making the store to X not dependent on the load from Y. +2. The generated isel could be made smarter in the case it can't + disambiguate the pointers. + +Number 1 is the preferred solution.