[PATCH] Don't create instructions in CFLAliasAnalysis. NFC
Pete Cooper
peter_cooper at apple.com
Fri Jun 12 09:20:48 PDT 2015
Thanks George! r239616.
Pete
> On Jun 10, 2015, at 11:51 PM, George Burgess IV <george.burgess.iv at gmail.com> wrote:
>
> LGTM
>
> - George
>
>> On Jun 11, 2015, at 1:24 AM, Pete Cooper <peter_cooper at apple.com> wrote:
>>
>> Hi Hal, George
>>
>> The CFLAA code currently calls ConstantExpr::getAsInstruction which creates an instruction from a constant expr.
>>
>> We then pass that instruction to the InstVisitor to analyze it.
>>
>> Its not necessary to create these instructions as we can just cast from Constant to Operator in the visitor. This is how other InstVisitor’s such as SelectionDAGBuilder handle ConstantExpr.
>>
>> This should provide a small speed improvement to this pass, but i’m actually doing this as an effort to devirtualize Value. This just happened to come up in that patch sequence.
>>
>> Cheers,
>> Pete
>>
>> <cfl.patch>
More information about the llvm-commits
mailing list