[PATCH] D33865: Mark llvm.*annotation intrinsics as NoMem and Speculatable

[Sanjoy Das via Phabricator via llvm-commits]

sanjoy added a comment.

In https://reviews.llvm.org/D33865#782258, @mareko wrote:

> Do we need to add more reviewers?


JFYI: so far you haven't added any reviewers. :)

However, if you want to make this change, at the very least you also need to edit the language reference entry for `llvm.var.annotation` to state that it isn't allowed to express control flow dependent properties.  Can you also please give some examples of how you use `llvm.var.annotate` (for discussion)?


https://reviews.llvm.org/D33865