[PATCH] Teach SimplifyCFG about address spaces

Quentin Colombet qcolombet at apple.com
Wed Oct 16 11:56:49 PDT 2013


  Hi Matt,

  Thanks for your comments.

  LGTM.

  -Quentin


================
Comment at: test/Transforms/SimplifyCFG/switch_create.ll:4
@@ -2,1 +3,3 @@
+
+; TODO: Other tests should also have check lines with datalayout
 
----------------
Matt Arsenault wrote:
> Quentin Colombet wrote:
> > When do you plan to do that?
> I've been waiting for my allow multiple prefix FileCheck patch to be committed to fix things like this.
Fair enough, just make sure you update the test case when the other patch lands.


http://llvm-reviews.chandlerc.com/D1602



More information about the llvm-commits mailing list