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