[PATCH] D29110: LangRef: Document the allowed metadata dropping transforms.

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 25 11:21:51 PST 2017


kparzysz added a comment.

In https://reviews.llvm.org/D29110#656562, @pcc wrote:

> In https://reviews.llvm.org/D29110#656551, @kparzysz wrote:
>
> > Would it still look like metadata to someone who has not read that piece of documentation?  Would this information invalidate previously correct understanding of the IR?
>
>
> [...] there's no magical way to make sure that clients don't make mistakes.


I see what you were referring to.  What I meant there was that the design should make it harder for users to make such mistakes.  If non-metadata and metadata looks the same, and metadata is safely removable, then the risk of someone accidentally removing non-metadata increases.  This is separate from any impact on existing code.


https://reviews.llvm.org/D29110





More information about the llvm-commits mailing list