[PATCH] D29866: [PDSE] Add PDSE.

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added inline comments.


================
Comment at: lib/Transforms/IPO/PassManagerBuilder.cpp:78
+static cl::opt<bool> RunPDSE("enable-pdse", cl::init(false), cl::Hidden,
+                             cl::desc("Run with PDSE instead of DSE."));
+
----------------
Is "PDSE" supposed to be a well known acronym? I feel that the description could be more friendly with `--help` readers.

It also sounds strange to me that "Partial DSE" is "stronger" than "DSE" alone.


Repository:
  rL LLVM

https://reviews.llvm.org/D29866