<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Jun 17, 2010, at 1:54 PM, Nico Weber wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div>the attached patch might fix PR7390.</div><div><br></div><div>Random trivia: GCC happily compiles `SEL a; @selector(foo) = a;`.</div> <div></div></blockquote></div><br><div>Fariborz, I saw that you just committed <a href="http://llvm.org/viewvc/llvm-project?view=rev&revision=106235">http://llvm.org/viewvc/llvm-project?view=rev&revision=106235</a> for the bug. With your patch, clang now treats @selectors() as lvalues and compiles <span class="Apple-style-span" style="font-family: arial, sans-serif; font-size: 13px; border-collapse: collapse; ">`SEL a; @selector(foo) = a;` without complaint. I think that's wrong.</span></div></blockquote><div><br class="webkit-block-placeholder"></div>Sorry, our patches crossed path. I will take a look at this regression.</div><div><br class="webkit-block-placeholder"></div><div>- Fariborz</div><div><br><blockquote type="cite"> <div><span class="Apple-style-span" style="font-family: arial, sans-serif; font-size: 13px; border-collapse: collapse; "><br></span></div><div><span class="Apple-style-span" style="font-family: arial, sans-serif; font-size: 13px; border-collapse: collapse; ">Nico</span></div> <meta charset="utf-8"></blockquote></div><br></body></html>