Auto merge of #21364 - cmr:fix-ttseq-ice, r=alexcrichton

Closes #21350

src/libsyntax/ext/tt/macro_rules.rs

@@ -281,19 +281,22 @@ pub fn compile<'cx>(cx: &'cx mut ExtCtxt,
 }
 
 fn check_lhs_nt_follows(cx: &mut ExtCtxt, lhs: &NamedMatch, sp: Span) {
-    // lhs is going to be like MatchedNonterminal(NtTT(TtDelimited(...))), where
+    // lhs is going to be like MatchedNonterminal(NtTT(TtDelimited(...))), where the entire lhs is
-    // the entire lhs is those tts.
+    // those tts. Or, it can be a "bare sequence", not wrapped in parens.
-    // if ever we get box/deref patterns, this could turn into an `if let
+    match lhs {
-    // &MatchedNonterminal(NtTT(box TtDelimited(...))) = lhs`
-    let matcher = match lhs {
         &MatchedNonterminal(NtTT(ref inner)) => match &**inner {
-            &TtDelimited(_, ref tts) => tts.tts.as_slice(),
+            &TtDelimited(_, ref tts) => {
-            _ => cx.span_bug(sp, "wrong-structured lhs for follow check")
+                check_matcher(cx, tts.tts.iter(), &Eof);
             },
-        _ => cx.span_bug(sp, "wrong-structured lhs for follow check")
+            tt @ &TtSequence(..) => {
+                check_matcher(cx, Some(tt).into_iter(), &Eof);
+            },
+            _ => cx.span_bug(sp, "wrong-structured lhs for follow check (didn't find \
+            a TtDelimited or TtSequence)")
+        },
+        _ => cx.span_bug(sp, "wrong-structured lhs for follow check (didn't find a \
+           MatchedNonterminal)")
     };
-
-    check_matcher(cx, matcher.iter(), &Eof);
     // we don't abort on errors on rejection, the driver will do that for us
     // after parsing/expansion. we can report every error in every macro this way.
 }

src/test/run-pass/issue-21350.rs

@@ -0,0 +1,17 @@
+// Copyright 2015 The Rust Project Developers. See the COPYRIGHT
+// file at the top-level directory of this distribution and at
+// http://rust-lang.org/COPYRIGHT.
+//
+// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
+// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
+// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
+// option. This file may not be copied, modified, or distributed
+// except according to those terms.
+
+// Make sure that "bare sequences" don't ICE in follow checking
+
+macro_rules! bare {
+    $($id:expr),+ => ( $($id)+ )
+}
+
+fn main() { }