[PATCH] D121268: [AMDGPU] Control flow pseudos are not part of block prologue
Ruiling, Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 8 19:37:24 PST 2022
ruiling added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/SIInstrInfo.cpp:7604
+ return false;
+ switch (MI.getOpcode()) {
+ case AMDGPU::COPY:
----------------
I think SI_IF_BREAK can be removed from the list as it does not modifies EXEC. SI_END_CF should be removed from the list, as SI_END_CF means the join point, we should treat it as block prologue. Other instructions should never be prologue instructions, so it is correct to list them here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121268/new/
https://reviews.llvm.org/D121268
More information about the llvm-commits
mailing list