[PATCH] D37153: [LoopUnroll] Properly update loop structure in case of successful peeling

[Davide Italiano via Phabricator via llvm-commits]

davide added a comment.

In https://reviews.llvm.org/D37153#854871, @mzolotukhin wrote:

> > I added a cl::opt...
>
> You seemed to forget to use it in the test though :)
>
> I would also give basic blocks and variables more meaningful names (specifically, I'd rename header and preheader blocks to show clearer, what happens in the test).
>
> Michael


Sorry, let me fix this right now.



================
Comment at: llvm/trunk/test/Transforms/LoopUnroll/pr33437.ll:2
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
+; RUN: opt -loop-unroll %s -S | FileCheck %s
+
----------------
mzolotukhin wrote:
> Won't `opt %s -S` overwrite the input file with the output?
No, I don't think so.
If it does, then I'll need to change the other hundreds of tests I've written :)


Repository:
  rL LLVM

https://reviews.llvm.org/D37153