[PATCH] D41638: [llvm-extract] Support extracting basic blocks

[Justin Bogner via Phabricator via llvm-commits]

bogner added inline comments.


================
Comment at: tools/llvm-extract/llvm-extract.cpp:321-322
 
+  // Extract the specified basic blocks from the module and erase the existing
+  // functions.
+  if (!ExtractBlocks.empty()) {
----------------
On interesting side effect of the "-bb function:block" approach is that we'll probably do something kind of strange if someone calls this like "llvm-extract -func f0 -bb f0:bb1" - will the result be functions f0 and f0_bb1, or will f0 be deleted and we only get f0_bb1? I suspect the latter. This is probably fine though - it seems unlikely that someone would really want to do that.



https://reviews.llvm.org/D41638