[PATCH] D121971: [AutoUpgrade] Don't lose attributes when upgrading mem intrinsics

[Nikita Popov via Phabricator via llvm-commits]

nikic added a reviewer: nikic.
nikic added inline comments.


================
Comment at: llvm/lib/IR/AutoUpgrade.cpp:4080
     NewCall = Builder.CreateCall(NewFn, Args);
+    NewCall->setAttributes(CI->getAttributes());
     auto *MemCI = cast<MemIntrinsic>(NewCall);
----------------
Don't you need to remove the attributes the the 4th argument (that is being dropped) from the attribute list?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121971