[PATCH] D33515: Fix the ManagedStatic list ordering when using DynamicLibrary::addPermanentLibrary.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 24 15:11:10 PDT 2017


efriedma added a comment.

In https://reviews.llvm.org/D33515#763847, @marsupial wrote:

> I'm sorry.
>  You're saying adding the changes to **Hello.cpp**,
>  **make -j8 opt LLVMHello**
>  **bin/opt -load=lib/LLVMHello.so -S /dev/null**
>  is still crashing?
>
> This patch definitively fixes that problem here.
>
> I'll try building clean with the instruction for polly (i.e. no cmake flags).
>  Could you do the same?


On Ubuntu 14.04.  Built with no CMake flags, r303808, with this patch plus my changes to LLVMHello.

  $ bin/opt -load=lib/LLVMHello.so -S /dev/null
  ; ModuleID = '/dev/null'
  source_filename = "/dev/null"
  Segmentation fault


https://reviews.llvm.org/D33515





More information about the llvm-commits mailing list