[PATCH] D65479: [Bugpoint redesign] Added pass to reduce parameters

[David Blaikie via Phabricator via llvm-commits]

dblaikie added inline comments.


================
Comment at: llvm/test/Reduce/remove-args.ll:12-41
+define void @func(i32* dereferenceable(4) %used, i32 %unused, i32* dereferenceable(4) %unused2) {
+entry:
+  %used.addr = alloca i32*, align 8
+  %unused.addr = alloca i32, align 4
+  %unused2.addr = alloca i32*, align 8
+  store i32* %used, i32** %used.addr, align 8
+  store i32 %unused, i32* %unused.addr, align 4
----------------
Some comments describing what purpose each part of this IR covers would be good - I probably wouldn't expect 'printf' to be in this test - maybe that one is to test functions declared but not defined? Naming the function to indicate that and/or adding comments to describe it would be helpful. (similarly for the other functions - main+func+func2 - what purpose does each one serve?)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65479