[PATCH] D13501: [ELF2] Add -wrap switch

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 13 13:46:53 PDT 2015


----- Original Message -----
> From: "Rafael EspĂ­ndola" <rafael.espindola at gmail.com>
> To: "Hal Finkel" <hfinkel at anl.gov>
> Cc: "Igor Kudrin" <ikudrin.dev at gmail.com>, "llvm-commits" <llvm-commits at lists.llvm.org>,
> reviews+D13501+public+cc48a9c32b74c6f4 at reviews.llvm.org
> Sent: Tuesday, October 13, 2015 3:26:18 PM
> Subject: Re: [PATCH] D13501: [ELF2] Add -wrap switch
> 
> > For the record, I'll want the --wrap feature eventually (we need it
> > to
> > support profiling tools on statically-linked binaries).
> 
> Would Rui's proposed algorithm work for what you want?

I've not thought about it deeply, but on a quick read, yes, I think it will.

 -Hal

> 
> With the addition of some checks for which symbols are defined, it
> seems a brilliant way to implement --wrap :-)
> 
> Cheers,
> Rafael
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory


More information about the llvm-commits mailing list