[PATCH] D50633: [AMDGPU] Add new Mode Register pass
Tim Corringham via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 30 04:21:39 PST 2018
timcorringham marked 2 inline comments as done.
timcorringham added inline comments.
================
Comment at: lib/Target/AMDGPU/SIModeRegister.cpp:243-246
+ if (NewInfo->Change.isCompatible(InstrMode)) {
+ // This instruction is compatible, so just merge the mode
+ NewInfo->Change = NewInfo->Change.merge(InstrMode);
+ } else {
----------------
nhaehnle wrote:
> nhaehnle wrote:
> > Hasn't this become redundant now?
> Okay, no, I see how it isn't redundant if there's a pre-existing s_setreg.
>
> But why the call to merge? isCompatible only returns true if InstrMode is a subset of the known bits of NewInfo->Change.
Good spot- this is now redundant. It is benign but unnecessary - I'll remove it.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D50633/new/
https://reviews.llvm.org/D50633
More information about the llvm-commits
mailing list