[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