[PATCH] D21274: [Bugpoint] Erase comdat annotations after removing a global's initializer.
David Majnemer via llvm-commits
llvm-commits at lists.llvm.org
Sun Jun 12 21:13:40 PDT 2016
majnemer added a comment.
In http://reviews.llvm.org/D21274#455769, @jlebar wrote:
> In http://reviews.llvm.org/D21274#455768, @majnemer wrote:
>
> > Do we need the same code in DeleteFunctionBody?
>
>
> Hm, I am not sure; I guess you're saying it's illegal to have a comdat annotation on a function declaration? It may only be present on a definition? But then, if we delete it on the declaration, do we also need to delete it on the corresponding global variable?
COMDATs are only valid on definitions.
http://reviews.llvm.org/D21274
More information about the llvm-commits
mailing list