[PATCH] D86295: [analyzer] Reorder the layout of MemRegion and cache by hand for optimal size

Balázs Benics via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 24 07:26:47 PDT 2020


steakhal added a comment.

In D86295#2233401 <https://reviews.llvm.org/D86295#2233401>, @vsavchenko wrote:

> Yep, I guess that is the cause.  I'll take a look.  Did you try it with this fast fix?

I tried, but it lacks further fixes.

Currently, I have this:

  diff --git a/clang/utils/analyzer/Dockerfile b/clang/utils/analyzer/Dockerfile
  index f74ff8aa95c..7727d92f98f 100644
  --- a/clang/utils/analyzer/Dockerfile
  +++ b/clang/utils/analyzer/Dockerfile
  @@ -1,4 +1,4 @@
  -FROM ubuntu:bionic
  +FROM ubuntu:bionic-20200526
   
   RUN apt-get update && apt-get install -y \
       apt-transport-https \
  @@ -12,12 +12,13 @@ RUN wget -O - https://apt.kitware.com/keys/kitware-archive-latest.asc 2>/dev/nul
   RUN apt-add-repository -y 'deb https://apt.kitware.com/ubuntu/ bionic main'
   
   # test system dependencies
  +# TODO: should we really depend on the cmake version 3.18.2 here? I don't think so.
   RUN apt-get update && apt-get install -y \
       git=1:2.17.1-1ubuntu0.7 \
       gettext=0.19.8.1-6ubuntu0.3 \
       python3=3.6.7-1~18.04 \
  -    python3-pip=9.0.1-2.3~ubuntu1.18.04.1 \
  -    cmake=3.17.3-0kitware1 \
  +    python3-pip=9.0.1-2.3~ubuntu1.18.04.2 \
  +    cmake=3.18.2-0kitware1 \
       ninja-build=1.8.2-1
   
   # box2d dependencies
   

I'm not sure if its the company firewall or something else.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86295/new/

https://reviews.llvm.org/D86295



More information about the cfe-commits mailing list