[lld] r173896 - [ELF] Add ELFTargetRelocationHandler.

Shankar Easwaran shankare at codeaurora.org
Wed Jan 30 10:42:21 PST 2013


Sean,

This is for the DefaultTargetHandler.

Shankar Easwaran

On 1/30/2013 12:27 PM, Sean Silva wrote:
> +  const TargetRelocationHandler<ELFT> &getRelocationHandler() const {
> +    llvm_unreachable("Target should provide implementation for function ");
> +  }
> +
>
> What's the point of overriding a pure virtual just to make it
> unreachable? Just leave it pure virtual.
>
> -- Sean Silva
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>


-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation




More information about the llvm-commits mailing list