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

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 17 13:28:30 PDT 2017


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





More information about the llvm-commits mailing list