search for: r324702

>Totally fair call, sorry it took me a while to come back around to this - added in r324702 Thanks ! Have a question: does it need "requres shell" ? I tried without and it worked for me under windows (had to change check to CHECK: .{{\\|/}}test.cpp:2:51 though). And I see nothing special probably that might require shell I think. We use cd/rm/cp in LLD testcases without requiri...

Nah, doesn't look like it. Removed it (& the place I copied it from) in: r324738 Thanks! On Fri, Feb 9, 2018 at 7:28 AM George Rimar <grimar at accesssoftek.com> wrote: > >Totally fair call, sorry it took me a while to come back around to this - > added in r324702 > > Thanks ! Have a question: does it need "requres shell" ? > I tried without and it worked for me under windows (had to change check > to CHECK: .{{\\|/}}test.cpp:2:51 though). > And I see nothing special probably that might require shell I think. We > use cd/rm/cp in...

Totally fair call, sorry it took me a while to come back around to this - added in r324702 On Thu, Nov 16, 2017 at 3:47 PM Rafael Avila de Espindola < rafael.espindola at gmail.com> wrote: > David Blaikie via llvm-dev <llvm-dev at lists.llvm.org> writes: > > > On Thu, Nov 16, 2017 at 11:44 AM Robinson, Paul <paul.robinson at sony.com> > > wrote: >...

...;t look like it. Removed it (& the place I copied it from) in: r324738 Thanks! On Fri, Feb 9, 2018 at 7:28 AM George Rimar <grimar at accesssoftek.com<mailto:grimar at accesssoftek.com>> wrote: >Totally fair call, sorry it took me a while to come back around to this - added in r324702 Thanks ! Have a question: does it need "requres shell" ? I tried without and it worked for me under windows (had to change check to CHECK: .{{\\|/}}test.cpp:2:51 though). And I see nothing special probably that might require shell I think. We use cd/rm/cp in LLD testcases without requiri...

David Blaikie via llvm-dev <llvm-dev at lists.llvm.org> writes: > On Thu, Nov 16, 2017 at 11:44 AM Robinson, Paul <paul.robinson at sony.com> > wrote: > >> There's no requirement that DW_AT_ranges (or high/low_pc) appear on the >> skeleton CU rather than the DWO CU. So it's quite possible that to get the >> address ranges covered by the CU one would

...ike it. Removed it (& the place I copied it from) in: > r324738 > > Thanks! > > On Fri, Feb 9, 2018 at 7:28 AM George Rimar <grimar at accesssoftek.com> > wrote: > >> >Totally fair call, sorry it took me a while to come back around to this >> - added in r324702 >> >> Thanks ! Have a question: does it need "requres shell" ? >> I tried without and it worked for me under windows (had to change check >> to CHECK: .{{\\|/}}test.cpp:2:51 though). >> And I see nothing special probably that might require shell I think. We &...