[PATCH] D121415: [FunctionAttrs] Infer argmemonly .

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

akdandrea added a comment.

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

> In D121415#3411555 <https://reviews.llvm.org/D121415#3411555>, @akdandrea wrote:
>
>> In D121415#3411543 <https://reviews.llvm.org/D121415#3411543>, @fhahn wrote:
>>
>>> In D121415#3411483 <https://reviews.llvm.org/D121415#3411483>, @akdandrea wrote:
>>>
>>>> Ah that's interesting that you couldn't reproduce it on x86.  I did this test on a RHEL 8.4 Power system.  Here was how I compiled and ran the test:
>>>>
>>>>   clang  -flto    -O3 -DNDEBUG   -w -Werror=date-time -ffp-contract=off -ffp-contract=off -DFMA_DISABLED=1 -ffp-contract=off -DFMA_DISABLED=1 -o linpack-pc.c.o   -c test-suite/SingleSource/Benchmarks/Linpack/linpack-pc.c
>>>>   clang -flto    -O3 -DNDEBUG      linpack-pc.c.o  -o linpack-pc  -lm
>>>
>>> I am unable to build for PowerPC. Could you share `  linpack-pc.c.o `?
>>
>> Sure, here it is:
>> F22579002: linpack-pc.c.o <https://reviews.llvm.org/F22579002>
>
> 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>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121415