[PATCH] D96320: [ThinLTO, Sanitizers] Skip instrumentation by testing backend

2021-02-10 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka added a comment. In D96320#2554493 , @aeubanks wrote: > This seems like the wrong fix to me. > ThinLTO only runs the simplification pipeline pre-link, then runs it again > plus the optimization pipeline. The optimization pipeline is the only pl

[PATCH] D96320: [ThinLTO, Sanitizers] Skip instrumentation by testing backend

2021-02-10 Thread Arthur Eubanks via Phabricator via cfe-commits
aeubanks added a comment. This seems like the wrong fix to me. ThinLTO only runs the simplification pipeline pre-link, then runs it again plus the optimization pipeline. The optimization pipeline is the only place that OptimizerLastEPCallback should run (aside from the -O0 pipeline). Basically,

[PATCH] D96320: [ThinLTO, Sanitizers] Skip instrumentation by testing backend

2021-02-08 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka created this revision. Herald added subscribers: steven_wu, hiraditya, inglorion. vitalybuka requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Distributed ThinLTO has special case when backend fallback to regular non-ThinLTO versio