[PATCH] D93654: [TableGen] Change getAllDerivedDefinitions() to return an ArrayRef

Paul C. Anagnostopoulos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 23 13:59:14 PST 2020


Paul-C-Anagnostopoulos added a comment.

I'm not yet clever enough to do profiling. It just seemed like building large record vectors multiple times was a waste of time. Perhaps I embarked on this change without sufficient evidence of its necessity.

I will move the copy comments above the definition.


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

https://reviews.llvm.org/D93654



More information about the llvm-commits mailing list