aeubanks added a comment.

I'm looking at enabling the -enable-npm-optnone flag and FullUnroll.ll fails. I 
understand that loop unrolling should be forced when some metadata is present, 
but the FullUnroll.ll test seems to check for a lot more than that. It checks 
for (roughly) two unconditional branches and no conditional branches. Running 
the test under -debug-pass-manager and --print-after-all, , it looks like 
InstCombine does a lot of work removing instructions, and SimplifyCFG is 
ultimately what removes all the conditional branches. But InstCombine and 
SimplifyCFG shouldn't run on an optnone function, right?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71687

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to