[PATCH] D91786: [GVN] Strengthen the updating of dominated users

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 29 23:32:49 PST 2020


lebedev.ri added a comment.

In D91786#2406652 <https://reviews.llvm.org/D91786#2406652>, @lebedev.ri wrote:

> As of D90231 <https://reviews.llvm.org/D90231> i'm personally still unconvinced that it is the LLVM code that needs fixing and not the particular code that is using `__builtin_constant_p()`.

^ still am.
Are langref changes needed to justify these optimization changes?


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

https://reviews.llvm.org/D91786



More information about the llvm-commits mailing list