[llvm] r263870 - Fix a const_cast related warning in GCC in the C API for libLTO

Mehdi Amini via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 19 14:33:51 PDT 2016


Yeah, I was sloppy on bridging to C...

r263885 should be better.

-- 
Mehdi



> On Mar 19, 2016, at 11:34 AM, David Blaikie <dblaikie at gmail.com> wrote:
> 
> Does LTOObjectBuffer actually need to modify? Or is it just const incorrect/should be constified? (if it does need to modify, perhaps we need to change the API here to expose something other than a MemoryBuffer for use here to document the mutability?)
> 
> On Fri, Mar 18, 2016 at 6:24 PM, Mehdi Amini via llvm-commits <llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>> wrote:
> Author: mehdi_amini
> Date: Fri Mar 18 20:24:23 2016
> New Revision: 263870
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=263870&view=rev <http://llvm.org/viewvc/llvm-project?rev=263870&view=rev>
> Log:
> Fix a const_cast related warning in GCC in the C API for libLTO
> 
> From: Mehdi Amini <mehdi.amini at apple.com <mailto:mehdi.amini at apple.com>>
> 
> Modified:
>     llvm/trunk/tools/lto/lto.cpp
> 
> Modified: llvm/trunk/tools/lto/lto.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/lto/lto.cpp?rev=263870&r1=263869&r2=263870&view=diff <http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/lto/lto.cpp?rev=263870&r1=263869&r2=263870&view=diff>
> ==============================================================================
> --- llvm/trunk/tools/lto/lto.cpp (original)
> +++ llvm/trunk/tools/lto/lto.cpp Fri Mar 18 20:24:23 2016
> @@ -469,7 +469,7 @@ LTOObjectBuffer thinlto_module_get_objec
>                                            unsigned int index) {
>    assert(index < unwrap(cg)->getProducedBinaries().size() && "Index overflow");
>    auto &MemBuffer = unwrap(cg)->getProducedBinaries()[index];
> -  return LTOObjectBuffer{(void *)MemBuffer->getBufferStart(),
> +  return LTOObjectBuffer{const_cast<char *>(MemBuffer->getBufferStart()),
>                           MemBuffer->getBufferSize()};
>  }
> 
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits <http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits>
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160319/2ee0f19e/attachment.html>


More information about the llvm-commits mailing list