[PATCH] D52367: Remove address taken, add optnone

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 18 13:09:13 PDT 2020


dblaikie added inline comments.
Herald added a project: LLVM.


================
Comment at: test/Transforms/HotColdSplit/split-cold-1.ll:29
+; CHECK-NOT: codeRepl:
+define void @bar() noinline optnone {
+entry:
----------------
I'm guessing if this is marked "noinline" then this test wouldn't fail with the patch not applied? (because it'd be caught by the noinline case a few lines later - producing the same behavior)

With the "noinline" removed, does this test fail without the patch applied?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D52367/new/

https://reviews.llvm.org/D52367



More information about the llvm-commits mailing list