[PATCH] D124699: [DeadArgElim] Set unused arguments for internal functions

[Kirill Stoimenov via Phabricator via cfe-commits]

kstoimenov added a comment.

This change is breaking memory sanitizer in some cases. We observed that  when the argument is not actually removed the pass is dropping the noundef attribute. See the diff snippet below. Is that an intended behavior?

We would like to revert this as it is breaking MSan tests.

Thanks, 
Kirill

  < define internal %struct._object* @coro_wrapper_close(%struct.PyCoroWrapper* noundef %cw, %struct._object* noundef %args) #0 !dbg !1065 {
  ---
  > define internal %struct._object* @coro_wrapper_close(%struct.PyCoroWrapper* noundef %cw, %struct._object* %args) #0 !dbg !1065 {


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124699