[PATCH] D15630: [WinEH] Update LCSSA to handle catchswitch with handlers inside and outside a loop

Reid Kleckner via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 17 17:54:11 PST 2015


rnk added inline comments.

================
Comment at: test/CodeGen/X86/catch-lcssa.ll:1
@@ +1,2 @@
+; RUN: opt -O2 -lcssa -S < %s | FileCheck %s
+
----------------
andrew.w.kaylor wrote:
> mzolotukhin wrote:
> > majnemer wrote:
> > > Can we move this test to "test/Transforms/LCSSA" and just use "opt -lcssa -S" ?
> > Also, could you please run "opt -instnamer" on the test?
> The reason I put it here was so that it wouldn't choke the non-X86 buildbots.  I know I could add a REQUIRES line to the test, but it doesn't seem like that is the typical way this is handled.
> 
> I'll see if I can strip it down enough to work without referencing anything X86-specific.
You can use an x86 target triple in opt without the x86 backend just fine. There should be other examples of this.


Repository:
  rL LLVM

http://reviews.llvm.org/D15630





More information about the llvm-commits mailing list