https://llvm.org/bugs/show_bug.cgi?id=27490

listm...@philipreames.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #8 from listm...@philipreames.com ---
A fixed version of this change landed as:
Sending        lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp
Sending        test/Transforms/InstCombine/atomic.ll
Transmitting file data ..
Committed revision 268809

For context, it's interesting to note that the case we were crashing in was
actually a miscompile without this change.  We happily rewrote the types of the
volatile store (which my assert triggered on), but dropped the atomicity and
volatility when doing so.  I added test cases to prevent us getting this wrong
in the future.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

Reply via email to