r229204 - CodeGen: Correctly convert atomic bool from i8 to i1

Ulrich Weigand Ulrich.Weigand at de.ibm.com
Tue Feb 17 10:25:14 PST 2015


David Majnemer wrote:

>+// CHECK: %[[load:.*]] = load atomic i8* @b seq_cst, align 1

This test now fails on SystemZ, because all global variables are aligned to
at least 2, and thus the "align 1" doesn't match.

Is there a reason the test needs to verify the alignment?  If so, we'd have
to do something like in the c-strings.c test case.  Otherwise, it would be
simplest to just remove the check for "align 1".


Mit freundlichen Gruessen / Best Regards

Ulrich Weigand

--
  Dr. Ulrich Weigand | Phone: +49-7031/16-3727
  STSM, GNU/Linux compilers and toolchain
  IBM Deutschland Research & Development GmbH
  Vorsitzende des Aufsichtsrats: Martina Koederitz | Geschäftsführung: Dirk
Wittkopp
  Sitz der Gesellschaft: Böblingen | Registergericht: Amtsgericht
Stuttgart, HRB 243294





More information about the cfe-commits mailing list