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

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

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.

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


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

https://reviews.llvm.org/D61409