[all-commits] [llvm/llvm-project] c8ffc7: [PartiallyInlineLibCalls] Don't crash when there's...

[Benjamin Kramer via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c8ffc73350dbb6044ca947bbead127b9b914cdf3
      https://github.com/llvm/llvm-project/commit/c8ffc73350dbb6044ca947bbead127b9b914cdf3
  Author: Benjamin Kramer <benny.kra at googlemail.com>
  Date:   2022-01-05 (Wed, 05 Jan 2022)

  Changed paths:
    M llvm/lib/Transforms/Scalar/PartiallyInlineLibCalls.cpp
    M llvm/test/Transforms/PartiallyInlineLibCalls/X86/good-prototype.ll

  Log Message:
  -----------
  [PartiallyInlineLibCalls] Don't crash when there's a writeonly attribute on the call

readnone subsumes writeonly, so just swap out the attributes. The
verifier doesn't allow us to have both on a call.


  Commit: 5f0a349738304caf5f8083166f785a91048f574d
      https://github.com/llvm/llvm-project/commit/5f0a349738304caf5f8083166f785a91048f574d
  Author: Benjamin Kramer <benny.kra at googlemail.com>
  Date:   2022-01-05 (Wed, 05 Jan 2022)

  Changed paths:
    M llvm/lib/Transforms/Utils/BuildLibCalls.cpp
    M llvm/test/Transforms/InferFunctionAttrs/annotate.ll
    A llvm/test/Transforms/InferFunctionAttrs/readonly_and_writeonly.ll

  Log Message:
  -----------
  Revert "Revert "[InferAttrs] Add writeonly to all the math functions""

This reverts commit 29b6e967f3e99ac45340ea37a70262c70e4e7528. The bug it
found in PartiallyInlineLibCalls was fixed in
c8ffc73350dbb6044ca947bbead127b9b914cdf3.


Compare: https://github.com/llvm/llvm-project/compare/96e220e68868...5f0a34973830