[PATCH] D29111: [SimplifyCFG] Do not sink and merge inline-asm instructions
Akira Hatanaka via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 24 23:06:17 PST 2017
Thanks for the review, committed in r293025.
Can we merge this into 4.0?
On Tue, Jan 24, 2017 at 9:02 PM, David Majnemer via Phabricator <
reviews at reviews.llvm.org> wrote:
> majnemer accepted this revision.
> majnemer added a comment.
> This revision is now accepted and ready to land.
>
> LGTM
>
>
>
> ================
> Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:1440
> +
> + // Conservatively return false if I is an inline-asm instruction.
> + if (const auto *C = dyn_cast<CallInst>(I))
> ----------------
> I'd mention the constraint issue, it quickly explains why this is needed.
>
>
> https://reviews.llvm.org/D29111
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170124/3a1defd8/attachment.html>
More information about the llvm-commits
mailing list