[PATCH] D51089: [LLD] [COFF] Add support for creating range extension thunks for ARM

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 13 09:10:18 PDT 2018


mstorsjo added a comment.

In https://reviews.llvm.org/D51089#1232372, @mstorsjo wrote:

> Since using a margin for adding thunks, I think it will be extremely tedious to actually create a testcase which would trigger more than one pass.


In order to get sensible test coverage, I could make the margin (used only in the first pass) configurable (somehow), and run the multipass test with a very small margin.

Even with the other approach suggested by @ruiu, testing of the case when one pass isn't enough would require a huge, pathological test.


https://reviews.llvm.org/D51089





More information about the llvm-commits mailing list