Nathan =?utf-8?q?Gauër?= <brio...@google.com>,
Nathan =?utf-8?q?Gauër?= <brio...@google.com>,
Nathan =?utf-8?q?Gauër?= <brio...@google.com>,
Nathan =?utf-8?q?Gauër?= <brio...@google.com>,
Nathan =?utf-8?q?Gauër?= <brio...@google.com>,
Nathan =?utf-8?q?Gauër?= <brio...@google.com>,
Nathan =?utf-8?q?Gauër?= <brio...@google.com>,
Nathan =?utf-8?q?Gauër?= <brio...@google.com>,
Nathan =?utf-8?q?Gauër?= <brio...@google.com>,
Nathan =?utf-8?q?Gauër?= <brio...@google.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/107...@github.com>


VyacheslavLevytskyy wrote:

That's really great, thank you.

> OpBranch/OpBranchConditional are not considered to be valid basic block 
> terminators
In lib/Target/SPIRV/SPIRVInstrInfo.td lines 625-632 we actually set 
isTerminator=1 for both of those, but let's double check then.

> ... leave the rest unfixed in this PR (which is already huge)
Sure!

https://github.com/llvm/llvm-project/pull/107408
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to