[PATCH] PR19958 wrong code at -O1 and above on x86_64-linux-gnu (InstCombine)
sardask01 at gmail.com
Sun Jul 13 10:14:13 PDT 2014
(I have run clang-format on the code :))
On Sun, Jul 13, 2014 at 10:38 PM, suyog <suyog.sarda at samsung.com> wrote:
> Hi Duncan,
> Made changes as per your suggestions :
> - Wrote the whole logic in separate function.
> Didn't make it 'static', took 'cue' from other 'Foldicmp' functions.
> - Removed TODO, will probably raise a separate PR for it.
> - Updated comments, removed extra braces.
> - defined variables just before their use.
> Can you please see if this looks good to you?
> Your comments/suggestions are most welcomed.
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits