Mark ~CPUCachingAllocator override - #22295
Conversation
Summary: MemoryAllocator declares a virtual destructor, so ~CPUCachingAllocator already overrides it implicitly. Without the keyword, -Winconsistent-missing-destructor-override fires, and it is -Werror on the iphoneos configs, which breaks every Apple build that reaches this header. Differential Revision: D117944096
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/22295
Note: Links to docs will display an error until the docs builds have been completed. ❌ 4 New Failures, 1 Unrelated FailureAs of commit 22423cc with merge base 3353bd5 ( NEW FAILURES - The following jobs have failed:
FLAKY - The following job failed but was likely due to flakiness present on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
@doggeral has exported this pull request. If you are a Meta employee, you can view the originating Diff in D117944096. |
This PR needs a
|
Summary:
MemoryAllocator declares a virtual destructor, so ~CPUCachingAllocator
already overrides it implicitly. Without the keyword,
-Winconsistent-missing-destructor-override fires, and it is -Werror on the
iphoneos configs, which breaks every Apple build that reaches this header.
Differential Revision: D117944096