search for: enableloadpr

...Transforms/Scalar/GVN.cpp b/lib/Transforms/Scalar/GVN.cpp index 2c47a8a..a3387e3 100644 --- a/lib/Transforms/Scalar/GVN.cpp +++ b/lib/Transforms/Scalar/GVN.cpp @@ -1767,7 +1767,7 @@ bool GVN::processNonLocalLoad(LoadInst *LI) { } // Step 4: Eliminate partial redundancy. - if (!EnablePRE || !EnableLoadPRE) + if (!EnableLoadPRE) return false; return PerformLoadPRE(LI, ValuesPerBlock, UnavailableBlocks); This will disable Scalar PRE without disabling load PRE. (note, again, however, that load PRE can create exactly the same GEP situation you are referring to)

...ansforms/Scalar/GVN.cpp index 2c47a8a..a3387e3 100644 > --- a/lib/Transforms/Scalar/GVN.cpp > +++ b/lib/Transforms/Scalar/GVN.cpp > @@ -1767,7 +1767,7 @@ bool GVN::processNonLocalLoad(LoadInst *LI) { > } > > // Step 4: Eliminate partial redundancy. > - if (!EnablePRE || !EnableLoadPRE) > + if (!EnableLoadPRE) > return false; > > return PerformLoadPRE(LI, ValuesPerBlock, UnavailableBlocks); > > > > > This will disable Scalar PRE without disabling load PRE. > > > (note, again, however, that load PRE can create exactly the same GEP situ...

Hi, Daniel: Thanks a lot for detailed background information, we are willing to provide the right fix, however it will take time, do you mind if you forward me the discussion you had 5 months ago? I may not be able to access it since I only joined ellvmdev list this week. I did some performance measurement last night, some of our critical benchmark degraded up to 30% with your patch, so we have

...ardRecognizer"}, {"_ZL11RewriterOpt", "RewriterOpt"}, {"_ZL14ScheduleSpills", "ScheduleSpills"}, {"_ZL17CriticalEdgeSplit", "CriticalEdgeSplit"}, {"_ZL9EnablePRE", "EnablePRE"}, {"_ZL13EnableLoadPRE", "EnableLoadPRE"}, {"_ZL9Threshold", "Threshold"}, {"_ZL16DisablePromotion", "DisablePromotion"}, {"_ZL15UnrollThreshold", "UnrollThreshold"}, {"_ZL11UnrollCount", "UnrollCount"},...