[PATCH] D148516: [TableGen] Emit separate computeRequiredFeatures() function

[Clement Courbet via Phabricator via llvm-commits]

courbet added inline comments.


================
Comment at: llvm/utils/TableGen/InstrInfoEmitter.cpp:748
   // Emit the available features compute function.
+  OS << "static ";
   SubtargetFeatureInfo::emitComputeAssemblerAvailableFeatures(
----------------
Won't this force emitting a copy in each compilation unit that includes the generated file ? Using `inline` instead would allow deduping the symbol.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148516