[PATCH] D121415: [FunctionAttrs] Infer argmemonly .
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 4 14:18:52 PDT 2022
fhahn added a comment.
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?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121415/new/
https://reviews.llvm.org/D121415
More information about the llvm-commits
mailing list