On Tue, Dec 11, 2012 at 1:58 AM, NAKAMURA Takumi <span dir="ltr"><<a href="mailto:geek4civic@gmail.com" target="_blank">geek4civic@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Should it be for llvm-commits?<br></blockquote><div><br></div><div>Yes, sorry, this was meant for llvm-commits.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

2012/12/11 Saleem Abdulrasool <<a href="mailto:compnerd@compnerd.org">compnerd@compnerd.org</a>>:<br>
<div><div class="h5">> Cleanup compiler warnings on discarding type qualifiers in casts.  Switch to C++ style casts.<br>
><br>
> <a href="http://llvm-reviews.chandlerc.com/D204" target="_blank">http://llvm-reviews.chandlerc.com/D204</a><br>
><br>
> Files:<br>
>   lib/Target/ARM/ARMCodeEmitter.cpp<br>
>   lib/Target/ARM/ARMConstantPoolValue.cpp<br>
><br>
> Index: lib/Target/ARM/ARMCodeEmitter.cpp<br>
> ===================================================================<br>
> --- lib/Target/ARM/ARMCodeEmitter.cpp<br>
> +++ lib/Target/ARM/ARMCodeEmitter.cpp<br>
> @@ -371,12 +371,16 @@<br>
>  }<br>
><br>
>  bool ARMCodeEmitter::runOnMachineFunction(MachineFunction &MF) {<br>
> -  assert((MF.getTarget().getRelocationModel() != Reloc::Default ||<br>
> -          MF.getTarget().getRelocationModel() != Reloc::Static) &&<br>
> +  TargetMachine &Target = const_cast<TargetMachine&>(MF.getTarget());<br>
> +<br>
> +  assert((Target.getRelocationModel() != Reloc::Default ||<br>
> +          Target.getRelocationModel() != Reloc::Static) &&<br>
>           "JIT relocation model must be set to static or default!");<br>
> -  JTI = ((ARMBaseTargetMachine &)MF.getTarget()).getJITInfo();<br>
> -  II = (const ARMBaseInstrInfo *)MF.getTarget().getInstrInfo();<br>
> -  TD = MF.getTarget().getDataLayout();<br>
> +<br>
> +  JTI = static_cast<ARMJITInfo*>(Target.getJITInfo());<br>
> +  II = static_cast<const ARMBaseInstrInfo*>(Target.getInstrInfo());<br>
> +  TD = Target.getDataLayout();<br>
> +<br>
>    Subtarget = &TM.getSubtarget<ARMSubtarget>();<br>
>    MCPEs = &MF.getConstantPool()->getConstants();<br>
>    MJTEs = 0;<br>
> Index: lib/Target/ARM/ARMConstantPoolValue.cpp<br>
> ===================================================================<br>
> --- lib/Target/ARM/ARMConstantPoolValue.cpp<br>
> +++ lib/Target/ARM/ARMConstantPoolValue.cpp<br>
> @@ -209,7 +209,7 @@<br>
>      S(strdup(s)) {}<br>
><br>
>  ARMConstantPoolSymbol::~ARMConstantPoolSymbol() {<br>
> -  free((void*)S);<br>
> +  free(const_cast<void*>(reinterpret_cast<const void *>(S)));<br>
>  }<br>
><br>
>  ARMConstantPoolSymbol *<br>
><br>
</div></div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org<br>
</div>