[PATCH] D88832: [TableGen] Add new getAllDerivedDefinitionsTwo function to RecordKeeper

[Jessica Clarke via Phabricator via llvm-commits]

jrtc27 added a comment.

In D88832#2319576 <https://reviews.llvm.org/D88832#2319576>, @Paul-C-Anagnostopoulos wrote:

> Now the two getAllDerivedDefinitions() functions use StringRef and ArrayRef.
>
> I cleaned up the lints and removed RISCVCompressInstEmitter.cpp from this patch.
>
> I could not use {param} to call the ArrayRef<StringRef> overload; it resulted in an "all paths are infinitely recursive" error.

I guess the single-element initialiser list competes with one of StringRef's own constructors then.


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

https://reviews.llvm.org/D88832