[PATCH] D45372: [AMDGPU] Fix issues for backend divergence tracking
Alexander via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 16 08:01:58 PDT 2018
alex-t added a comment.
In https://reviews.llvm.org/D45372#1068728, @dstuttard wrote:
>
> 2. Remove the test altogether since the VirtReg2Value.clear() is definitely required and was originally missed as an oversight - and the test might be overkill.
Sounds reasonable. It really was an oversight :) It's funny that the change was reviewed by lots of people for 2 months but nobody noticed not clearing state.
Repository:
rL LLVM
https://reviews.llvm.org/D45372
More information about the llvm-commits
mailing list