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

Paul C. Anagnostopoulos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 8 09:10:25 PDT 2020


Paul-C-Anagnostopoulos updated this revision to Diff 296985.
Paul-C-Anagnostopoulos added a comment.

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.


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

https://reviews.llvm.org/D88832

Files:
  llvm/docs/TableGen/BackGuide.rst
  llvm/include/llvm/TableGen/Record.h
  llvm/lib/TableGen/Record.cpp
  llvm/utils/TableGen/PseudoLoweringEmitter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88832.296985.patch
Type: text/x-patch
Size: 6404 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201008/eaa312b8/attachment.bin>


More information about the llvm-commits mailing list