[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