[PATCH] D82761: SpeculativeExecution: Fix for logic change introduced in D81730.
Daniil Fukalov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 30 07:00:59 PDT 2020
dfukalov marked 2 inline comments as done.
dfukalov added inline comments.
================
Comment at: llvm/test/Transforms/SpeculativeExecution/PR46267.ll:6
+
+define i8* @_Z9void_castP1B(%class.B* readonly %b) {
+; CHECK-LABEL: _Z9void_castP1B
----------------
arsenm wrote:
> I don't understand what this is checking. Can you ad a comment and/or rename the function?
This is a reduced buildbot testcase started to fail after D81730. The change started to hoist two bitcasts to upper BB although it was not intended in fix for PR46267.
This test checks that bitscasts stayed in their BB.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82761/new/
https://reviews.llvm.org/D82761
More information about the llvm-commits
mailing list