[PATCH] D138525: llvm-reduce: Reduce ifuncs

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 23 09:28:03 PST 2022


aeubanks added inline comments.


================
Comment at: llvm/test/tools/llvm-reduce/remove-ifunc.ll:62
+define i32 @fn5(double %arg) {
+  %call = call i32 bitcast (float(i64)* @ifunc6 to float(i64)*)(double %arg)
+  ret i32 %call
----------------
arsenm wrote:
> aeubanks wrote:
> > mixing `ptr` and `*` will just end up using opaque pointers everywhere, so this part is pointless
> I'm constantly wondering if I should be trying to write code that bothers trying to work with typed pointers 
I wouldn't worry about it, typed pointers are deprecated and soon to be removed


================
Comment at: llvm/tools/llvm-reduce/deltas/ReduceAliases.cpp:36
+      Constant *Resolver = GI.getResolver();
+      GI.replaceAllUsesWith(Resolver);
+      GI.eraseFromParent();
----------------
arsenm wrote:
> I'm wondering if it would be better to lower this by emitting a call to the resolver, plus the indirect call of the result
that does make more sense


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

https://reviews.llvm.org/D138525



More information about the llvm-commits mailing list