[llvm] r215341 - Revert "Increase the size of this SmallVector in PeepholeOptimizer." (r215340)

Eric Christopher echristo at gmail.com
Mon Aug 11 09:48:20 PDT 2014


Can you switch the one use to a SmallPtrSetImpl<MachineInstr *> the
way we do for SmallVector?

-eric

On Sun, Aug 10, 2014 at 7:34 PM, Hans Wennborg <hans at hanshq.net> wrote:
> Author: hans
> Date: Sun Aug 10 21:34:52 2014
> New Revision: 215341
>
> URL: http://llvm.org/viewvc/llvm-project?rev=215341&view=rev
> Log:
> Revert "Increase the size of this SmallVector in PeepholeOptimizer." (r215340)
>
> That broke the build:
>
> /data/buildslave/clang-amd64-freebsd/src-llvm/lib/CodeGen/PeepholeOptimizer.cpp:729:46: error: non-const lvalue reference to type 'SmallPtrSet<[...], 8>' cannot bind to a value of unrelated type 'SmallPtrSet<[...], 16>'
>         Changed |= optimizeExtInstr(MI, MBB, LocalMIs);
>                                              ^~~~~~~~
> /data/buildslave/clang-amd64-freebsd/src-llvm/lib/CodeGen/PeepholeOptimizer.cpp:265:49: note: passing argument to parameter 'LocalMIs' here
>                  SmallPtrSet<MachineInstr*, 8> &LocalMIs) {
>                                                 ^
>
>
>
> Modified:
>     llvm/trunk/lib/CodeGen/PeepholeOptimizer.cpp
>
> Modified: llvm/trunk/lib/CodeGen/PeepholeOptimizer.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/PeepholeOptimizer.cpp?rev=215341&r1=215340&r2=215341&view=diff
> ==============================================================================
> --- llvm/trunk/lib/CodeGen/PeepholeOptimizer.cpp (original)
> +++ llvm/trunk/lib/CodeGen/PeepholeOptimizer.cpp Sun Aug 10 21:34:52 2014
> @@ -687,7 +687,7 @@ bool PeepholeOptimizer::runOnMachineFunc
>      MachineBasicBlock *MBB = &*I;
>
>      bool SeenMoveImm = false;
> -    SmallPtrSet<MachineInstr*, 16> LocalMIs;
> +    SmallPtrSet<MachineInstr*, 8> LocalMIs;
>      SmallSet<unsigned, 4> ImmDefRegs;
>      DenseMap<unsigned, MachineInstr*> ImmDefMIs;
>      SmallSet<unsigned, 16> FoldAsLoadDefCandidates;
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits



More information about the llvm-commits mailing list