Revert r330742: Let TableGen write output only if it changed, instead of doing so...
authorChandler Carruth <chandlerc@gmail.com>
Mon, 7 May 2018 23:41:48 +0000 (23:41 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Mon, 7 May 2018 23:41:48 +0000 (23:41 +0000)
commit029cea90fa4abe3beb1915094e47995d63fcb743
treebf25a3b5a2ad3893d4516b480ab665809e639ed4
parente2953dcbf702ab909d4c0fa154a7469e27246640
Revert r330742: Let TableGen write output only if it changed, instead of doing so in cmake.

This change causes us to re-run tablegen for every single target on
every single build. This is much, much worse than the problem being
fixed AFAICT.

On my system, it makes a clean rebuild of `llc` with nothing changed go
from .5s to over 8s. On systems with less parallelism, slower file
systems, or high process startup overhead this will be even more
extreme.

The only way I see this could be a win is in clean builds where we churn
the filesystem. But I think incremental rebuild is more important, and
so if we want to re-instate this, it needs to be done in a way that
doesn't trigger constant re-runs of tablegen.

llvm-svn: 331702
llvm/cmake/modules/TableGen.cmake
llvm/lib/TableGen/Main.cpp