[PATCH] D24548: Add AArch64 unit tests

Diana Picus via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 12 02:10:34 PDT 2016


rovka added a comment.

In https://reviews.llvm.org/D24548#567443, @MatzeB wrote:

> In general though these programmatic unit tests are harder to understand and more work to maintain than (well done) tests using one of the llvm tool. Of course to do this effectively we need a means to print/test the information/functionality we are interested in and right now there is no way to print the computed size next to each instruction or similar.


Would it be ok to add a pass that annotates the MIR with the instruction sizes (as comments) just so we can have the usual kind of tests with FileCheck? Just wondering, I'm definitely leaving that for the next person that has to deal with an instruction-size related bug :)

Thanks for reviewing.


https://reviews.llvm.org/D24548





More information about the llvm-commits mailing list