[PATCH] D144652: [Assignment Tracking] getIntrinsicInstrCost: set dbg.assign cost to zero

Orlando Cazalet-Hyams via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 23 09:12:12 PST 2023


Orlando added a subscriber: jryans.
Orlando added a comment.

Note that `dbg_addr` is also missing from this list. `dbg_addr` is in the process of being removed (cc @jryans) but maybe it's worth fixing in the mean time, since it's such a tiny fix? YMMV.

The result of having `dbg_addr` and `dbg_assign` missing from the list is that they're considered to cost `1` rather than `0`, which changes optimisation decisions (unrolling, peeling, inlining, etc).


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

https://reviews.llvm.org/D144652



More information about the llvm-commits mailing list