[PATCH] D68049: Propeller: Clang options for basic block sections

Sriraman Tallam via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 11 18:23:25 PDT 2020


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


================
Comment at: lld/ELF/LTO.cpp:80
   // Check if basic block sections must be used.
   // Allowed values for --lto-basicblock-sections are "all", "labels",
   // "<file name specifying basic block ids>", or none.  This is the equivalent
----------------
shenhan wrote:
> Maybe "--lto-basicblock-sections" should be changed to "--lto-basic-block-sections" too?
Ok if I do this as a separate patch?  llc options need to renamed too so I clean up independently after this.


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

https://reviews.llvm.org/D68049





More information about the llvm-commits mailing list