[all-commits] [llvm/llvm-project] f209e8: [CIR] Don't delete 'sub' op during 'convertToIndex...

[Erich Keane via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f209e827084a99d284abdb534d1f6ca3e68fc039
      https://github.com/llvm/llvm-project/commit/f209e827084a99d284abdb534d1f6ca3e68fc039
  Author: Erich Keane <ekeane at nvidia.com>
  Date:   2026-09-22 (Tue, 22 Sep 2026)

  Changed paths:
    M clang/lib/CIR/Lowering/DirectToLLVM/LowerToLLVM.cpp
    A clang/test/CIR/Lowering/ptr-stride-index-width.cir
    A clang/test/CIR/Lowering/ptr-stride-index-width.cpp

  Log Message:
  -----------
  [CIR] Don't delete 'sub' op during 'convertToIndexTy' (#225231)

This showed up in a SPEC benchmark: IF that sub operation is still being
used, deleting it causes us to orphan its other uses, causing a
conversion failure. We can't actually delete it, as we have no way of
knowing whether it has other uses (since we're in the process of
    removing at least one of its uses?). SO this patch is going to just
count on dead-code elemination to remove this sub after we stop using
its value, if at all possible.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications