[PATCH] D80249: WIP: CodeGen: Don't lazily construct MachineFunctionInfo
Francis Visoiu Mistrih via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 16 19:10:43 PDT 2020
thegameg added a comment.
I think there are a bunch of APIs that have this issue, so I think the general direction is good. I don't expect this to have any memory/compile-time impact, since probably even the smallest input will create the MachineFunctionInfo anyway. Was this the initial intention of having it build lazily?
So you should probably go ahead, I doubt anyone has any objections after all this time.
Nit: maybe commit the changes from `AMDGPUMachineFunction.cpp` separately
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80249/new/
https://reviews.llvm.org/D80249
More information about the llvm-commits
mailing list