[PATCH] D71353: Fix denormal-fp-math flag and attribute interaction

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 12 15:45:24 PDT 2020


echristo added a comment.

So I'm still concerned that this is on TargetOptions at all. Ideally for new attributes like this we'd only put them on the IR - at the way you've got it we're making resetTargetOptions even harder to remove. I don't think this is the right path forward (as removing resetTargetOptions is the right path) and this is only making that harder.


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

https://reviews.llvm.org/D71353





More information about the llvm-commits mailing list