[PATCH] D70450: [AArch64] Teach Load/Store optimizier to rename store operands for pairing.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 11 08:18:09 PST 2019


fhahn added a comment.

In D70450#1779756 <https://reviews.llvm.org/D70450#1779756>, @phosek wrote:

> We're seeing an assertion error in Clang when compiling compiler-rt builtins for aarch64-linux-gnu with this change:
>
>   FAILED: CMakeFiles/clang_rt.builtins-aarch64.dir/multc3.c.o 
>   /b/s/w/ir/k/recipe_cleanup/clanga0lNQB/llvm_build_dir/./bin/clang --target=aarch64-unknown-linux-gnu --sysroot=/b/s/w/ir/k/cipd/linux-arm64 -DVISIBILITY_HIDDEN  -O2 -g -DNDEBUG    -std=c11 -fPIC -fno-builtin -fvisibility=hidden -fomit-frame-pointer -MD -MT CMakeFiles/clang_rt.builtins-aarch64.dir/multc3.c.o -MF CMakeFiles/clang_rt.builtins-aarch64.dir/multc3.c.o.d -o CMakeFiles/clang_rt.builtins-aarch64.dir/multc3.c.o   -c /b/s/w/ir/k/llvm-project/compiler-rt/lib/builtins/multc3.c
>   clang-10: /b/s/w/ir/k/llvm-project/llvm/include/llvm/MC/MCRegisterInfo.h:677: llvm::MCRegUnitIterator::MCRegUnitIterator(llvm::MCRegister, const llvm::MCRegisterInfo *): Assertion `Reg && "Null register has no regunits"' failed.
>   clang-10: error: unable to execute command: Aborted
>   clang-10: error: clang frontend command failed due to signal (use -v to see invocation)
>


Looks like the patch missed to skip debug operands in an assertion. I will push a fix shortly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70450





More information about the llvm-commits mailing list