[PATCH] D136560: llvm-reduce: Try to turn calls into something else

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added inline comments.


================
Comment at: llvm/tools/llvm-reduce/deltas/ReduceOpcodes.cpp:139
+  if (DataArg)
+    return B.CreateStore(DataArg, PtrArg, true);
+  return B.CreateLoad(CB->getType(), PtrArg, true);
----------------
why volatile?


================
Comment at: llvm/tools/llvm-reduce/deltas/ReduceOpcodes.cpp:194
+    case 3:
+      return B.CreateIntrinsic(Intrinsic::fshl, {CB->getType()}, Arguments);
+    default:
----------------
IMO a call to an intrinsic is less reduced than a call to some arbitrary external function.
Perhaps it's finally time to create a flag for reducing IR optimizing for codegen?


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

https://reviews.llvm.org/D136560