[all-commits] [llvm/llvm-project] 1fb415: [AMDGPU][FIX] Proper load-store-vectorizer result ...

Johannes Doerfert via All-commits all-commits at lists.llvm.org
Fri Apr 15 11:44:32 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1fb415fee98efff99188507da2eedaf780ff2bab
      https://github.com/llvm/llvm-project/commit/1fb415fee98efff99188507da2eedaf780ff2bab
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2022-04-15 (Fri, 15 Apr 2022)

  Changed paths:
    M llvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp
    A llvm/test/Transforms/LoadStoreVectorizer/AMDGPU/opaque_ptr.ll

  Log Message:
  -----------
  [AMDGPU][FIX] Proper load-store-vectorizer result with opaque pointers

The original code relied on the fact that we needed a bitcast
instruction (for non constant base objects). With opaque pointers there
might not be a bitcast. Always check if reordering is required instead.

Fixes: https://github.com/llvm/llvm-project/issues/54896

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


  Commit: 3f7a6ce0de804a27954706d3a2fc2aef0c9554b7
      https://github.com/llvm/llvm-project/commit/3f7a6ce0de804a27954706d3a2fc2aef0c9554b7
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2022-04-15 (Fri, 15 Apr 2022)

  Changed paths:
    M llvm/lib/CodeGen/AsmPrinter/DwarfExpression.cpp
    A llvm/test/CodeGen/AMDGPU/dwarf-multi-register-use-crash.ll

  Log Message:
  -----------
  [DWARF][FIX] Handle the use of multiple registers gracefully

Certain applications crashed for us with the AMDGPU backend. While this
is not a proper fix it allows us to compile the code for now. I left a
TODO for someone that understands DWARF.

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


Compare: https://github.com/llvm/llvm-project/compare/9a8bb4bc635d...3f7a6ce0de80


More information about the All-commits mailing list