On Fri, Nov 1, 2013 at 6:57 PM, Dmitri Gribenko <[email protected]> wrote: > On Fri, Nov 1, 2013 at 6:47 PM, Dmitri Gribenko <[email protected]> wrote: >> On Thu, Oct 31, 2013 at 6:47 PM, Richard Smith <[email protected]> wrote: >>> I had a go at writing a test for this (attached). However... my test fails >>> both with and without your change. >> >> Indeed. Seems like it fails because: >> - recursive_visibility_a.inner is not imported (only recursive_visibility_a >> is), >> - thus the 'inner' submodule is not showing up in any of the import lists, >> - and because of this getExportedModules() is not returning the >> correct module set -- it only considers modules that are imported. > > Oh, and this means that this modification to your testcase: > > -@import recursive_visibility_a; > +@import recursive_visibility_a.inner; > > Actually produces a testcase that used to fail, but now passes.
I have found a way to handle the original testcase as well. Please take a look: http://llvm-reviews.chandlerc.com/D2100 Dmitri -- main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if (j){printf("%d\n",i);}}} /*Dmitri Gribenko <[email protected]>*/ _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
