[llvm-dev] [RFC] Simple GVN hoist

Momchil Velikov via llvm-dev llvm-dev at lists.llvm.org
Wed Sep 15 04:18:01 PDT 2021


On Tue, Sep 14, 2021 at 7:56 PM Sjoerd Meijer via llvm-dev
<llvm-dev at lists.llvm.org> wrote:
> Fix GVNHoist. I like your patch because it's small and concise, but missing in the RFC is a discussion why we don't try to re-enable GVNHoist. I know it was briefly enabled by default, which was reverted due to correctness (or was it regressions?) problems. But if this belongs in GVNHoist, could this for example be added to GVNHoist, and only this part enabled? Not sure if that's possible as I haven't looked at GVNHoist.

Yes, GVNHoist will do what we need


More information about the llvm-dev mailing list