[PATCH] D139767: [DFAPacketizer] Move DefaultVLIWScheduler class declaration to header file
Darshan Bhat via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Dec 10 20:18:56 PST 2022
DarshanRamakant added a subscriber: dexonsmith.
DarshanRamakant added a comment.
In D139767#3986476 <https://reviews.llvm.org/D139767#3986476>, @dexonsmith wrote:
> I don't know much about this code; resigning as reviewer. But I took a quick look. I wonder if this the right fix, or whether `VLIWPacketizerList::VLIWScheduler`'s type should be `ScheduleDAGInstrs`, and downcast when necessary on use, allowing `DefaultVLIWScheduler` to stay private. (Up to others to sort out!)
Thanks @dexonsmith for the suggestion. But I think even if we want to downcast ( dynamic_cast) the declaration should be available to the derived class, which is not the case in the current implementation.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139767/new/
https://reviews.llvm.org/D139767
More information about the llvm-commits
mailing list