On Mon, 2005-05-09 at 17:45 +0300, Mostafa Hagog wrote: > > > > It appears that GCSE considers "read only memory" as call clobbered, which > is not the case in CSE. I have took the test for read-only memory from CSE > and add it to GCSE where we compute the transparency. Here is a patch that > does so. This patch makes gcse eliminate redundant loads after stores for > the following example. The difference is seen when we compile with the > options: > "-O3 --param max-gcse-passes=3" with/without "-fgcse-las" (on a > powerpc-linux target). If this looks a reasonable change I will > regression-test and bootstrap the patch and ask for commit. > [ ... ] Yes, it looks quite reasonable. Please go ahead with the full testing cycle and consider the patch pre-approved once complete.
jeff
