[llvm-commits] [llvm] r95975 - in /llvm/trunk: include/llvm/Analysis/ lib/Analysis/ lib/CodeGen/ lib/Transforms/Scalar/ test/CodeGen/ARM/ test/CodeGen/Thumb2/ test/CodeGen/X86/ test/Transforms/IndVarSimplify/ test/Transforms/LoopStrengthReduce/

Daniel Dunbar daniel at zuster.org
Mon Feb 15 09:47:02 PST 2010


On Mon, Feb 15, 2010 at 9:38 AM, David Greene <dag at cray.com> wrote:
> On Monday 15 February 2010 11:33:20 Daniel Dunbar wrote:
>> Please do not add more command line options to the backend.
>>
>> I think it should be up to the front-ends whether or not they want
>> this verifier pass. What about moving this code to llvm-backend.cpp
>> (llvm-gcc) and Backend.cpp (clang).
>
> The verifier has saved me many times.  I would hate to move this call out of
> LLVM proper.  We've found many problems with LSR in the past.  Perhaps Nicolas
> can describe his use-case and we can find a solution that works for everyone.

You are free to add as many verifier passes as you want, I didn't
suggest removing it, I merely suggested making it a client decision.
Libraries shouldn't impose policy decisions on their clients.

 - Daniel

>                                                   -Dave
>
> _______________________________________________
> 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