[PATCH] D69557: AsmParser: Allow FMF on varargs call

Tim Renouf via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 29 10:07:57 PDT 2019


tpr added a comment.

In D69557#1725564 <https://reviews.llvm.org/D69557#1725564>, @foad wrote:

> In D69557#1725496 <https://reviews.llvm.org/D69557#1725496>, @spatel wrote:
>
> > Does D69161 <https://reviews.llvm.org/D69161> cover this change? If so, we could add the test there and fix the problem in 1 step.
>
>
> I think this bug fix is orthogonal to D69161 <https://reviews.llvm.org/D69161>, which adds new functionality.


The way you have changed the call check to !isa<FPMathOperator>(CI) means you might have accidentally fixed my bug anyway. I'll check later.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69557





More information about the llvm-commits mailing list