diff options
| author | Sanjay Patel <spatel@rotateright.com> | 2014-09-02 20:03:00 +0000 |
|---|---|---|
| committer | Sanjay Patel <spatel@rotateright.com> | 2014-09-02 20:03:00 +0000 |
| commit | b2325b9ab36584c43b762295aa36db0a62413c93 (patch) | |
| tree | bcc7b1864a315e09569727011cbeb1ee3acec70f /llvm/lib/Analysis/MemoryDependenceAnalysis.cpp | |
| parent | 0e893f53dd302f2af3550c058c2baef715204a30 (diff) | |
| download | bcm5719-llvm-b2325b9ab36584c43b762295aa36db0a62413c93.tar.gz bcm5719-llvm-b2325b9ab36584c43b762295aa36db0a62413c93.zip | |
Fix a logic bug when copying fast-math flags.
"Setting" does not equal "copying". This bug has sat dormant for 2 reasons:
1. The unit test was not adequate.
2. Every current user of the "copyFastMathFlags" API is operating on a new instruction.
(ie, all existing fast-math flags are off). If you copy flags to an existing
instruction that has some flags on already, you will not necessarily turn them off
as expected.
I uncovered this bug while trying to implement a fix for PR20802.
llvm-svn: 216939
Diffstat (limited to 'llvm/lib/Analysis/MemoryDependenceAnalysis.cpp')
0 files changed, 0 insertions, 0 deletions
