mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2025-08-09 11:25:55 +00:00
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
This commit is contained in:
@@ -187,4 +187,13 @@ fn1:
|
|||||||
ret
|
ret
|
||||||
|
|
||||||
The problem is the store's chain operand is not the load X but rather
|
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.
|
||||||
|
Reference in New Issue
Block a user