[all-commits] [llvm/llvm-project] b8f9ec: precommit test for pr60855

Nick Desaulniers via All-commits all-commits at lists.llvm.org
Wed Mar 1 15:27:45 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b8f9ec6db77353fbcb2384fa60e4ca9a6d4e3245
      https://github.com/llvm/llvm-project/commit/b8f9ec6db77353fbcb2384fa60e4ca9a6d4e3245
  Author: Nick Desaulniers <ndesaulniers at google.com>
  Date:   2023-03-01 (Wed, 01 Mar 2023)

  Changed paths:
    A llvm/test/CodeGen/X86/callbr-asm-outputs-regallocfast.mir

  Log Message:
  -----------
  precommit test for pr60855

This test demonstrates an issue with callbr outputs being used along
indirect edges when using regallocfast.

Link: https://github.com/llvm/llvm-project/issues/60855

Differential Revision: https://reviews.llvm.org/D144906


  Commit: 9cec2b246e719533723562950e56c292fe5dd5ad
      https://github.com/llvm/llvm-project/commit/9cec2b246e719533723562950e56c292fe5dd5ad
  Author: Nick Desaulniers <ndesaulniers at google.com>
  Date:   2023-03-01 (Wed, 01 Mar 2023)

  Changed paths:
    M llvm/lib/CodeGen/RegAllocFast.cpp
    M llvm/test/CodeGen/X86/callbr-asm-outputs-regallocfast.mir

  Log Message:
  -----------
  [RegAllocFast] insert additional spills along indirect edges of INLINEASM_BR

When generating spills (stores) for values produced by INLINEASM_BR
instructions, make sure to insert one spill per indirect target.
Otherwise the reload generated may load from a stack slot that has not
yet been stored to (resulting in a load of an uninitialized stack slot).

Link: https://github.com/llvm/llvm-project/issues/53562
Fixes: https://github.com/llvm/llvm-project/issues/60855

Reviewed By: MatzeB

Differential Revision: https://reviews.llvm.org/D144907


Compare: https://github.com/llvm/llvm-project/compare/a7f8b7cd8e49...9cec2b246e71


More information about the All-commits mailing list