[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