[PATCH] D81632: Fix undefined behavior in PeepholeOptimizer.

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 8 09:35:53 PST 2021


MatzeB added a comment.

In D81632#2092363 <https://reviews.llvm.org/D81632#2092363>, @linzj wrote:

>> I think SubIdx is supposed to be set by TII->isCoalescableExtInstr(). If it's not doing that, then the error's probably in that call and not in this function.
>
> Then most target instr infos need to patch.

`SubIdx` only needs to be set the function ends up doing `return true;`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81632



More information about the llvm-commits mailing list