aboutsummaryrefslogtreecommitdiff
path: root/clang/lib/Sema/DeclSpec.cpp
diff options
context:
space:
mode:
authorMatt Arsenault <Matthew.Arsenault@amd.com>2023-07-26 11:35:21 -0400
committerMatt Arsenault <arsenm2@gmail.com>2023-10-02 15:16:40 +0300
commitbc7d88faf1a595ab59952a2054418cdd0d9eeee8 (patch)
tree2f1009bd85329f7bbb1ec26ab99778773049b8d7 /clang/lib/Sema/DeclSpec.cpp
parente9b33d085da05da6ce943dc563134f5ba91ca35b (diff)
downloadllvm-bc7d88faf1a595ab59952a2054418cdd0d9eeee8.zip
llvm-bc7d88faf1a595ab59952a2054418cdd0d9eeee8.tar.gz
llvm-bc7d88faf1a595ab59952a2054418cdd0d9eeee8.tar.bz2
CodeGen: Disable isCopyInstrImpl if there are implicit operands
This is a conservative workaround for broken liveness tracking of SUBREG_TO_REG to speculatively fix all targets. The current reported failures are on X86 only, but this issue should appear for all targets that use SUBREG_TO_REG. The next minimally correct refinement would be to disallow only implicit defs. The coalescer now introduces implicit-defs of the super register to track the dependency on other subregisters. If we see such an implicit operand, we cannot simply treat the subregister def as the result operand in case downstream users depend on the implicitly defined parts. Really target implementations should be considering the implicit defs and trying to interpret them appropriately (maybe with some generic helpers). The full implicit def could possibly be reported as the move result, rather than the subregister def but that requires additional work. Hopefully fixes #64060 as well. This needs to be applied to the release branch. https://reviews.llvm.org/D156346
Diffstat (limited to 'clang/lib/Sema/DeclSpec.cpp')
0 files changed, 0 insertions, 0 deletions