[PATCH] D92722: [TableGen] [CodeGenTarget] Cache the target's instruction namespace

[David Blaikie via Phabricator via llvm-commits]

dblaikie added a comment.

Is there much data to suggest this is an observable performance issue? (if the number of times it's queried is small, or the number of elements it searches through is small, etc - perhaps it doesn't matter & the simpler code might be better?)



================
Comment at: llvm/utils/TableGen/CodeGenTarget.h:63
 
+  mutable StringRef InstNamespace = StringRef();
   mutable std::vector<const CodeGenInstruction*> InstrsByEnum;
----------------
The ` = StringRef()` is redundant here and should be omitted.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92722