[PATCH] D137149: Use PassGate from LLVMContext if any otherwise global one

ebrevnov updated this revision to Diff 477706. ebrevnov added a comment. Herald added a project: Flang. Fixed build error in flang/lib/Frontend/FrontendActions.cpp Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137149/new/

[PATCH] D137149: Use PassGate from LLVMContext if any otherwise global one

ebrevnov updated this revision to Diff 477446. ebrevnov marked an inline comment as done. ebrevnov added a comment. Herald added a project: clang. Herald added a subscriber: cfe-commits. Updated Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D102090: [CMake][ELF] Link libLLVM.so and libclang-cpp.so with -Bsymbolic-functions

ebrevnov added a comment. > ! In D102090#3174187 , @rnk wrote: > In any case, -Bsymbolic is not a good default for the project because of > issues like this. I see. Thank you. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D102090: [CMake][ELF] Link libLLVM.so and libclang-cpp.so with -Bsymbolic-functions

ebrevnov added a comment. In D102090#3172538 , @MaskRay wrote: > To add on rnk's comment (deduplication of vague linkage data which may be > included in multiple shared objects), another big reason is -Bsymbolic data > does not work with copy

[PATCH] D102090: [CMake][ELF] Link libLLVM.so and libclang-cpp.so with -Bsymbolic-functions

ebrevnov added a comment. In D102090#3169726 , @rnk wrote: > In D102090#3169439 , @ebrevnov > wrote: > >> While -Bsymbolic-funtions brings nice performance improvements it also >> changes symbol resolution

[PATCH] D102090: [CMake][ELF] Link libLLVM.so and libclang-cpp.so with -Bsymbolic-functions

ebrevnov added a comment. While -Bsymbolic-funtions brings nice performance improvements it also changes symbol resolution order. That means we effectively disabled preemption for functions and all references from inside libLLVM*.so will be resolved locally. But references to global data can

[PATCH] D65286: [OpenCL] Allow OpenCL C style vector initialization in C++

ebrevnov added a comment. Please be aware about build bot failure (http://lab.llvm.org:8011/builders/clang-cmake-thumbv7-full-sh/builds/2185) most likely caused by this change. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65286/new/