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

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson accepted this revision.
tejohnson added a comment.
This revision is now accepted and ready to land.

LGTM with one last question/nit. Do you have commit access and if not do you want me to submit for you? Do you have follow-on patches that use this about ready to go for review?



================
Comment at: test/Bitcode/thinlto-function-summary-functionattrs.ll:27
+define noalias i8* @i() {
+   %r = call noalias i8* @malloc(i32 1)
+   ret i8* %r
----------------
Is malloc call here necessary for test case?


https://reviews.llvm.org/D36233