Riyaz Puthiyapurayil via llvm-dev
2021-Feb-23 01:51 UTC
[llvm-dev] How to eliminate a redundant alloca, load and store?
In the following code, I know that the alloca, load and store are actually
unnecessary (it is safe to pass @foo directly to the function boo). Are there
any attributes one can set on function `boo` AND/OR its argument pointer AND/OR
on the call so that the three instructions can be eliminated and the argument of
call @boo be replaced with @foo (is there a transform that can actually do this
today?)
; ModuleID = '<stdin>'
source_filename = "test.c"
target datalayout =
"e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128"
target triple = "x86_64-unknown-linux-gnu"
@foo = common dso_local local_unnamed_addr global i32 0, align 4
; Function Attrs: noinline nounwind uwtable
define dso_local void @f() local_unnamed_addr #0 {
%A = alloca i32, align 4
%L = load i32, i32* @foo, align 4
store i32 %L, i32* %A, align 4
call void @boo(i32* nonnull %A) #2
ret void
}
declare dso_local void @boo(i32* readonly noalias nocapture) local_unnamed_addr
argmemonly #1
attributes #0 = { noinline nounwind uwtable
"correctly-rounded-divide-sqrt-fp-math"="false"
"disable-tail-calls"="false"
"frame-pointer"="all"
"less-precise-fpmad"="false"
"min-legal-vector-width"="0"
"no-infs-fp-math"="false"
"no-jump-tables"="false"
"no-nans-fp-math"="false"
"no-signed-zeros-fp-math"="false"
"no-trapping-math"="false"
"stack-protector-buffer-size"="8"
"target-cpu"="x86-64"
"target-features"="+cx8,+fxsr,+mmx,+sse,+sse2,+x87"
"unsafe-fp-math"="false"
"use-soft-float"="false" }
attributes #1 = {
"correctly-rounded-divide-sqrt-fp-math"="false"
"disable-tail-calls"="false"
"frame-pointer"="all"
"less-precise-fpmad"="false"
"no-infs-fp-math"="false"
"no-nans-fp-math"="false"
"no-signed-zeros-fp-math"="false"
"no-trapping-math"="false"
"stack-protector-buffer-size"="8"
"target-cpu"="x86-64"
"target-features"="+cx8,+fxsr,+mmx,+sse,+sse2,+x87"
"unsafe-fp-math"="false"
"use-soft-float"="false" }
attributes #2 = { nounwind }
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20210223/efce4710/attachment.html>
Johannes Doerfert via llvm-dev
2021-Feb-23 02:45 UTC
[llvm-dev] How to eliminate a redundant alloca, load and store?
[This time with list]
Short of making `@foo` private I cannot see a way right now.
Imagine:
```c
int was_called_with_foo;
void boo(int *A) { was_called_with_foo |= (A == &foo); }
```
which captures the fact you used @foo or not.
~ Johannes
On 2/22/21 7:51 PM, Riyaz Puthiyapurayil via llvm-dev
wrote:> In the following code, I know that the alloca, load and store are actually
unnecessary (it is safe to pass @foo directly to the function boo). Are there
any attributes one can set on function `boo` AND/OR its argument pointer AND/OR
on the call so that the three instructions can be eliminated and the argument of
call @boo be replaced with @foo (is there a transform that can actually do this
today?)
>
> ; ModuleID = '<stdin>'
> source_filename = "test.c"
> target datalayout =
"e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128"
> target triple = "x86_64-unknown-linux-gnu"
>
> @foo = common dso_local local_unnamed_addr global i32 0, align 4
>
> ; Function Attrs: noinline nounwind uwtable
> define dso_local void @f() local_unnamed_addr #0 {
> %A = alloca i32, align 4
> %L = load i32, i32* @foo, align 4
> store i32 %L, i32* %A, align 4
> call void @boo(i32* nonnull %A) #2
> ret void
> }
>
> declare dso_local void @boo(i32* readonly noalias nocapture)
local_unnamed_addr argmemonly #1
>
> attributes #0 = { noinline nounwind uwtable
"correctly-rounded-divide-sqrt-fp-math"="false"
"disable-tail-calls"="false"
"frame-pointer"="all"
"less-precise-fpmad"="false"
"min-legal-vector-width"="0"
"no-infs-fp-math"="false"
"no-jump-tables"="false"
"no-nans-fp-math"="false"
"no-signed-zeros-fp-math"="false"
"no-trapping-math"="false"
"stack-protector-buffer-size"="8"
"target-cpu"="x86-64"
"target-features"="+cx8,+fxsr,+mmx,+sse,+sse2,+x87"
"unsafe-fp-math"="false"
"use-soft-float"="false" }
> attributes #1 = {
"correctly-rounded-divide-sqrt-fp-math"="false"
"disable-tail-calls"="false"
"frame-pointer"="all"
"less-precise-fpmad"="false"
"no-infs-fp-math"="false"
"no-nans-fp-math"="false"
"no-signed-zeros-fp-math"="false"
"no-trapping-math"="false"
"stack-protector-buffer-size"="8"
"target-cpu"="x86-64"
"target-features"="+cx8,+fxsr,+mmx,+sse,+sse2,+x87"
"unsafe-fp-math"="false"
"use-soft-float"="false" }
> attributes #2 = { nounwind }
>
>
> _______________________________________________
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev