[PATCH] D80957: [LiveDebugValues] Cache LexicalScopes::getMachineBasicBlocks, NFCI

[Orlando Cazalet-Hyams via Phabricator via llvm-commits]

Orlando added inline comments.


================
Comment at: llvm/lib/CodeGen/LexicalScopes.cpp:328
+  return Set->count(MBB) != 0;
 }
 
----------------
I'm not familiar with the extent of the usage of `LexicalScopes` throughout llvm, so I wonder if there's ever a case where we don't want the caching behaviour. i.e. is it conceivable that the result of `dominates` should change for the same inputs between calls?

Given this unfamiliarity I probably shouldn't be the one to give the final LGTM on this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80957