[PATCH] D96087: Option to ignore llvm[.compiler].used uses in hasAddressTaken()

[Duncan P. N. Exon Smith via Phabricator via llvm-commits]

dexonsmith added a comment.

In D96087#2552612 <https://reviews.llvm.org/D96087#2552612>, @rampitec wrote:

> In D96087#2552603 <https://reviews.llvm.org/D96087#2552603>, @dexonsmith wrote:
>
>> (Passing comment not really related to the patch, but I wonder how hard / useful it would be to convert `@llvm.used` and `@llvm.compiler.used` to attributes (or similar)...?)
>
> At the very least these variables are defined in the LLVM IR specification, so it would be a compatibility break.

Right, it'd certainly be a semantic change; we'd need an auto-upgrade for bitcode (surely no compatibility break there) and a transition of some sort for the C API (I imagine there's a reasonable way of handling both, but maybe you're seeing some difficulty?). Probably it could / should be modelled as part of the linkage somehow, rather than an attribute, but it might not be worth a bit on `Value`. (But probably this isn't the right venue to discuss this; carry on...)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96087/new/

https://reviews.llvm.org/D96087