[PATCH] D36233: [ThinLTO] Add FunctionAttrs to ThinLTO index

[Charles Saternos via Phabricator via llvm-commits]

ncharlie added inline comments.


================
Comment at: test/Bitcode/thinlto-function-summary-functionattrs.ll:4
+
+; CHECK: <GLOBALVAL_SUMMARY_BLOCK
+; ensure @f is marked readnone
----------------
tejohnson wrote:
> Can you add a check for noalias as well?
Sure - is there some trick to force a function to be marked as noalias?

I can't find any examples of noalias functions in any of the tests and this doesn't seem to work:
```
define noalias i32* @i() {
   %d = alloca i32;
   ret i32* %d;
}
```


https://reviews.llvm.org/D36233