Displaying 7 results from an estimated 7 matches for "aarch64instructionselector".
2017 May 31
0
Buildbots timing out on full builds
Is https://reviews.llvm.org/differential/diff/100829/ replacement for
r303341?
If so LGTM.
r303542 msan AArch64InstructionSelector.cpp: 1m17.209s
r303542+diff/100829/ <https://reviews.llvm.org/differential/diff/100829/>
msan AArch64InstructionSelector.cpp: 1m24.724s
On Wed, May 31, 2017 at 6:13 AM, Daniel Sanders <daniel_l_sanders at apple.com>
wrote:
> Great! I expect I'll be able to cut it down furt...
2017 May 31
2
Buildbots timing out on full builds
...el Sanders <daniel_l_sanders at apple.com <mailto:daniel_l_sanders at apple.com>> wrote:
> Hi Diana and Vitaly,
>
> Could you give https://reviews.llvm.org/differential/diff/100829/ <https://reviews.llvm.org/differential/diff/100829/> a try? When measuring the compile of AArch64InstructionSelector.cpp.o with asan enabled and running under instruments's Allocation profiler, my machine reports that the cumulative memory allocations is down to ~3.5GB (was ~10GB), the number of allocations down to ~4 million (was ~23 million), and the compile time down to ~15s (was ~60s).
>
> The patc...
2017 Dec 20
6
[GlobalISel] gen-global-isel failed to work
...gt;>> Hi Leslie,
>>>>
>>>> There should be a definition of GPR8RegClassID in
>>>> $build_dir/lib/Target/AVR/AVRGenRegisterInfo.inc which should be included by
>>>> AVRRegisterInfo.h. AArch64 includes its AArch64RegisterInfo.h in
>>>> AArch64InstructionSelector.cpp but it seems that ARM gets it indirectly when
>>>> it includes ARMSubtarget.h. It looks like you need to add '#include
>>>> "AVRRegisterInfo.h"' to AVRInstructionSelector.h.
>>>>
>>>> Hope that helps
>>>
>>> Sti...
2017 May 24
2
Buildbots timing out on full builds
...39;ll finish off the patches and post
> them individually.
>
> The first one removes the single-use lambdas in the generated code. These
> turn out to be _really_ expensive. Replacing them with equivalent gotos
> saves 11 million allocations (~57%) during the course of compiling
> AArch64InstructionSelector.cpp.o. The cumulative number of bytes allocated
> also drops by ~4GB (~36%).
>
(this is outside my wheelhouse, so just as an aside): Could you explain
further what aspect of the change was that saved allocations? Lambdas
themselves don't allocate memory (std::function of a stateful lambd...
2017 May 31
2
Buildbots timing out on full builds
Hi Diana and Vitaly,
Could you give https://reviews.llvm.org/differential/diff/100829/ <https://reviews.llvm.org/differential/diff/100829/> a try? When measuring the compile of AArch64InstructionSelector.cpp.o with asan enabled and running under instruments's Allocation profiler, my machine reports that the cumulative memory allocations is down to ~3.5GB (was ~10GB), the number of allocations down to ~4 million (was ~23 million), and the compile time down to ~15s (was ~60s).
The patch is based...
2017 May 22
2
Buildbots timing out on full builds
Nope, no sanitizers.
On 22 May 2017 at 11:38, Daniel Sanders <daniel_l_sanders at apple.com> wrote:
> Is that with -fsanitize=memory too?
>
> I'm currently building ToT with r303258 reverted. Once that's done I'll commit the revert and start investigating fixes.
>
>> On 22 May 2017, at 10:22, Diana Picus <diana.picus at linaro.org> wrote:
>>
2017 May 25
2
Buildbots timing out on full builds
Thanks for trying that patch. I agree that 34 mins still isn't good enough but we're heading in the right direction.
Changing the partitioning predicate to the instruction opcode rather than the number of operands in the top-level instruction will hopefully cut it down further. I also have a patch that shaves a small amount off of the compile-time by replacing the various