[PATCH] D73674: Propeller: LLVM support for basic block sections (Base Patch - Part 2)

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 14 16:39:49 PST 2020


efriedma added a comment.

In D73674#1877420 <https://reviews.llvm.org/D73674#1877420>, @tmsriram wrote:

> Apologies, I misread the original question.  Yes, in cases where exception blocks are hot, it does not help placing them along with cold blocks.  One of the SPEC benchmarks had a performance improvement doing this.


Oh, okay.  And I guess you can't emit multiple exception sections due to some arcane exception-handling stuff? (I remember you mentioned this at some point, but I don't remember the details.)  Therefore the compromise is emitting them separately from the cold blocks?  I guess that's reasonable.


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

https://reviews.llvm.org/D73674





More information about the llvm-commits mailing list