Some of this is boilerplate which doesn't do anything yet.
Diff Detail
Event Timeline
llvm/include/llvm/Transforms/IPO/Attributor.h | ||
---|---|---|
4856 | Comment copy paste error |
LG, assuming we actually don't need the manifest. If we do, we need to see why and what to do about it.
llvm/lib/Transforms/IPO/AttributorAttributes.cpp | ||
---|---|---|
10251 | We never call manifest for what we assume is dead (https://github.com/llvm/llvm-project/blob/2e8ef93439a4a430fe5f96b9d999ce96405345f4/llvm/lib/Transforms/IPO/Attributor.cpp#L2141). | |
10262 | I doubt you need a manifest. Do you see the problems you are trying to avoid above? |
Comment copy paste error