[PATCH] D100339: [Attributor] Run lightweight version of the Attributor by default.

Thu Apr 15 09:08:30 PDT 2021

fhahn added a comment.

In D100339#2686649 <https://reviews.llvm.org/D100339#2686649>, @sstefan1 wrote:

> I just want to mention that this is just a starting point, as suggested in D99769 <https://reviews.llvm.org/D99769>. It is definitely not our attention to rush and do this without an RFC or numbers that are in favour of the switch.
>
> The main reason for this 'lightweight' version is exactly the compile time and I will definitely measure it. This only introduces the lightweight version, so that it will be easier to switch, when we come to that point.
>
> I hope we are on the same page now.

Thanks for putting up the patch to get things started! I'd recommend trying to build https://github.com/llvm/llvm-test-suite in various configurations and investigating the crashes. For example, I just tried to do so with `AttributorRunOption::ALL` with `-O3 -flto` on X86 and there's a crash building `MultiSource/Applications/lambda-0.1.3`.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100339/new/

https://reviews.llvm.org/D100339