[PATCH] D16835: InstCombine: Restrict computeKnownBits() on everyValue to OptLevel > 2

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 2 18:40:20 PST 2016


joker.eph added a comment.

Two questions on the concept:

- Are we sure that a boolean is enough? (per comparison to a "level", like Opt_level).
- The trigger between expensive combine or not could (should?) be adjusted depending on where we stand in the pipeline (I mean not always turning it on or off depending on the opt level, but also depending on the position in the pipeline)


Repository:
  rL LLVM

http://reviews.llvm.org/D16835





More information about the llvm-commits mailing list