[PATCH] D142860: [X86] Use autogenerated result for switch-bt.ll test; NFC

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 30 01:37:00 PST 2023


RKSimon added inline comments.


================
Comment at: llvm/test/CodeGen/X86/switch-bt.ll:14
 ;      CHECK: testq %rax, %r
 ; CHECK-NEXT: j
 
----------------
Don't these need removing as well? 


================
Comment at: llvm/test/CodeGen/X86/switch-bt.ll:258
 
 ; 73 = 2^0 + 2^3 + 2^6
 
----------------
can you move these helper comments someplace suitable (either in the switch statement for the 0/3/6 + 1/4/7 case groups or the test header comment)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142860



More information about the llvm-commits mailing list