[PATCH] Stop resetting SanitizeAddress in TargetMachine::resetTargetOptions

Akira Hatanaka ahatanak at gmail.com
Fri May 8 16:16:42 PDT 2015


In http://reviews.llvm.org/D9570#169482, @echristo wrote:

> Hi Akira,
>
> This is just moving the resetting to a different place. How about we get
>  rid of the resetting completely?


Hi Eric,

Wouldn't getting rid of the resetting completely make it impossible to turn instrumentation on and off on a per-function basis? I don't know how important it is to have that capability, but it looks like r206971 was committed to enable it.


http://reviews.llvm.org/D9570

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list