[PATCH] D85023: [JumpThreading] Consider freeze as a zero-cost instruction

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 3 13:54:42 PDT 2020


efriedma added a comment.

Can we combine all the thread-two-bbs tests into one file, instead of keeping them in separate numbered files?



================
Comment at: llvm/test/Transforms/JumpThreading/thread-two-bbs7.ll:9
 
-; FIXME: show that freeze has no cost
-
 define void @foo(i32 %cond1, i32 %cond2) {
 ; CHECK-LABEL: @foo(
----------------
Please add a comment explaining what this is testing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85023



More information about the llvm-commits mailing list