[PATCH] D136476: llvm-reduce: Add flag reduction pass

John Regehr via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 21 11:46:10 PDT 2022


regehr added a comment.

In D136476#3875502 <https://reviews.llvm.org/D136476#3875502>, @arsenm wrote:

> My idea of simpler to understand is codegens to fewer instructions. Things that introduce undefined behavior are more likely to enable optimizations and different lowerings that reduce instructions emitted

right, adding UB enables more optimizations -- making the compiler execution harder to understand. I'll absolutely die on this hill but if you feel strongly enough in the other direction we should just add flags controlling this sort of thing. but my way is a better default.


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

https://reviews.llvm.org/D136476



More information about the llvm-commits mailing list