search for: hl_else_after_return

...; per http://llvm.org/docs/CodingStandards.html#hl_predicateloops + // Store cannot be volatile (case 2) and must-alias with our pointer. + if (S->isVolatile()) { + return false; + } else { no need for the 'else after return': http://llvm.org/docs/CodingStandards.html#hl_else_after_return + AliasAnalysis& AA = getAnalysis<AliasAnalysis>(); + if (AA.alias(S->getPointerOperand(), 1, pointer, 1) != + AliasAnalysis::MustAlias) + return false; Knowing that the loaded and stored pointer must alias is important, but you also need to check that the loaded and st...

Hello, Sorry for my stupid mistakes. I hope that everything is fine now. This patch fixes PR2218. There are no loads in example, however "instcombine" changes memcpy() into store/load. Regards, Jakub Staszak -------------- next part -------------- A non-text attachment was scrubbed... Name: pr2218-2.patch Type: application/octet-stream Size: 6525 bytes Desc: not available URL:

...s.html#hl_predicateloops > > > + // Store cannot be volatile (case 2) and must-alias with our > pointer. > + if (S->isVolatile()) { > + return false; > + } else { > > no need for the 'else after return': http://llvm.org/docs/CodingStandards.html#hl_else_after_return > > + AliasAnalysis& AA = getAnalysis<AliasAnalysis>(); > + if (AA.alias(S->getPointerOperand(), 1, pointer, 1) != > + AliasAnalysis::MustAlias) > + return false; > > Knowing that the loaded and stored pointer must alias is important, > but you als...