[PATCH] D121415: [FunctionAttrs] Infer argmemonly .

[Adam D'Andrea via Phabricator via llvm-commits]

akdandrea added a comment.

In D121415#3427704 <https://reviews.llvm.org/D121415#3427704>, @fhahn wrote:

> In D121415#3412110 <https://reviews.llvm.org/D121415#3412110>, @akdandrea wrote:
>
>> snip
>>
>>> Looks like the issue is already happening when preparing for LTO. Could you also share the original input IR, before optimizations (passing `-emit-llvm -mllvm -disable-llvm-optzns` to the first clang invocation)?
>>
>> Sure, here is the LLVM IR generated:
>> F22580860: linpack-pc.ll <https://reviews.llvm.org/F22580860>
>
> Thanks for sharing the IR. There are a couple of places where we now infer `argmemonly`, but they seem to be correctly marked as argmemonly, so I suspect some other optimization may be causing the issue. Do you happen to know which function is getting miscompiled?

Just to close the loop on this, it appears that I can now get this to pass with LLVM ToT on Power.  So, I don't think there is anything further investigation required at this time.  I appreciate your help on this, thanks again


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121415