[llvm] r300200 - Re-apply "[GVNHoist] Move GVNHoist to function simplification part of pipeline."

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 26 20:28:49 PDT 2017


On Wed, Apr 26, 2017 at 5:01 PM, Chandler Carruth via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> Finally got a good test case for this:
>
> http://llvm.org/PR32821
>
> I'm going to flip the flag to "off" as suggested by Geoff, I completely
> agree with that rather than moving it around.
>

Another testcase where GVNHoist makes bad choices
https://bugs.llvm.org/show_bug.cgi?id=32811

Thanks,

-- 
Davide

"There are no solved problems; there are only problems that are more
or less solved" -- Henri Poincare


More information about the llvm-commits mailing list