Re: [Diffusion] rL246882: Don't crash on a self-alias declaration

2015-11-11 Thread Tom Stellard via cfe-commits
tstellarAMD added a project: 3.7.1-merged. Users: hfinkel (Author, Auditor) 3.7-release (Auditor) cfe-commits (Auditor) tstellarAMD (Auditor) rjmccall (Auditor) http://reviews.llvm.org/rL246882 ___ cfe-commits mailing list cfe-commits@lists

Re: [Diffusion] rL246882: Don't crash on a self-alias declaration

2015-11-11 Thread Tom Stellard via cfe-commits
tstellarAMD added a subscriber: tstellarAMD. tstellarAMD accepted this commit. tstellarAMD added a comment. r252808 Users: hfinkel (Author, Auditor) 3.7-release (Auditor) cfe-commits (Auditor) tstellarAMD (Auditor) rjmccall (Auditor) http://reviews.llvm.org/rL246882 ___

Re: [Diffusion] rL246882: Don't crash on a self-alias declaration

2015-11-10 Thread John McCall via cfe-commits
rjmccall added a subscriber: rjmccall. rjmccall added a comment. Hmm. That cast to GlobalAlias seems quite brittle anyway; it would be good to fix it (and diagnose the fact that forming the alias failed). But this change seems independently useful. Approved for 3.7. Users: hfinkel (Author

Re: [Diffusion] rL246882: Don't crash on a self-alias declaration

2015-11-09 Thread hfin...@anl.gov via cfe-commits
hfinkel added auditors: rjmccall. hfinkel added a comment. John is the right auditor for this, not me. Users: hfinkel (Author, Auditor) 3.7-release (Auditor) cfe-commits (Auditor) tstellarAMD (Auditor) rjmccall (Auditor) http://reviews.llvm.org/rL246882

Re: [Diffusion] rL246882: Don't crash on a self-alias declaration

2015-11-04 Thread Dimitry Andric via cfe-commits
dim added a subscriber: dim. dim added auditors: 3.7-release, cfe-commits, tstellarAMD, hfinkel. dim added a comment. This got reported again in PR25397, and occurs during compilation of mpich. Looks like a good candidate for 3.7.1. Users: hfinkel (Author, Auditor) 3.7-release (Auditor)