[PATCH] D69482: [MachineOuliner][NFC] Refactoring code to make outline rerunning a cleaner diff.

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 09:04:52 PDT 2019


paquette added a comment.

I think the implementation of `doOutline` is missing here?



================
Comment at: llvm/lib/CodeGen/MachineOutliner.cpp:899
 
+  /// Calls doOutline.
+  bool runOnModule(Module &M) override;
----------------
I think that if you write this as `doOutline()`, Doxygen will automatically create a link to its entry on doOutline. That might be useful to people looking at the docs.

(Judging by http://www.doxygen.nl/manual/autolink.html)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69482





More information about the llvm-commits mailing list