This is an archive of the discontinued LLVM Phabricator instance.

[VP][DAGCombiner] Remove unused-private-field VPMatchContext.Root
AcceptedPublic

Authored by DamonFool on Feb 7 2023, 10:24 PM.

Details

Summary

LLVM fails to build after D141891 due to -Wunused-private-field of VPMatchContext.Root
llvm-project/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:870:11: error: private field 'Root' is not used [-Werror,-Wunused-private-field]

SDNode *Root;
        ^

1 error generated.

Diff Detail

Event Timeline

DamonFool created this revision.Feb 7 2023, 10:24 PM
DamonFool requested review of this revision.Feb 7 2023, 10:24 PM

LGTM. Thank you.

fakepaper56 accepted this revision.Feb 7 2023, 10:52 PM
This revision is now accepted and ready to land.Feb 7 2023, 10:52 PM

Thanks @fakepaper56 for your review.
It had been fixed by @MaskRay .
So just close it.