[PATCH] D149207: [llvm-exegesis] Create a proper LLVM IR Function for MachineFunctions

Aiden Grossman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 30 16:12:21 PDT 2023


aidengrossman added inline comments.


================
Comment at: llvm/tools/llvm-exegesis/lib/Assembler.cpp:88
   // contains no IR.
   F->setIsMaterializable(true);
+  BasicBlock *BB = BasicBlock::Create(Module->getContext(), "", F);
----------------
I'm thinking we should be able to get rid of this comment and function call now that there will be IR within the function?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149207



More information about the llvm-commits mailing list