[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