[PATCH] D62625: LFTR for multiple exit loops

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo added a comment.

In D62625#1552791 <https://reviews.llvm.org/D62625#1552791>, @reames wrote:

> In D62625#1552681 <https://reviews.llvm.org/D62625#1552681>, @mstorsjo wrote:
>
> > I just ran into a performance regression due to this commit. With https://martin.st/temp/glew-preproc-i686.c, compiling with `clang -target i686-w64-mingw32 -c -O3 glew-preproc-i686.c` went from 73 seconds before this commit, to 133 seconds afterwards.
>
>
> Clarification here.  Do you mean this was a compile time regression?  Or a runtime regression?  (i.e. is your clang here build with a different compiler?  Or is this a self host?)


Compile time regression. (I build the project it's part of with a 120 second timeout per translation unit, enforced with `ulimit -t 120`, that's why I noticed.)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62625