[PATCH] D121944: [LICM] Add allowspeculation pass options.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 18 03:52:42 PDT 2022


fhahn marked an inline comment as done.
fhahn added inline comments.


================
Comment at: llvm/include/llvm/Transforms/Scalar/LICM.h:74
+                             AllowSpeculation)) {}
+  LICMPass(LICMOptions Opts) : Opts(Opts) {}
+
----------------
aeubanks wrote:
> should also implement
> ```
> void printPipeline(raw_ostream &OS,
>                      function_ref<StringRef(StringRef)> MapClassName2PassName);
> ```
> 
> with a corresponding test in `llvm/test/Other/new-pm-print-pipeline.ll`
Done, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121944



More information about the llvm-commits mailing list