[PATCH] D43962: [GlobalISel][utils] Adding the init version of Instruction Select Testgen
Roman Tereshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 26 19:11:15 PDT 2018
rtereshin updated this revision to Diff 144261.
rtereshin added a comment.
Hi Daniel @dsanders
I've addressed all of the inline comments, I believe.
> We'll need to find a good way of handling immediates, C++ predicates, ComplexPattern and similar but that should definitely be left for later patches.
Yes.
> Some targets have a rather large number of rules (e.g. X86 is around 17k IIRC). Do we have a mechanism for keeping the number of generated tests to a reasonable level?
Not yet, but it's not a problem yet either as we import only a fraction of the rules as of now.
> with few function-level comments
+ @aemerson (Hi Amara)
> there's little documentation in the code of how this actually works under the hood. Can you add more descriptions so someone can follow what's happening when it comes to maintenance later, e.g. what's LiveInRA supposed to do, what are the pre-conditions and expected outputs of each phase of this tool?
All of this improved noticeably, I hope, but not everything is commented and I'll probably add on it later.
Thank you!
Roman
Repository:
rL LLVM
https://reviews.llvm.org/D43962
Files:
include/llvm/CodeGen/GlobalISel/InstructionSelectTestgen.h
include/llvm/CodeGen/GlobalISel/InstructionSelector.h
include/llvm/CodeGen/GlobalISel/InstructionSelectorImpl.h
include/llvm/CodeGen/GlobalISel/InstructionSelectorTestgen.h
include/llvm/InitializePasses.h
lib/CodeGen/GlobalISel/CMakeLists.txt
lib/CodeGen/GlobalISel/GlobalISel.cpp
lib/CodeGen/GlobalISel/InstructionSelectTestgen.cpp
lib/CodeGen/GlobalISel/InstructionSelectorTestgen.cpp
test/TableGen/GlobalISelEmitter.td
utils/TableGen/GlobalISelEmitter.cpp
utils/update_instruction_select_testgen_tests.sh
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D43962.144261.patch
Type: text/x-patch
Size: 64434 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180427/7b35182e/attachment.bin>
More information about the llvm-commits
mailing list