[PATCH] D67342: [Attributor][Fix] Manifest nocapture only in CSArgument or Argument

Hideto Ueno via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 10 01:20:19 PDT 2019


uenoku added a comment.

In D67342#1663869 <https://reviews.llvm.org/D67342#1663869>, @jdoerfert wrote:

> Test missing.


Yes, test is missing but I think test is not necessary. If we query whether a callsite retured value is captured in the scope and the value has `nocapture`, this kind of error will be raised.

  Attributes 'byval', 'inalloca', 'nest', 'sret', 'nocapture', 'returned', 'swiftself', and 'swifterror' do not apply to return values!
    %B = tail call noalias nocapture i8* @malloc(i64 4)
  in function test12_1
  LLVM ERROR: Broken function found, compilation aborted!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67342/new/

https://reviews.llvm.org/D67342





More information about the llvm-commits mailing list