[llvm-dev] Removing LLVM_ALWAYS_INLINE from ADT classes (original) (raw)

George Karpenkov via llvm-dev llvm-dev at lists.llvm.org
Fri Jan 4 15:39:52 PST 2019

+1, I think that’s a great change.

On Jan 4, 2019, at 3:36 PM, Duncan P. N. Exon Smith via llvm-dev <llvm-dev at lists.llvm.org> wrote:

This makes sense to me. One concern is that this in itself will slow down the build, since tablegen will get even slower. Ideally, there would be some (perhaps default?) configuration where we build the tablegen binaries with optimizations on and then use them in the build, as if we were cross-compiling.

On Jan 4, 2019, at 15:14, Davide Italiano via llvm-dev <llvm-dev at lists.llvm.org> wrote:

Hi, I would like to propose, based on a previous discussion on llvm-dev, the following change. https://reviews.llvm.org/D56337 The main motivation for annotating member functions of ADT clases with LLVMALWAYSINLINE was that of speeding up check-llvm at -O0. Turns out this significantly degrades the debuggability of fundamental classes in llvm itself, e.g. StringRef or SmallVector. After discussing we agreed that it's reasonable to drop LLVMALWYASINLINE from ADT classes member functions and add a note in the developer's manual. If you have any feedbacks or concerns, please speak up. If nothing major arises, I'm going to commit this patch in a week (or such). Thanks for your time, -- Davide "There are no solved problems; there are only problems that are more or less solved" -- Henri Poincare

LLVM Developers mailing list llvm-dev at lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev

LLVM Developers mailing list llvm-dev at lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev

More information about the llvm-dev mailing list