search for: berghammer

IMO having to disable 2/3 of the tests means the patch isn't ready yet. On Fri, Aug 28, 2015 at 9:31 AM, Jason Kim <jasonk at codeaurora.org> wrote: > > > > -----Original Message----- > > From: Renato Golin [mailto:renato.golin at linaro.org] > > > TESTS! > > > Currently, about 2/3 tests for tsan fail/flake on android+aarch64. > > >

Those seem to be comments by the original author? Is this a problem? On Wed, Jul 8, 2015 at 10:31 AM Tamas Berghammer <tberghammer at google.com> wrote: > Hi Manual, > > Since the Phabricator update I noticed that the username of the reviewer > isn't displayed for some comment added to the review (e.g.: > http://reviews.llvm.org/D11016). Can you check it out what is causing > this issu...

On Fri, Jul 3, 2015 at 10:02 AM Andrew Wilkins <axwalk at gmail.com> wrote: > On Fri, 3 Jul 2015 at 01:43 Manuel Klimek <klimek at google.com> wrote: > >> On Thu, Jul 2, 2015 at 7:27 PM Jonathan Roelofs < >> jonathan at codesourcery.com> wrote: >> >>> Even further off topic, in phab wishlist land: It'd be awesome if it >>> were

On Wed, Aug 19, 2015 at 1:15 PM, Renato Golin <renato.golin at linaro.org> wrote: > On 19 August 2015 at 11:29, Dmitry Vyukov <dvyukov at google.com> wrote: >> Wait, this change is not submitted yet, right? Or you mean mailing of >> this change in bad shape? > > Right. > > Jason has submitted high quality patches before, so this is in no way > a reprimand