[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