[PATCH] D45541: [globalisel] Update GlobalISel emitter to match new representation of extending loads

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 8 09:54:30 PDT 2018


aemerson added a comment.

In https://reviews.llvm.org/D45541#1091441, @rtereshin wrote:

> Hi Amara,
>
> > I think we lost 0.5% on compile time on CTMark because of this. Is there any scope left here for performance improvements? With the addition of the combiner landing later, I expect a further CT hit so anything we can do here would be good.
>
> Just a quick question: 0.5% loss on the entire compile time or only some part of the pipeline?


The 0.5% is a regression is relative to fast-isel on CTMark. I.e.  we were 6.8% behind fast-isel, now we are 7.3%. On individual benchmarks though:

- 1.87% on lencod
- 1.68% on pairlocalalign
- 1.68% on sqlite3
- 1.16% on tramp3d
- 1.01% on consumer-typeset


Repository:
  rL LLVM

https://reviews.llvm.org/D45541





More information about the llvm-commits mailing list