[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