[PATCH] D89590: [NPM][SimplifyCFGPass] For OptForFuzzing attribute, disable SimplifyCondBranch and FoldTwoEntryPHINode in NPM

Fri Oct 16 13:34:21 PDT 2020

zequanwu added a comment.

In D89590#2335885 <https://reviews.llvm.org/D89590#2335885>, @lebedev.ri wrote:

> In D89590#2335864 <https://reviews.llvm.org/D89590#2335864>, @aeubanks wrote:
>
>> In D89590#2335840 <https://reviews.llvm.org/D89590#2335840>, @lebedev.ri wrote:
>>
>>> Why is this NPM specific?
>>
>> This is something the legacy pass did but was not properly ported to the NPM pass.
>
> Right. But why is this PM-specific at all, why aren't the folds themselves guarded with that?

It's from here: https://github.com/llvm/llvm-project/commit/cdd006eec9409923f9a56b9026ce2cb72e7b71dc. It was guarded by FoldTwoEntryPHINode and simplifyCondBranch.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89590