[llvm] [CodeGen][GC] Remove `GCInfoPrinter` (PR #75033)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 11 16:20:33 PST 2023
paperchalice wrote:
> Can you expand the statement "This pass is broken and looks like no one uses it." ? I suspect that it is broken due to the following inverted condition
>
> ```
> bool Printer::runOnFunction(Function &F) {
> if (F.hasGC()) ///////
> return false;
> ```
This code triggers the assertion:
https://github.com/llvm/llvm-project/blob/2e197602305be18b963928e6ae024a004a95af6d/llvm/lib/CodeGen/GCMetadata.cpp#L64-L66
No one has noticed it for the last 15 years.
https://github.com/llvm/llvm-project/pull/75033
More information about the llvm-commits
mailing list