[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