[PATCH] D66602: Add a LineTable class to GSYM and test it.

[Greg Clayton via llvm-commits]

Looks like dblaikie fixed the build with 371681. Thanks dblaikie!


> On Sep 11, 2019, at 3:29 PM, Keno Fischer via Phabricator <reviews at reviews.llvm.org> wrote:
> 
> loladiro added a comment.
> 
> This broke the build for me locally:
> 
>  In file included from /home/keno/llvm-project/llvm/lib/DebugInfo/GSYM/FunctionInfo.cpp:10:0:
>  /home/keno/llvm-project/llvm/include/llvm/DebugInfo/GSYM/FunctionInfo.h:36:29: error: declaration of ‘llvm::Optional<llvm::gsym::LineTable> llvm::gsym::FunctionInfo::LineTable’ [-fpermissive]
>     llvm::Optional<LineTable> LineTable;
>                               ^~~~~~~~~
>  In file included from /home/keno/llvm-project/llvm/include/llvm/DebugInfo/GSYM/FunctionInfo.h:14:0,
>                   from /home/keno/llvm-project/llvm/lib/DebugInfo/GSYM/FunctionInfo.cpp:10:
>  /home/keno/llvm-project/llvm/include/llvm/DebugInfo/GSYM/LineTable.h:118:7: error: changes meaning of ‘LineTable’ from ‘class llvm::gsym::LineTable’ [-fpermissive]
>   class LineTable {
>         ^~~~~~~~~
>  [720/3364] Building CXX object lib/Analysis/CMakeFiles/LLVMAnalysis.dir/ScalarEvolution.cpp.o
> 
> 
> CHANGES SINCE LAST ACTION
>  https://reviews.llvm.org/D66602/new/
> 
> https://reviews.llvm.org/D66602
> 
> 
>