[llvm] [MachineLICM] Relax overlay conservative PHI check (PR #67186)

Hendrik Greving via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 9 06:07:06 PDT 2023


hgreving2304 wrote:

Sorry that was an accidental merge, I've reverted it, can you confirm?

On Mon, Oct 9, 2023 at 2:33 PM Nikita Popov ***@***.***>
wrote:

> Due to the merge we have some compile-time numbers:
> http://llvm-compile-time-tracker.com/compare.php?from=7b3bbd83c0c24087072ec5b22a76799ab31f87d5&to=71a8d2e3064fcb3ff76565e6e8529613f90aa51b&stat=instructions%3Au
>
> This causes significant overhead in some cases, e.g. libclamav_autoit.c
> goes up by 3.5% and libclamav_htmlnorm.c goes up by 6%. Please mitigate
> compile-time regressions before resubmitting this.
>
>> Reply to this email directly, view it on GitHub
> <https://github.com/llvm/llvm-project/pull/67186#issuecomment-1752925373>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AKK7Q5KJCPOVTGU4PJA4FJLX6PVIBAVCNFSM6AAAAAA5DRNYXSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJSHEZDKMZXGM>
> .
> You are receiving this because you modified the open/close state.Message
> ID: ***@***.***>
>


https://github.com/llvm/llvm-project/pull/67186


More information about the llvm-commits mailing list