search for: loopinstsimplify

Hi, I think we should remove the LoopInstSimplify pass, as it has no test coverage and no users (afaik). If you are using the pass, or think that it should stay in tree for some other reason, please let me know. Here's the patch: https://reviews.llvm.org/D44053 <https://reviews.llvm.org/D44053> vedant -------------- next part --------...

...often true on entry to a loop pass manager, but is not upheld as passes run.) To be clear, the above list is aspirational.  We definitely don't do all of the above today.  :) Philip On 03/02/2018 04:36 PM, Vedant Kumar via llvm-dev wrote: > Hi, > > I think we should remove the LoopInstSimplify pass, as it has no test > coverage and no users (afaik). > > If you are using the pass, or think that it should stay in tree for > some other reason, please let me know. > > Here's the patch: https://reviews.llvm.org/D44053 > > vedant > > > ___________________...

...gt;> >> To be clear, the above list is aspirational.  We definitely don't do >> all of the above today.  :) >> >> Philip >> >> On 03/02/2018 04:36 PM, Vedant Kumar via llvm-dev wrote: >>> Hi, >>> >>> I think we should remove the LoopInstSimplify pass, as it has no >>> test coverage and no users (afaik). >>> >>> If you are using the pass, or think that it should stay in tree for >>> some other reason, please let me know. >>> >>> Here's the patch: https://reviews.llvm.org/D44053 &gt...

...ass manager, but is not upheld as passes run.) > To be clear, the above list is aspirational. We definitely don't do all of the above today. :) > > Philip > > On 03/02/2018 04:36 PM, Vedant Kumar via llvm-dev wrote: >> Hi, >> >> I think we should remove the LoopInstSimplify pass, as it has no test coverage and no users (afaik). >> >> If you are using the pass, or think that it should stay in tree for some other reason, please let me know. >> >> Here's the patch: https://reviews.llvm.org/D44053 <https://reviews.llvm.org/D44053> >&...

...een removed now all in-tree users have been updated to `LLVM_DEBUG`. [r333091](https://reviews.llvm.org/rL333091). * The CostPerUse attribute is now set on RISC-V registers which are unlikely to be accessible in compressed (16-bit) instructions. [r333132](https://reviews.llvm.org/rL333132). * The LoopInstSimplify pass has been restored and will see improvements and integration into the loop pass pipeline. [r333250](https://reviews.llvm.org/rL333250). ## Clang commits * Support was committed for the GNUstep Objective-C ABI version 2. [r332950](https://reviews.llvm.org/rL332950). * `__builtin_classify_typ...

...d to support reporting multiple 'micro-test' results, to enable it to be used for collecting microbenchmark results. [r327422](http://reviews.llvm.org/rL327422). * The handling of dangling debug info in the SelectionDAG has been improved. [r327303](http://reviews.llvm.org/rL327303). * The LoopInstSimplify pass has been removed due to the fact that it is not used in-tree and has no tests. [r327329](http://reviews.llvm.org/rL327329). * dsymutil will now perform `analyzeContextInfo` and `CloneDIEs` in parallel, leading to performance improvements of up to ~30%. [r327399](http://reviews.llvm.org/rL3273...

...elease+Asserts build llvm[3]: Compiling LoopIdiomRecognize.cpp for Release+Asserts build llvm[3]: Compiling GlobalsModRef.cpp for Release+Asserts build llvm[3]: Compiling DIEHash.cpp for Release+Asserts build llvm[3]: Compiling X86MachineFunctionInfo.cpp for Release+Asserts build llvm[3]: Compiling LoopInstSimplify.cpp for Release+Asserts build llvm[3]: Compiling IPA.cpp for Release+Asserts build llvm[3]: Compiling X86PadShortFunction.cpp for Release+Asserts build llvm[3]: Compiling DbgValueHistoryCalculator.cpp for Release+Asserts build llvm[3]: Compiling InlineCost.cpp for Release+Asserts build llvm[3]: Com...