[PATCH] D61409: [SimplifyCFG] Added condition assumption for unreachable blocks

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 2 00:53:12 PDT 2019


lebedev.ri added a comment.

In D61409#1487366 <https://reviews.llvm.org/D61409#1487366>, @nikic wrote:

> This is a good idea in theory, but I'm somewhat concerned that this will backfire in practice because assumes often pessimize optimization. For example LLVM has a really bad offender where it inserts assumes for alignment annotations during inlining -- thankfully there's an option to disable it, because it's a major perf regression otherwise.


Pessimization of the final produced code, or of the llvm itself?

> This //feels// like it should be less problematic though.




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

https://reviews.llvm.org/D61409





More information about the llvm-commits mailing list