rengolin added inline comments.

================
Comment at: lib/Basic/Targets.cpp:4456
@@ -4455,3 +4455,1 @@
 
-    if (ArchVersion < 6  || 
-       (ArchVersion == 6 && ArchProfile == llvm::ARM::PK_M))
----------------
labrinea wrote:
> rengolin wrote:
> > Why is this not necessary any more?
> The logic is still necessary for the back-end to emit aligned loads and 
> stores, but it should be moved to the Driver, where +strict-align is set for 
> all arch+OS combinations as shown below.
Oh, so this ended up as a work-around for the correct change that you just did, 
below?


http://reviews.llvm.org/D13217



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to